All dependencies must be prefixed by project name, So please apply new {project}:{module} format for dependencies in info.yml file

It is supported since 8.0 and 7.40 Change Record: Project namespaces can now be added for module dependencies, and is now a Best Practice Define project dependencies in core module .info files).

It is useful for Installation profiles to work well. So sooner is better.
Hope this helps.

CommentFileSizeAuthor
#2 Update_dependencies_format_in_info.yml_3005134-2.patch1.2 KBchipway

Comments

chipway created an issue. See original summary.

chipway’s picture

chipway commented
Assigned: chipway » Unassigned
Status: Needs work » Needs review
StatusFileSize
new Update_dependencies_format_in_info.yml_3005134-2.patch1.2 KB

Here is the patch. Please review.

sean_e_dietrich’s picture

sean_e_dietrich
he/him
Primary language English
Location Sacramento, CA
commented
Status: Needs review » Needs work

Patch applies but I can't find the ggroup module as part of the group project. Marking module back as Needs review.

chipway’s picture

chipway commented

Thanks Sean for review.

In fact, may be, @axelm should remove ggroup from dependencies, as https://www.drupal.org/project/group/issues/2736233 (Port Subgroup (ggroup) to the D8 version) still needs work, or install patch via composer.json (but https://www.drupal.org/files/issues/2018-06-28/2736233-158.patch does not fix all).

Waiting for recommendation from @axelm.

chipway’s picture

chipway commented
Status: Needs work » Needs review
axelm’s picture

axelm
Primary language English
commented

Thanks
It will be fixed in the next release

axelm’s picture

axelm
Primary language English
commented
Status: Needs review » Patch (to be ported)
axelm’s picture

axelm
Primary language English
commented
Status: Patch (to be ported) » Fixed
axelm’s picture

axelm
Primary language English
commented
Status: Fixed » Closed (fixed)