Page MenuHomeDevCentral

Solve Composer conflict dependency
ClosedPublic

Authored by dereckson on Oct 17 2017, 13:25.
Tags
None
Referenced Files
F3768146: D1061.diff
Sat, Nov 23, 06:48
Unknown Object (File)
Sat, Nov 9, 15:08
Unknown Object (File)
Wed, Nov 6, 01:54
Unknown Object (File)
Sat, Nov 2, 18:00
Unknown Object (File)
Fri, Nov 1, 16:01
Unknown Object (File)
Thu, Oct 31, 15:11
Unknown Object (File)
Mon, Oct 28, 16:27
Unknown Object (File)
Sat, Oct 26, 13:06
Subscribers
None

Details

Summary

Conflict:

  • Phan 0.9.5 introduced a dependency to nikic/php-parser, in 3.1.1
  • psysh 0.7 requires the same dependency, but in version 1.2.2-2.1.1
  • We currently use Laravel 5.2, which requires psysh 0.6 or 0.7
  • We need psysh 0.8+ to solve this dependency issue
  • Laravel 5.3+ is happy with psysh 0.8+

Plan is so to force the last version of Phan before the dependnecy
to nikic/php-parser, ie 0.9.4.

When the code will be upgraded to use Laravel 5.3+,
to bump the Phan version will then be possible.

The composer package has been renamed to phan/phan, so this change
also switches to this new vendor name.

Fixes T1258.

Test Plan
$ rm -rf vendor composer.lock
$ composer install

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Passed
Unit
No Test Coverage
Branch
master
Build Status
Buildable 1675
Build 1923: arc lint + arc unit