drupal-composer / drupal-packagist

:package: Drupal Packagist server to automatically build package information from projects on drupal.org
http://packagist.drupal-composer.org
28 stars 4 forks source link

Drush version of extensions may be too restrictive #5

Closed webflo closed 9 years ago

webflo commented 9 years ago

From @derhasi on January 13, 2015 8:32

When I look at registry_rebuild and drush_language, the version constraint for those extensions seem to be too restrictive, as this will not allow to have drush 7 (dev-master) fulfill the requirement.

Copied from original issue: webflo/packagist#10

webflo commented 9 years ago

The version constraints are correct but you need https://packagist.org/packages/drush/drush instead of the drush package definitions from our drupal-packagist. But drupal-packagist has a higher priority than packagist.org

webflo commented 9 years ago

Always require "drush/drush" instead of "drupal/drush"

webflo commented 9 years ago

From @derhasi on January 13, 2015 11:10

I know "drush/drush", but the given version constraint is too restrictive: drush/drush: 6.* and that does not allow to install dev-masteror 7.*.

webflo commented 9 years ago

From @derhasi on January 13, 2015 11:12

Where is that drush version comming from. I cannot find code for that.

webflo commented 9 years ago

From @derhasi on January 21, 2015 8:50

For reference: the drush version is set in https://github.com/drupal-composer/drupal-parse-composer/blob/58ebd43f71f1f23bb96052fa0f95e2d6cfc09273/src/Project.php#L108

webflo commented 9 years ago

We need different drush versions depending on the core version.

greg-1-anderson commented 9 years ago

I ran into this problem today. It would be good if Drush extensions were flagged to work with Drush 6 or later.

greg-1-anderson commented 9 years ago

I just added 8264eb so that Drush dev-master will match composer version ranges.

greg-1-anderson commented 9 years ago

c.f. PR above to change this.

webflo commented 9 years ago

Fixed by #21