Closed (fixed)
Project:
Drush
Version:
All-versions-4.x-dev
Component:
PM (dl, en, up ...)
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
25 Mar 2011 at 18:53 UTC
Updated:
20 Jun 2011 at 01:41 UTC
Jump to comment: Most recent file
Comments
Comment #1
moshe weitzman commentedhow does drupal core handle newly added dependencies? this is sort of tricky.
Comment #2
greg.1.anderson commentedAs a workaround, how about
drush dis modulename && drush en modulename?Comment #3
damienmckennaWhat if disabling a module brings a huge array of consequences?
Comment #4
moshe weitzman commentedNone of those consequences should entail data loss (thats what uninstall is for) - but I agree. Not a wise path.
How about we enhance pm-enable such that it works for already enabled extensions. Namely, it actually calls module_enable() and thus any new dependencies as enabled. would work on D7 at least.
Comment #5
jonhattanThat's one small step for a man, ...
example:
Comment #6
jonhattanpolished
Comment #7
greg.1.anderson commentedWell done.
Comment #8
jonhattanCommitted. Perhaps Mark wants this in 4.x
Comment #9
jonhattanComment #10
msonnabaum commentedI really like this patch, but I'm on the fence about whether this is safe to backport because it does change behavior.
I could use a second opinion on this if anyone else has a different take.
Comment #11
greg.1.anderson commentedI think it is okay to backport; it extends old behavior, should not break any existing scripts or code.
Comment #12
msonnabaum commentedk, backported.