9front - general discussion about 9front
 help / color / mirror / Atom feed
From: unobe@cpan.org
To: 9front@9front.org
Subject: Re: [9front] [PATCH] git: add missing usages for subcommands
Date: Sun, 01 Oct 2023 16:34:00 -0700	[thread overview]
Message-ID: <A1FA07F302666A10482EAB62ED14EE37@smtp.pobox.com> (raw)
In-Reply-To: <D9543B955541EE3E81A5728709F8DF67@eigenstate.org>

Quoth ori@eigenstate.org:
> Quoth unobe@cpan.org:
> > Quoth ori@eigenstate.org:
> > > these were intentionally ommitted; you shouldn't need to use
> > > them. They exist for scripts like 'git/commit', 'git/push', and
> > > 'git/pull'.
> > 
> > Thanks, Ori.  I didn't realize that programs existing in the
> > distribution shouldn't be documented.
> >
> > Two follow-ups:
> > 
> > 1) Is that something that is standard and I've just never realized it?
> 
> Not sure about 'standard'; it's a judgement call.
> 
> I wouldn't mind having some form of documentation, but
> putting them in the main git manpage just confuses the
> user on how to do things, IMO.

What if, in the long-form area that expanded on the different git
programs, the ones which aren't meant to be used directly are
mentioned as such?  For me, some documentation of all programs is more
useful than none, even ones meant for internal-use only?  Something
like this at the end:
"Git/get, git/repack, git/save, and git/send are primarily internal
commands, and therefore not documented beyond their usage lines above
and their own code."

> It's the same reason that debug flags are typically not
> documented; if you're interacting with them, you're
> typically already in the code.

Do you mean the debug flag isn't documented in the 'usage:' output, or
in the man page?  I know debug flags are documented in quite a few
programs (viz., mothra, doc2txt, gzip, proof, and ssh come to mind) so
just trying to understand what is typical and what is not.  I don't
think its a foolish consistency to document them where they're
available, particularly if all they're doing is providing output to
stderr or what-not: without even looking at the code, some debug
output is useful to a passerby.
> 
> > 2) I didn't include git/hist in the doc patch: that's a "shouldn't
> >    need to use" program as well?
> 
> No, that's one that's quite useful; it's basically
> git blame on a budget (because it's honesty a pain
> in the ass to implement 'blame'.
> 

Ah, okay, good to know!  I had been wondering what the
(semi-)corollary to git/blame was!


  parent reply	other threads:[~2023-10-01 23:39 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-01  6:18 Romano
2023-10-01 14:48 ` ori
2023-10-01 15:49   ` unobe
2023-10-01 16:42     ` ori
2023-10-01 20:14       ` ori
2023-10-01 23:34       ` unobe [this message]
2023-10-02  3:40         ` Kurt H Maier

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=A1FA07F302666A10482EAB62ED14EE37@smtp.pobox.com \
    --to=unobe@cpan.org \
    --cc=9front@9front.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).