drupal-composer / drupal-parse-composer

:mag: Components used in Drupal-Packagist to parse package information from drupal.org
10 stars 11 forks source link

Add core Drupal 8 modules, themes, profiles to core components list #50

Closed hussainweb closed 8 years ago

hussainweb commented 8 years ago

This change adds the list of modules, themes, and profiles from Drupal 8 (specifically Drupal 8.1.x-dev) to the info file parser. This allows correctly setting version information if any contrib module lists one of these core components as dependencies with version constraint.

webflo commented 8 years ago

Sorry for the trouble, i tried to keep this list of the version parser for D8 so far. Because we have multiple branches and modules move to core every few month. This means we have to recompile the package definitions for all packages after we change the list.

I added support for 3-digit version constrains in https://github.com/drupal-composer/drupal-parse-composer/pull/49/files

We could add a special case for system module or change the version requirement in migrate_plus and migrate_update. (I looked it up in the database, these are the only to module with this problem).

hussainweb commented 8 years ago

Thank you for the quick reply. I understand you have to recompile the package informations.

I don't understand how support for 3 digit version constraints fix this specific issue. Also, these two might be the only modules now but won't there be more soon? I understand that there are new modules going into the core at all times, but it is not very likely that contrib modules would depend on a specific version. I have only included the whole lst of modules for completeness but honestly, I don't think even 10 of these would be used. The most common one used could be system, followed by modules like field, etc...

webflo commented 8 years ago

Lets reduce the list to system for now. Please extend the test case in InfoFileSpec.php? Thanks!

hussainweb commented 8 years ago

@webflo, is this what you meant?

webflo commented 8 years ago

Looks good.