Page MenuHomeDevCentral

Require Laravel 5.2, but not Laravel 5.3
ClosedPublic

Authored by dereckson on Aug 31 2016, 15:35.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 9, 12:03
Unknown Object (File)
Sat, Nov 9, 12:03
Unknown Object (File)
Sat, Nov 9, 12:00
Unknown Object (File)
Sat, Nov 9, 10:51
Unknown Object (File)
Sat, Nov 9, 08:45
Unknown Object (File)
Sun, Oct 27, 05:35
Unknown Object (File)
Sun, Oct 20, 16:58
Unknown Object (File)
Oct 8 2024, 20:34
Subscribers

Details

Summary

The Docker Image doesn't build, as at artisan optimize step,
a fatal error occurs, as our EventServiceProvider::boot() isn't
compatible anymore with parent class signature.

This is caused by two factors:

  • upstream: commit fa9f3220 made the class boot method now use facades instead of dependency injection to the method
  • locally: commit d69820ed127c required Laravel 5.2.41+ to get session code update, but didn't guard agaisnt 5.3.*

We require 5.2.*, using ~ and not ^ in Composer version syntax.
The caret operator is more appropraited when semver is better followed.

References:

This is a follow-up for d69820ed127c.
Fixes T997.

Test Plan

Ensure composer install latest 5.2:

$ rm -rf vendor/
$ composer install

  - Installing laravel/framework (v5.2.39)
    Loading from cache

Diff Detail

Repository
rGROVE Auth Grove
Lint
Lint Passed
Unit
No Test Coverage
Branch
master

Event Timeline

dereckson retitled this revision from to Require Laravel 5.2, but not Laravel 5.3.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: Sandlayth.
Sandlayth edited edge metadata.
This revision is now accepted and ready to land.Aug 31 2016, 16:27
This revision was automatically updated to reflect the committed changes.