git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Björn Steinbrink" <B.Steinbrink@gmx.de>
To: David Abrahams <dave@boostpro.com>
Cc: Jeff King <peff@peff.net>, Michael Witten <mfwitten@gmail.com>,
	"J. Bruce Fields" <bfields@fieldses.org>,
	git@vger.kernel.org
Subject: Re: [doc] User Manual Suggestion
Date: Sat, 25 Apr 2009 00:45:17 +0200	[thread overview]
Message-ID: <20090424224517.GA10155@atjola.homenet> (raw)
In-Reply-To: <2B5084A3-9BDB-4463-8530-3C8AB2E09A1F@boostpro.com>

On 2009.04.24 17:06:27 -0400, David Abrahams wrote:
> On Apr 24, 2009, at 4:24 PM, Jeff King wrote:
>> On Fri, Apr 24, 2009 at 03:00:19PM -0400, David Abrahams wrote:
>>> It would even help a lot if the plumbing were all spelled "git-xxx"
>>> and the high level stuff were "git xxx."
>>
>> Differentating calling conventions like that was proposed when dashed
>> forms were deprecated and removed from the PATH. But if we had dashed
>> forms for plumbing (i.e., not forwarding them via the "git" wrapper),
>> then you have to do one of:
>>
>>  - put them in the user's PATH. Now tab completion or looking in your
>>    PATH means you see _just_ the plumbing commands, and none of the
>>    high level ones. Which is one of the reasons they were removed
>>    from the PATH in the first place (due to numerous user
>>    complaints).
>>
>>  - put them elsewhere, and force plumbing users to add $GIT_EXEC_PATH
>>    to their PATH. That becomes very annoying for casual plumbing
>>    users. If you come to the mailing list with a problem, I would
>>    have to jump through extra hoops to ask you to show me the output
>>    of "git ls-files".
>
> I see your point.
>
>   llgit xxx
>
> ?

If that was the exclusive way of calling the low-level commands, that
would still break existing scripts. And if you keep e.g. "git
write-tree" and just add "llgit write-tree" as an alias, that will IMHO
just cause more confusion once old and new git users meet. And I agree
with Peff, it's not important whether it's "git foo", "llgit foo", "git
lowlevel foo" or something else. It's just about how much your users
really _need_ to know and how you tell them to use the stuff.

> I think UI/API works way better than porcelain/plumbing. We are, after
> all, programmers.

We are programmers, but not all git users are programmers.

> It would also be good to link to a definition any time you use a term
> of art in the docs. I would even do that in the case of UI/API since
> the distinction could appear to be subtle.
>
> I should also say, most of the docs and interfaces I see in Git (and
> its wrappers, web interfaces, etc.) give the SHA1 hashes way too much
> exposure. The times when it's actually more convenient to use a hash
> instead of one of the other notations are rare,

How often do you need a name for a commit shown by a command and can
accept that it is not stable? I usually need a name because I
want to reference that commit later on, either because I need to talk to
other users, or because I'm working on something and might need to look
at that commit now and then, regardless on my current state of things.
One big exception in my workflow is when I use "git blame", then I
usually just need the name once to look at the full commit. But then I
prefer a 7-8 characters long sha-1 prefix to something like
improve_foo_speed~132^12~1^3. And "pseudo-stable" numbers have been
discussed to death.

> and if hashes weren't so exposed I bet most interfaces would make
> those other names more available. One reason I think hashes retain
> their prominent exposure is that you have no other reasonably stable
> way of referring to commits, since branch~NN counts backward from
> HEAD. Adding such a thing would help.

It counts backwards from "branch".

> Oh, one other specific issue: the rev-parse manpage uses $GIT_DIR
> without saying what it is. I *think* that means the root of the
> working copy and has nothing to do with environment variables, but
> it's hard to be sure, and if I'm right about that, it's misleading
> notation.

$GIT_DIR means the .git directory of a non-bare repo.

Björn

  reply	other threads:[~2009-04-24 22:47 UTC|newest]

Thread overview: 90+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-22 19:38 [doc] User Manual Suggestion David Abrahams
2009-04-23 17:57 ` J. Bruce Fields
2009-04-23 18:37   ` Michael Witten
2009-04-23 20:16     ` Jeff King
2009-04-23 20:45       ` Michael Witten
2009-04-23 21:31         ` David Abrahams
2009-04-24  0:31           ` Michael Witten
2009-04-24 14:18           ` Jeff King
2009-04-24 14:20             ` J. Bruce Fields
2009-04-24 17:28             ` David Abrahams
2009-04-24 18:15               ` Jeff King
2009-04-24 19:00                 ` David Abrahams
2009-04-24 20:24                   ` Jeff King
2009-04-24 21:06                     ` David Abrahams
2009-04-24 22:45                       ` Björn Steinbrink [this message]
2009-04-25  0:39                         ` David Abrahams
2009-04-26 23:35                           ` Björn Steinbrink
2009-04-24 14:11         ` Jeff King
2009-04-24 14:30           ` Michael Witten
2009-04-24 14:33             ` Michael Witten
2009-04-24 15:04             ` Jeff King
2009-04-24 15:18               ` Michael Witten
2009-04-24 17:38                 ` J. Bruce Fields
2009-04-24 18:27                   ` Jeff King
2009-04-24 18:35                     ` J. Bruce Fields
     [not found]                   ` <34BD51FF-0908-48A8-BBBC-E27B0EFB32E5@boostpro.com>
2009-04-24 18:52                     ` J. Bruce Fields
2009-04-25 10:35                       ` Felipe Contreras
2009-04-24 19:12                   ` Michael Witten
2009-04-23 21:26       ` David Abrahams
2009-04-23 22:51         ` Johan Herland
2009-04-24  0:30           ` Michael Witten
2009-04-24 20:30             ` Johan Herland
2009-04-24 21:34               ` Daniel Barkalow
2009-04-24 21:38                 ` Jeff King
2009-04-24 22:18                   ` Michael Witten
2009-04-24 22:25                     ` Michael Witten
2009-04-24 23:11                       ` Daniel Barkalow
2009-04-24 23:14                         ` Jeff King
2009-04-24 23:18                           ` Michael Witten
2009-04-24 23:31                           ` Michael Witten
2009-04-24 23:35                             ` Jeff King
2009-04-25  0:19                               ` Michael Witten
2009-04-25 10:18                           ` Felipe Contreras
2009-04-24 23:26                         ` Michael Witten
2009-04-25 18:55                           ` Daniel Barkalow
2009-04-25 19:16                             ` Michael Witten
2009-04-25 19:24                               ` Felipe Contreras
2009-04-25 19:36                                 ` David Abrahams
2009-04-25 20:53                                   ` Felipe Contreras
2009-04-26 11:28                                   ` Björn Steinbrink
2009-04-26 13:55                                     ` David Abrahams
2009-04-26 17:56                                       ` Björn Steinbrink
2009-04-26 20:17                                         ` David Abrahams
2009-04-26 22:25                                           ` Björn Steinbrink
2009-04-27  1:41                                             ` David Abrahams
2009-04-27 16:30                                             ` David Abrahams
2009-04-27 16:52                                               ` Michael Witten
2009-04-26 16:36                                     ` Michael Witten
2009-04-26 18:12                                       ` Björn Steinbrink
2009-04-26 20:20                                         ` David Abrahams
2009-04-25  0:41                         ` David Abrahams
2009-04-24 23:16                     ` Björn Steinbrink
2009-04-25  0:01                       ` Michael Witten
2009-04-25  0:48                         ` David Abrahams
2009-04-26 22:42                           ` Björn Steinbrink
2009-05-02 15:53                         ` Björn Steinbrink
2009-05-02 18:36                           ` Michael Witten
2009-05-02 21:11                             ` Björn Steinbrink
2009-05-02 23:13                               ` Michael Witten
2009-05-02 23:32                                 ` Björn Steinbrink
2009-05-03  1:10                                   ` Michael Witten
2009-05-03  1:48                                     ` Björn Steinbrink
2009-05-03  1:18                                 ` Mark Lodato
2009-05-03  1:26                                   ` Michael Witten
2009-04-24 23:21                   ` Daniel Barkalow
2009-04-24 23:25                     ` Jeff King
2009-04-26 23:41                       ` Björn Steinbrink
2009-04-24 23:29                     ` Michael Witten
2009-04-27  0:00                       ` Björn Steinbrink
2009-04-25  0:19                   ` David Abrahams
2009-04-25  0:26                     ` Michael Witten
2009-04-25  0:35                     ` Jeff King
2009-04-25  0:53                       ` David Abrahams
2009-04-29  6:34                         ` Jeff King
2009-04-29 13:27                           ` David Abrahams
2009-04-29 14:05                             ` Jeff King
2009-04-24  2:29     ` J. Bruce Fields
2009-04-24  2:34       ` Michael Witten
2009-04-24  4:06       ` David Abrahams
2009-04-24 14:10         ` J. Bruce Fields

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=20090424224517.GA10155@atjola.homenet \
    --to=b.steinbrink@gmx.de \
    --cc=bfields@fieldses.org \
    --cc=dave@boostpro.com \
    --cc=git@vger.kernel.org \
    --cc=mfwitten@gmail.com \
    --cc=peff@peff.net \
    /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).