Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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.