git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <junkio@cox.net>
To: Christian Meder <chris@absolutegiganten.org>
Cc: git@vger.kernel.org
Subject: Re: Notes and questions while reading the documentation
Date: Wed, 05 Oct 2005 16:30:01 -0700	[thread overview]
Message-ID: <7v1x2zfsp2.fsf@assigned-by-dhcp.cox.net> (raw)
In-Reply-To: <1128549966.11363.29.camel@localhost> (Christian Meder's message of "Thu, 06 Oct 2005 00:06:06 +0200")

Christian Meder <chris@absolutegiganten.org> writes:

> * a lot of the manpages include something like "v0.1, June 2005" in the
> header; these versions tags are pretty obscure to interpret, timestamp
> when last edited ? version of the manpage ? version of git when the
> manpage was included ? maturity the author assigned to the content ?
> If these tags don't follow some sane schema they should be removed.

I think the original intent was the last modification datestamp
and the version of the documentation, but I agree it should be
removed.  I do not think they show on the HTML version, nor man
pages, although I have to admit that I haven't looked at
generated manpages for some time.

> * git-applymbox: -q for interactivity seems like a strange choice, ok I
> knew -i for interactive and -q for quiet but -q for interactive editing
> is _not_ really intuitive

Last night I felt the same way, and an rewrite [*1*] of
applymbox I am working on uses '-i' instead.  If users do not
object, I would vote for changing applymbox to use '-i' as well.

The user community consensus does not have to be unanimous, but
anybody who has linux kernel tree on kernel.org servers has a
veto on this, I should say.  It's the tool they use every day.

> * the usage of git, Git and GIT isn't consistent in the documentation.
> I'd vote for only using git.

Sounds sane.  What would we do if we need to start sentences with it?

> * git-clone says that http transport is not supported yet I used it to
> clone the git repo from kernel.org yesterday. Should the documentation
> get updated ?

Thanks for noticing.  Yes, now HTTP can handle both of the
trickier setups (packed, and alternates); credit goes to Daniel.

> * the manpage synopsises aren't consistent wrt command naming; it's "git
> commit" but "git-branch"; I guess all the manpages should reference
> their commands as "git-x" and not "git x"

Agreed.

Again, thanks for taking the time to do this.

[Footnote]

*1* Why rewrite?  One reason was I was afraid to break things
for Linus ;-).  And I wanted to add a bit more interactivity and
restartability.  The ultimate goal is to make 'git-rebase' and
'git-cherry-pick' faster and easier to use.  The idea is not to
always do 3-way merge, but essentailly feed format-patch output
(now it can do --stdout) into the new applymbox, and when patch
applies cleanly things will go faster, otherwise it will fall
back to 3-way merge behaviour.

  reply	other threads:[~2005-10-05 23:30 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-10-05 22:06 Notes and questions while reading the documentation Christian Meder
2005-10-05 23:30 ` Junio C Hamano [this message]
2005-10-10 21:22   ` Christian Meder

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=7v1x2zfsp2.fsf@assigned-by-dhcp.cox.net \
    --to=junkio@cox.net \
    --cc=chris@absolutegiganten.org \
    --cc=git@vger.kernel.org \
    /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).