For some reason the help text is "path: A space separated list of project names." in the drush help for devel-reinstall.

"project names: A space separated list of project names." is easier to understand. I attached a patch that makes that small change. It works on both D6,7, and 8.

CommentFileSizeAuthor
path2projects.patch498 bytesnaxoc

Comments

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

Committed to 6/7/8. Thanks.

salvis’s picture

salvis
he/his
commented

I believe it ought to be "space-separated" (with a hyphen), no?

moshe weitzman’s picture

moshe weitzman commented

@salvis - that does not look right to me. i think those are real, distinct words that need no hyphen.

salvis’s picture

salvis
he/his
commented

Isn't this a hyphenated compound adjective?

I find

    "A space separated list of project names."

hard to read and awfully close to

    "A space separated lists of project names."

which is something completely different (a full subject-verb-object sentence in past tense).

Read the two out loud until you feel the differences. Then compare it to

    "A space-separated list of project names."

Disambiguating this early on is the purpose of the hyphen.

moshe weitzman’s picture

moshe weitzman commented
Status: Fixed » Active

Well, thats quite a thoughtful answer. You are also a master of clear English. Impressive. Sure, lets add the replace the space with a hyphen.

salvis’s picture

salvis
he/his
commented
Status: Active » Fixed

Committed to 8/7/6. Please forgive me for insisting on this nit...

naxoc’s picture

naxoc commented

You are so right Salvis! I am not a native English speaker but I totally would have done the same had this been Danish. Nice catch!

Status: Fixed » Closed (fixed)

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