Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hey folks,
Using Mailhandler in a profile, and I'm finding that it won't install because of the following line in the info file:
dependencies[] = system (>=7.8)
While this is documented in the info file creation guide, it seems that at install, version # isn't handled. See attached screenshot for error (Being run on Drupal 7.19) While this is likely a core bug, in the mean time, I'm attaching a patch that removes the requirement and adds a hook_requirements condition on install for the drupal version to be greater than or equal to 7.8. Patch below.
Comment | File | Size | Author |
---|---|---|---|
#3 | panopoly_core_install_load_profile-1961780-3.patch | 1.02 KB | Pancho |
#1 | mailhandler-version-requirement-1961780-1.patch | 1.15 KB | wbobeirne |
Screen shot 2013-04-04 at 5.52.53 PM.png | 27.1 KB | wbobeirne |
Comments
Comment #1
wbobeirne CreditAttribution: wbobeirne commentedComment #2
PanchoHit this bug, too, when trying to install the first Alpha of Open Atrium 2.
Definitely looks like a core bug, so IMHO it should be rightaway filed and fixed there, not worked around here.
Comment #3
PanchoNope, this is no core bug. Core doesn't check any second-level requirements, see: #1253774: Dependencies of dependencies are ignored by installation profiles (and by test setUp methods).
Rather this is a Panopoly bug:
panopoly_core_install_load_profile() goes to some lengths filling the gap. While the required versions aren't processed, this is absolutely acceptable. However the version information needs to be stripped off the dependency string, otherwise the Drupal installer will be looking for a module called "system (>=7.8)" which is obviously wrong.
Enclosed patch should fix this correctly. Please test.
Comment #4
populist CreditAttribution: populist commentedThanks so much for this patch. Reviewing now.
Comment #5
populist CreditAttribution: populist commentedLooking good. Tested and committed to dev.