Hello, I'd like to report a bug and a feature.

Let's start with the bug. When running on my local environment (Drush 4.5, Drush make 2.3) I get this fatal error:
- Unknown options: drush-make-update-default-url, php.
I don't know where it takes these options from and why it considers them "unknown". My fix - commented the whole check.

The feature is simple but helpful in sense of usability - no need to know the real vocabulary ID.
One can provide machine name instead. My fix - added additional check and loaded by machine name (god bless Drupal has a special function for this).

Sorry for mixing everything here - got no time for two issues.

CommentFileSizeAuthor
#1 drush-1395090-1.patch3.14 KBlex0r

Comments

lex0r’s picture

lex0r commented
StatusFileSize
new drush-1395090-1.patch3.14 KB

Patch added.

Daniel_KM’s picture

Daniel_KM commented

Hi,

Thanks for your patch. I committed your patch for use of machine name (I updated documentation too), but I need some time to investigate the drush bug. Unknown options are used with user CSV formats of import/export (see folder 'formats' of the module) and I'm currently creating a new one, so I can't remove for now.

Sincerely,

Daniel Berthereau
Infodoc & Knowledge management

Daniel_KM’s picture

Daniel_KM commented
Status: Active » Fixed

Hi,

In fact, the issue is related to the use of Drush 4.5 (I used Drush 4.4) and the merge with drush make.

The check "Unknown option" is only use to check your command line, for example if you use --delimitre in lieu of --delimiter.

I committed your patch, updated.

Sincerely,

Daniel Berthereau
Infodoc & Knowledge management

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.