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)
Tue, Apr 23, 01:49
Unknown Object (File)
Sun, Apr 21, 16:44
Unknown Object (File)
Sun, Apr 21, 09:49
Unknown Object (File)
Wed, Apr 17, 23:53
Unknown Object (File)
Wed, Apr 17, 22:24
Unknown Object (File)
Sun, Apr 14, 05:02
Unknown Object (File)
Sat, Apr 13, 20:00
Unknown Object (File)
Fri, Apr 12, 08:02
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 Not Applicable
Unit
Tests Not Applicable

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.