threads / discuss / 46888

"man git-config", "--list" option misleadingly refers to "config file" (singular)

Subject: "man git-config", "--list" option misleadingly refers to "config file" (singular)

## tl;dr

4 messages between Oct 3, 2017 and Oct 3, 2017.

replies: 3people: 2as markdown or json

rpjday@crashcourse.ca· Oct 3, 2017, 10:34 UTC · lore
   (i suppose that if i'm going to continue whining about stuff, i might
as well clone the git source and start submitting patches.)
   in "man git-config":
     -l
     --list
           List all variables set in config file, along with their values.
                                            ^^^^

except that, AIUI, "git config --list" will list the combination of all config values in (if it exists) /etc/gitconfig, then the user's global settings, and finally the repo's config settings if one happens to be in a working directory, so technically, it lists the contents of *all* of the config files (plural), no?

   i can submit a patch to clarify that, unless there's some weird definition
of "config file" that implicitly means the combination of all of them.
rday
Jeff King· Oct 3, 2017, 13:49 UTC · re: rpjday@crashcourse.ca · lore

Re: "man git-config", "--list" option misleadingly refers to "config file" (singular)

On Tue, Oct 03, 2017 at 06:34:34AM -0400, rpjday@crashcourse.ca wrote:
>   (i suppose that if i'm going to continue whining about stuff, i might
> as well clone the git source and start submitting patches.)
Yes, please. :)
Show 12 quoted lines
>   in "man git-config":
> 
>     -l
>     --list
>           List all variables set in config file, along with their values.
>                                            ^^^^
> 
> except that, AIUI, "git config --list" will list the combination of all
> config values in (if it exists) /etc/gitconfig, then the user's global
> settings, and finally the repo's config settings if one happens to be
> in a working directory, so technically, it lists the contents of *all* of
> the config files (plural), no?

It does that by default, or it lists the contents of a specific file if given (either by --file, or with --system, --global, or --local).

So I agree it's not quite accurate, but you probably want some phrasing that leaves this unsaid (the actual rules are described earlier in the description section). Maybe just refer to it as the "config source" or something?

-Peff
Robert P. J. Day· Oct 3, 2017, 20:54 UTC · re: Jeff King · lore

Re: "man git-config", "--list" option misleadingly refers to "config file" (singular)

On Tue, 3 Oct 2017, Jeff King wrote:
Show 27 quoted lines
> On Tue, Oct 03, 2017 at 06:34:34AM -0400, rpjday@crashcourse.ca wrote:
>
> >   (i suppose that if i'm going to continue whining about stuff, i might
> > as well clone the git source and start submitting patches.)
>
> Yes, please. :)
>
> >   in "man git-config":
> >
> >     -l
> >     --list
> >           List all variables set in config file, along with their values.
> >                                            ^^^^
> >
> > except that, AIUI, "git config --list" will list the combination
> > of all config values in (if it exists) /etc/gitconfig, then the
> > user's global settings, and finally the repo's config settings if
> > one happens to be in a working directory, so technically, it lists
> > the contents of *all* of the config files (plural), no?
>
> It does that by default, or it lists the contents of a specific file
> if given (either by --file, or with --system, --global, or --local).
>
> So I agree it's not quite accurate, but you probably want some
> phrasing that leaves this unsaid (the actual rules are described
> earlier in the description section). Maybe just refer to it as the
> "config source" or something?
  i think the simplest phrasing is, "List all variables set in the
current configuration, along with their values."
  sound fair?
rday
-- 
========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                        http://crashcourse.ca

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================
Jeff King· Oct 3, 2017, 23:12 UTC · re: Robert P. J. Day · lore

Re: "man git-config", "--list" option misleadingly refers to "config file" (singular)

On Tue, Oct 03, 2017 at 04:54:32PM -0400, Robert P. J. Day wrote:
Show 12 quoted lines
> > It does that by default, or it lists the contents of a specific file
> > if given (either by --file, or with --system, --global, or --local).
> >
> > So I agree it's not quite accurate, but you probably want some
> > phrasing that leaves this unsaid (the actual rules are described
> > earlier in the description section). Maybe just refer to it as the
> > "config source" or something?
> 
>   i think the simplest phrasing is, "List all variables set in the
> current configuration, along with their values."
> 
>   sound fair?
Yes, sounds reasonable to me.
-Peff

← back to recent threads