Posted by greg.1.anderson on November 7, 2009 at 7:07am
Jump to:
| Project: | Drush |
| Version: | All-versions-5.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
After my comment in #586466: Drush.php is no longer directly executable (can interfere with backend invoke), it occurred to me that it would be possible to just alias 'sql sync' to also be 'sql load'. This works because, despite the fact that it uses quite a different mechanism, sql sync is command-line compatible with sql load, and should be a drop-in replacement for the removed function. This was by design from the beginning, but as sql sync became more capable, I sort of forgot that it was compatible. A quick review showed that it still is.
This patch adds 'sql load' as an alias for 'sql sync', and also removes the 'sql load' command definitions that were left behind when the 'sql load' implementation was backed out.
| Attachment | Size |
|---|---|
| drush-HEAD-2009_11_06.patch | 3.57 KB |
Comments
#1
I committed this, without the alias to sql load. i think we are just confusing things long term by having two long names for the same command. at upgrade time, a few people will have to edit their scripts. no biggie.
however, this did uncover a bigger problem. the 2 options that print when drush_drupal_major_version() >= 7 are not printing on a d7 site. it looks like that function does not work properly anymore, at least during help command. possible related to sitealias changes, since drush_drupal_version() does not know where drupal_root is.
#2
Automatically closed -- issue fixed for 2 weeks with no activity.