{"thread":{"id":"46888","subject":"\"man git-config\", \"--list\" option misleadingly refers to \"config file\" (singular)","startedAt":"2017-10-03T10:34:40Z","lastAt":"2017-10-03T23:12:50Z","messageCount":4,"participants":["rpjday@crashcourse.ca","Jeff King","Robert P. J. Day"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"329549","messageId":"20171003063434.Horde.nbQwMWrv0wS_naxhawU5YlG@crashcourse.ca","threadId":"46888","inReplyTo":null,"subject":"\"man git-config\", \"--list\" option misleadingly refers to \"config file\" (singular)","fromName":"","fromEmail":"rpjday@crashcourse.ca","sentAt":"2017-10-03T10:34:34Z","receivedAt":"2017-10-03T10:34:40Z","isPatch":false,"sender":{"key":"rpjday@crashcourse.ca","avatar":"https://avatars.githubusercontent.com/u/226084077?v=4"},"body":"\n   (i suppose that if i'm going to continue whining about stuff, i might\nas well clone the git source and start submitting patches.)\n\n   in \"man git-config\":\n\n     -l\n     --list\n           List all variables set in config file, along with their values.\n                                            ^^^^\n\nexcept that, AIUI, \"git config --list\" will list the combination of all\nconfig values in (if it exists) /etc/gitconfig, then the user's global\nsettings, and finally the repo's config settings if one happens to be\nin a working directory, so technically, it lists the contents of *all* of\nthe config files (plural), no?\n\n   i can submit a patch to clarify that, unless there's some weird definition\nof \"config file\" that implicitly means the combination of all of them.\n\nrday\n\n"},{"id":"329573","messageId":"20171003134954.cptlpaipmiulfgug@sigill.intra.peff.net","threadId":"46888","inReplyTo":"20171003063434.Horde.nbQwMWrv0wS_naxhawU5YlG@crashcourse.ca","subject":"Re: \"man git-config\", \"--list\" option misleadingly refers to \"config file\" (singular)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2017-10-03T13:49:54Z","receivedAt":"2017-10-03T13:50:01Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Tue, Oct 03, 2017 at 06:34:34AM -0400, rpjday@crashcourse.ca wrote:\n\n>   (i suppose that if i'm going to continue whining about stuff, i might\n> as well clone the git source and start submitting patches.)\n\nYes, please. :)\n\n>   in \"man git-config\":\n> \n>     -l\n>     --list\n>           List all variables set in config file, along with their values.\n>                                            ^^^^\n> \n> except that, AIUI, \"git config --list\" will list the combination of all\n> config values in (if it exists) /etc/gitconfig, then the user's global\n> settings, and finally the repo's config settings if one happens to be\n> in a working directory, so technically, it lists the contents of *all* of\n> the config files (plural), no?\n\nIt does that by default, or it lists the contents of a specific file if\ngiven (either by --file, or with --system, --global, or --local).\n\nSo I agree it's not quite accurate, but you probably want some phrasing\nthat leaves this unsaid (the actual rules are described earlier in the\ndescription section). Maybe just refer to it as the \"config source\" or\nsomething?\n\n-Peff\n"},{"id":"329605","messageId":"alpine.LFD.2.21.1710031653360.8461@localhost.localdomain","threadId":"46888","inReplyTo":"20171003134954.cptlpaipmiulfgug@sigill.intra.peff.net","subject":"Re: \"man git-config\", \"--list\" option misleadingly refers to \"config file\" (singular)","fromName":"Robert P. J. Day","fromEmail":"rpjday@crashcourse.ca","sentAt":"2017-10-03T20:54:32Z","receivedAt":"2017-10-03T20:54:40Z","isPatch":false,"sender":{"key":"rpjday@crashcourse.ca","avatar":"https://avatars.githubusercontent.com/u/226084077?v=4"},"body":"On Tue, 3 Oct 2017, Jeff King wrote:\n\n> On Tue, Oct 03, 2017 at 06:34:34AM -0400, rpjday@crashcourse.ca wrote:\n>\n> >   (i suppose that if i'm going to continue whining about stuff, i might\n> > as well clone the git source and start submitting patches.)\n>\n> Yes, please. :)\n>\n> >   in \"man git-config\":\n> >\n> >     -l\n> >     --list\n> >           List all variables set in config file, along with their values.\n> >                                            ^^^^\n> >\n> > except that, AIUI, \"git config --list\" will list the combination\n> > of all config values in (if it exists) /etc/gitconfig, then the\n> > user's global settings, and finally the repo's config settings if\n> > one happens to be in a working directory, so technically, it lists\n> > the contents of *all* of the config files (plural), no?\n>\n> It does that by default, or it lists the contents of a specific file\n> if given (either by --file, or with --system, --global, or --local).\n>\n> So I agree it's not quite accurate, but you probably want some\n> phrasing that leaves this unsaid (the actual rules are described\n> earlier in the description section). Maybe just refer to it as the\n> \"config source\" or something?\n\n  i think the simplest phrasing is, \"List all variables set in the\ncurrent configuration, along with their values.\"\n\n  sound fair?\n\nrday\n\n-- \n\n========================================================================\nRobert P. J. Day                                 Ottawa, Ontario, CANADA\n                        http://crashcourse.ca\n\nTwitter:                                       http://twitter.com/rpjday\nLinkedIn:                               http://ca.linkedin.com/in/rpjday\n========================================================================\n"},{"id":"329617","messageId":"20171003231242.oem4equq6cvicxcw@sigill.intra.peff.net","threadId":"46888","inReplyTo":"alpine.LFD.2.21.1710031653360.8461@localhost.localdomain","subject":"Re: \"man git-config\", \"--list\" option misleadingly refers to \"config file\" (singular)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2017-10-03T23:12:42Z","receivedAt":"2017-10-03T23:12:50Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Tue, Oct 03, 2017 at 04:54:32PM -0400, Robert P. J. Day wrote:\n\n> > It does that by default, or it lists the contents of a specific file\n> > if given (either by --file, or with --system, --global, or --local).\n> >\n> > So I agree it's not quite accurate, but you probably want some\n> > phrasing that leaves this unsaid (the actual rules are described\n> > earlier in the description section). Maybe just refer to it as the\n> > \"config source\" or something?\n> \n>   i think the simplest phrasing is, \"List all variables set in the\n> current configuration, along with their values.\"\n> \n>   sound fair?\n\nYes, sounds reasonable to me.\n\n-Peff\n"}]}