public inbox for git@vger.kernel.org 
 help / color / mirror / Atom feed
From: Jonathan Nieder <jrnieder@gmail•com>
To: Thomas Ackermann <th.acker@arcor•de>
Cc: gitster@pobox•com, git@vger•kernel.org, davvid@gmail•com,
	Fredrik Gustafsson <iveqy@iveqy•com>,
	Jens Lehmann <Jens.Lehmann@web•de>
Subject: Re: [PATCH 2/6] fixup! fixup! fixup! Change 'git' to 'Git' whenever the whole system is referred to #1
Date: Fri, 1 Feb 2013 11:43:08 -0800	[thread overview]
Message-ID: <20130201194308.GC12368@google.com> (raw)
In-Reply-To: <242116275.1090080.1359742076320.JavaMail.ngmail@webmail18.arcor-online.net>

Hi,

Thomas Ackermann wrote:

> Found by Junio:
> Change git-dir to $GIT_DIR and git-file to gitfile.
>
> Signed-off-by: Thomas Ackermann <th.acker@arcor•de>
> ---
>  Documentation/git-rev-parse.txt | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/git-rev-parse.txt b/Documentation/git-rev-parse.txt
> index c743469..14386ed 100644
> --- a/Documentation/git-rev-parse.txt
> +++ b/Documentation/git-rev-parse.txt
> @@ -187,9 +187,9 @@ print a message to stderr and exit with nonzero status.
>  	Flags and parameters to be parsed.
>  
>  --resolve-git-dir <path>::
> -	Check if <path> is a valid git-dir or a git-file pointing to a valid
> -	git-dir. If <path> is a valid git-dir the resolved path to git-dir will
> -	be printed.
> +	Check if <path> is a valid `$GIT_DIR` or a gitfile pointing to a valid
> +	`$GIT_DIR`. If <path> is a valid `$GIT_DIR` the resolved path to `$GIT_DIR`
> +	will be printed.

Hm, I don't find the old or the new version very easy to understand.  Perhaps the
idea is something like this?

	Check if <path> is a valid git repository (.git or <project>.git
	directory) or "gitdir:" file.  If <path> is a "gitdir:" file
	then the resolved path to the corresponding real git repository
	will be printed.

Thanks,
Jonathan

  reply	other threads:[~2013-02-01 19:43 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-20 20:16 Aw: Re: [PATCH 1/2] Change old system name 'GIT' to 'Git' Thomas Ackermann
2013-01-20 21:08 ` Junio C Hamano
2013-01-21 18:36   ` [PATCH v2 0/6] GIT, Git, git Thomas Ackermann
2013-01-21 18:38     ` [PATCH v2 1/6] Change old system name 'GIT' to 'Git' Thomas Ackermann
2013-01-21 18:40     ` [PATCH v2 2/6] Change 'git' to 'Git' whenever the whole system is referred to #1 Thomas Ackermann
2013-01-21 18:51     ` Aw: [PATCH v2 0/6] GIT, Git, git Thomas Ackermann
2013-01-21 19:12     ` [PATCH v3 " Thomas Ackermann
2013-01-21 19:16       ` [PATCH v3 1/6] Change old system name 'GIT' to 'Git' Thomas Ackermann
2013-01-22  0:39         ` Junio C Hamano
2013-01-22  9:44           ` Aw: " Thomas Ackermann
2013-01-22 16:45             ` Junio C Hamano
2013-01-21 19:17       ` [PATCH v3 2/6] Change 'git' to 'Git' whenever the whole system is referred to #1 Thomas Ackermann
2013-01-22  0:41         ` Junio C Hamano
2013-01-22 23:48         ` Junio C Hamano
2013-01-23  0:19           ` Junio C Hamano
2013-01-23  2:24             ` Junio C Hamano
2013-01-23  2:32             ` Junio C Hamano
2013-01-21 19:19       ` [PATCH v3 3/6] Change 'git' to 'Git' whenever the whole system is referred to #2 Thomas Ackermann
2013-01-23  0:53         ` Junio C Hamano
2013-01-23  2:48           ` Junio C Hamano
2013-01-23  3:44         ` David Aguilar
2013-01-23  4:17           ` Junio C Hamano
2013-01-23  8:58             ` Aw: " Thomas Ackermann
2013-02-01  0:46               ` Junio C Hamano
2013-02-01 18:04                 ` Aw: " Thomas Ackermann
2013-02-01 18:06                   ` [PATCH 1/6] fixup! fixup! " Thomas Ackermann
2013-02-01 18:07                   ` [PATCH 2/6] fixup! fixup! fixup! Change 'git' to 'Git' whenever the whole system is referred to #1 Thomas Ackermann
2013-02-01 19:43                     ` Jonathan Nieder [this message]
2013-02-01 20:16                       ` Junio C Hamano
2013-02-01 18:08                   ` [PATCH 3/6] fixup! Documentation: avoid poor-man's small caps Thomas Ackermann
2013-02-01 18:44                     ` Junio C Hamano
2013-02-01 18:10                   ` [PATCH 4/6] Fix places where 'Git' should be 'git Thomas Ackermann
2013-02-01 18:11                   ` [PATCH 5/6] Add a description for 'gitfile' to glossary Thomas Ackermann
2013-02-01 18:41                     ` Junio C Hamano
2013-02-01 19:26                       ` Junio C Hamano
2013-02-01 19:46                         ` Jonathan Nieder
2013-02-01 18:12                   ` [PATCH 6/6] Use consistent links for User Manual and Everyday Git; Fix a quoting error Thomas Ackermann
2013-02-01 18:43                     ` Junio C Hamano
2013-02-01 19:51                     ` Jonathan Nieder
2013-02-01 20:17                       ` Junio C Hamano
2013-02-01 20:25                       ` Aw: " Thomas Ackermann
2013-02-01 18:37                   ` Aw: Re: Aw: Re: [PATCH v3 3/6] Change 'git' to 'Git' whenever the whole system is referred to #2 Junio C Hamano
2013-01-21 19:21       ` [PATCH v3 4/6] Change 'git' to 'Git' whenever the whole system is referred to #3 Thomas Ackermann
2013-01-23  2:49         ` Junio C Hamano
2013-01-21 19:22       ` [PATCH v3 5/6] Change 'git' to 'Git' whenever the whole system is referred to #4 Thomas Ackermann
2013-01-23  3:16         ` Junio C Hamano
2013-01-21 19:24       ` [PATCH v3 6/6] Add rule for when to use 'git' and when to use 'Git' Thomas Ackermann
2013-01-21 19:56     ` [PATCH v2 0/6] GIT, Git, git Junio C Hamano
2013-01-21 20:01       ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130201194308.GC12368@google.com \
    --to=jrnieder@gmail$(echo .)com \
    --cc=Jens.Lehmann@web$(echo .)de \
    --cc=davvid@gmail$(echo .)com \
    --cc=git@vger$(echo .)kernel.org \
    --cc=gitster@pobox$(echo .)com \
    --cc=iveqy@iveqy$(echo .)com \
    --cc=th.acker@arcor$(echo .)de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox