Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Peg Symfony to version 2 in require-dev. #2799

Closed
wants to merge 1 commit into from

Conversation

greg-1-anderson
Copy link
Member

The global installation of Drush 9 is only unofficially supported. All the same, there is a problem that the global install does not support Drupal 8.3.x out-of-the-box due to the composer.json requirement of Symfony 3 components.

This experimental PR requires the older version of Symfony in the require-dev section. This makes the global install of Drush 9 work with Drupal 8.3.x and Drupal 8.4.x.

This will be ignored for site-local Drush, so the supported configurations will continue to work as before.

If this is committed at all, it should be backed out at some point in the future. Unsure if we should do this for master; we probably should do it for Drush 8.x, but it is more convenient to test with the sut on the master branch, so starting here first.

@weitzman
Copy link
Member

weitzman commented Jun 8, 2017

Drush 8.x is already restricted to Symfony2 - https://github.com/drush-ops/drush/blob/8.x/composer.json

@greg-1-anderson
Copy link
Member Author

This PR is for the master branch. The 8.x PR adds Symfony 3, but restricts the global install to Symfony 2. See: #2800 for 8.x.

@greg-1-anderson
Copy link
Member Author

The global install for Drush 9 will not support Drupal 8.3.x or earlier. Use Drush 8.x. for that.

Unsupported workaround:

$ mkdir my-global-drush
$ cd my-global-drush
$ composer init
$ composer config prefer-stable 1
$ composer config minimum-stability dev
$ composer require symfony/console:~2.7 drush/drush:~9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants