From: Frank Lichtenheld <frank@lichtenheld.de>
To: Sven Verdoolaege <skimo@liacs.nl>
Cc: git@vger.kernel.org
Subject: Re: [PATCH 2/5] git-config: add --remote option for reading config from remote repo
Date: Fri, 4 May 2007 23:35:53 +0200 [thread overview]
Message-ID: <20070504213553.GC30324@planck.djpig.de> (raw)
In-Reply-To: <20070504211004.GA1719@liacs.nl>
On Fri, May 04, 2007 at 11:10:05PM +0200, Sven Verdoolaege wrote:
> On Fri, May 04, 2007 at 11:03:40PM +0200, Frank Lichtenheld wrote:
> >
> > Some comments on the documentation:
> >
> > On Fri, May 04, 2007 at 12:56:40PM +0200, Sven Verdoolaege wrote:
> > > +'git-config' [--system | --global | --remote=[<host>:]<directory ]
> > > + [type] name [value [value_regex]]
> >
> > maybe something more like
> >
> > 'git-config' [ scope ]
>
> Sounds reasonable, although I'd probably say [<scope>].
Note that this is inconsistent with the rest of the documentation
(e.g. it says [type], not [<type>]).
> > > +--remote=[<host>:]<directory
> > > + Use remote config instead of the repository .git/config.
> > > + Only available for reading options.
> >
> > Why did you add it to all options in the SYNOPSYS then?
>
> Because I wasn't thinking.
Another small error: You seem to be missing a '>' after each occourence of
'<directory'.
Gruesse,
--
Frank Lichtenheld <frank@lichtenheld.de>
www: http://www.djpig.de/
next prev parent reply other threads:[~2007-05-04 21:38 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-04 10:56 Initial support for cloning submodules Sven Verdoolaege
2007-05-04 10:56 ` [PATCH 1/5] Add dump-config Sven Verdoolaege
2007-05-04 10:56 ` [PATCH 2/5] git-config: add --remote option for reading config from remote repo Sven Verdoolaege
2007-05-04 21:03 ` Frank Lichtenheld
2007-05-04 21:10 ` Sven Verdoolaege
2007-05-04 21:35 ` Frank Lichtenheld [this message]
2007-05-04 10:56 ` [PATCH 3/5] http.h: make fill_active_slots a function pointer Sven Verdoolaege
2007-05-04 10:56 ` [PATCH 4/5] git-config: read remote config files over HTTP Sven Verdoolaege
2007-05-04 10:56 ` [PATCH 5/5] git-clone: add --submodules for cloning submodules Sven Verdoolaege
2007-05-04 22:52 ` Initial support " Junio C Hamano
2007-05-05 8:14 ` Sven Verdoolaege
2007-05-05 8:46 ` Junio C Hamano
2007-05-06 4:13 ` Alon Ziv
2007-05-18 19:33 ` Sven Verdoolaege
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=20070504213553.GC30324@planck.djpig.de \
--to=frank@lichtenheld.de \
--cc=git@vger.kernel.org \
--cc=skimo@liacs.nl \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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;
as well as URLs for NNTP newsgroup(s).