Referring to line 641 of /commands/core/core.drush.inc
// Hide 'Modules path' and 'Themes path' as well, because that's
// the way it's always been.

Why not have it show them if the --full parameter is passed, along with alias files?

It seems this would be more useful to show now that outputformat is a thing, and they can be parsed easily with --pipe (aka --format=ini)

CommentFileSizeAuthor
#3 drush-why-does-drush-status-1993784-3.patch2.2 KBgreg.1.anderson
#2 drush_status_show_paths_with_full-1993784-2.patch1.85 KBStephenBrown
drush_status_show_paths_with_full.patch1.51 KBStephenBrown

Comments

greg.1.anderson’s picture

greg.1.anderson commented
Status: Active » Needs work

Seems okay to me. The help text for --full will need to be updated to match the adjusted behavior.

StephenBrown’s picture

StephenBrown commented
StatusFileSize
new drush_status_show_paths_with_full-1993784-2.patch1.85 KB

Fixed the help text, and edited the relevant comment.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Needs work » Needs review
StatusFileSize
new drush-why-does-drush-status-1993784-3.patch2.2 KB

#2 applied, but with warnings (patch unexpectedly ends in middle of line). Here is a re-roll of the same patch, unchanged.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Needs review » Reviewed & tested by the community

I'm fine with this; looks like an improvement. Will commit in a couple of days if there are no comments (or someone else can commit).

moshe weitzman’s picture

moshe weitzman commented

Looks fine to me.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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