threads / patch / 19991

patchDocumentation: update description of shell aliases

Subject: [PATCH] Documentation: update description of shell aliases

## tl;dr

4 messages between Jul 1, 2009 and Jul 2, 2009. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

Sitaram Chamarty· Jul 1, 2009, 16:00 UTC · lore

Aliases that invoke shell commands start from the top-level directory, but this was not documented.

Signed-off-by: Sitaram Chamarty <sitaramc@gmail.com>
---
 Documentation/config.txt |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)
Show changes to Documentation/config.txt +3 −1
diff --git a/Documentation/config.txt b/Documentation/config.txt
index 2fecbe3..bf97e1d 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -456,7 +456,9 @@ If the alias expansion is prefixed with an
exclamation point,
 it will be treated as a shell command.  For example, defining
 "alias.new = !gitk --all --not ORIG_HEAD", the invocation
 "git new" is equivalent to running the shell command
-"gitk --all --not ORIG_HEAD".
+"gitk --all --not ORIG_HEAD".  Note that shell commands will be
+executed from the top-level directory of a repository, which may
+not necessarily be the current directory.

 apply.whitespace::
 	Tells 'git-apply' how to handle whitespaces, in the same way

-- 
1.6.3.3
Junio C Hamano· Jul 1, 2009, 18:21 UTC · re: Sitaram Chamarty · lore

Re: [PATCH] Documentation: update description of shell aliases

Sitaram Chamarty <sitaramc@gmail.com> writes:
> Aliases that invoke shell commands start from the top-level directory,
> but this was not documented.

Could you please also think if it is worth mentioning what happens when you do not even have a top-level directory? I.e. in a bare repository.

Sitaram Chamarty· Jul 2, 2009, 00:57 UTC · re: Junio C Hamano · lore

Re: [PATCH] Documentation: update description of shell aliases

On Wed, Jul 1, 2009 at 11:51 PM, Junio C Hamano<gitster@pobox.com> wrote:
Show 7 quoted lines
> Sitaram Chamarty <sitaramc@gmail.com> writes:
>
>> Aliases that invoke shell commands start from the top-level directory,
>> but this was not documented.
>
> Could you please also think if it is worth mentioning what happens when
> you do not even have a top-level directory?  I.e. in a bare repository.

Yes I thought about it. I decided not to add that because it is:

  - not common enough (how many new users fiddle in bare
    repos?)
  - not surprising enough (I did test it; it just uses the
    current -- bare -- directory, as even a new user would
    expect)
(In contrast, the part I added _was_ somewhat surprising.)

And if we go that far, why not also document what happens with various combinations of $PWD,$GIT_DIR and $GIT_WORK_TREE? I did not get a chance to test those, but if you think we should add those also, I will do so and document what happens.

I just think it may be a bit overkill, but if you think it's not, and it adds value, just say so and I will test and document it.

Andreas Ericsson· Jul 2, 2009, 07:21 UTC · re: Sitaram Chamarty · lore

Re: [PATCH] Documentation: update description of shell aliases

Sitaram Chamarty wrote:
Show 14 quoted lines
> On Wed, Jul 1, 2009 at 11:51 PM, Junio C Hamano<gitster@pobox.com> wrote:
>> Sitaram Chamarty <sitaramc@gmail.com> writes:
>>
>>> Aliases that invoke shell commands start from the top-level directory,
>>> but this was not documented.
>> Could you please also think if it is worth mentioning what happens when
>> you do not even have a top-level directory?  I.e. in a bare repository.
> 
> Yes I thought about it.  I decided not to add that because
> it is:
> 
>   - not common enough (how many new users fiddle in bare
>     repos?)
>

I do that quite a lot, but I can't recall ever using an alias in a bare repo. It would be odd if I did, as the only alias I've got is this:

   alias.wsfix=!git diff HEAD >P.diff && git reset --hard HEAD && git apply --whitespace=fix P.diff && rm -f P.diff

which isn't always doing the right thing even in a non-bare repository, and never in a bare one.

-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231

Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
on peace.

← back to recent threads