So that I do not forget. This is important for consistency.

CommentFileSizeAuthor
#6 self-alias-2.patch3.4 KBgreg.1.anderson
#3 self-alias.patch2.77 KBgreg.1.anderson

Comments

moshe weitzman’s picture

moshe weitzman commented

And @self should show up in sa command, no? It currently does not at AFAICT

greg.1.anderson’s picture

greg.1.anderson commented
Issue tags: +drush-3.0
greg.1.anderson’s picture

greg.1.anderson commented
Status: Active » Needs review
StatusFileSize
new self-alias.patch2.77 KB

Here is a patch that creates the alias "@self". Two notes.

  1. "current" was removed a while ago, so you won't see a reference to that in this patch.
  2. I don't think that @self should show up in the sa command, as @self represents an alias that is already in the list. If @self was included, then sa --pipe would show one site twice, which would be inconvenient.
moshe weitzman’s picture

moshe weitzman commented

I think it would help folks if @self shows up. That way we can write tutorials including sa command and have them make sense. We can filter it from --pipe.

greg.1.anderson’s picture

greg.1.anderson commented

Hm, actually, @self does show up in sa in patch #3, but only if some site is bootstrapped (after drush_bootstrap_max).

This patch also insures that only unique values will appear in the output of drush sa --pipe

greg.1.anderson’s picture

greg.1.anderson commented
StatusFileSize
new self-alias-2.patch3.4 KB

And now with patch...

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community
greg.1.anderson’s picture

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

Committed.

Status: Fixed » Closed (fixed)
Issue tags: -drush-3.0

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

Issue tags: +drush-3.0

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.