From: Stefan Beller <sbeller@google.com>
To: Duy Nguyen <pclouds@gmail.com>
Cc: git <git@vger.kernel.org>, Junio C Hamano <gitster@pobox.com>
Subject: Re: [PATCH 00/59] Split config.txt
Date: Mon, 22 Oct 2018 12:22:19 -0700 [thread overview]
Message-ID: <CAGZ79kZyjjG9oRzC4jJ4ZqD2efkzR5GyFHUkZKf=811emE=tHA@mail.gmail.com> (raw)
In-Reply-To: <20181020123848.2785-1-pclouds@gmail.com>
On Sat, Oct 20, 2018 at 5:39 AM Nguyễn Thái Ngọc Duy <pclouds@gmail.com> wrote:
>
> I started this a couple months back, moving a couple big config
> sections out of config.txt to make it more manageable. This series
> almost completes that. It moves all configs (except http.* which have
> changes on 'pu') out of config.txt. config.txt is now about the
> syntax, and a list of config sections. http section can be moved out
> later.
>
> I did a doc-diff on this series and the only change is ssh.variant is
> moved down a bit to keep it in order, which is intended.
>
> I thought of grouping all these config files in a separate directory
> Documentation/config to avoid cluttering Documentation/ too much but
> let's wait and see if people really find Documentation growing too
> large first.
Have you considered to use config as a prefix, i.e.
have config-add.txt instead of add-config.txt ?
currently any command is documented in a file
that is prefixed with "git-". There are others such as
gitcli or gitsubmodules, but as they lack the '-' they
surely are not about a command.
With the config- prefix it would be easier to sort and
cope with the individual files, and it would allow
grepping through Documentation/conf* (which may
be easier than Documentation/*conf* or similar)
I have no strong objection to the series as is
(I have not looked at any patch), but I would think
either a config directory or prefix may help further.
Stefan
prev parent reply other threads:[~2018-10-22 19:22 UTC|newest]
Thread overview: 67+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-10-20 12:37 [PATCH 00/59] Split config.txt Nguyễn Thái Ngọc Duy
2018-10-20 12:37 ` [PATCH 01/59] config.txt: move advice.* to a separate file Nguyễn Thái Ngọc Duy
2018-10-20 12:37 ` [PATCH 02/59] config.txt: move core.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:37 ` [PATCH 03/59] config.txt: move add.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:37 ` [PATCH 04/59] config.txt: move alias.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:37 ` [PATCH 05/59] config.txt: move am.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:37 ` [PATCH 06/59] config.txt: move apply.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:37 ` [PATCH 07/59] config.txt: move blame.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:37 ` [PATCH 08/59] config.txt: move branch.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:37 ` [PATCH 09/59] config.txt: move browser.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:37 ` [PATCH 10/59] config.txt: move checkout.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 11/59] config.txt: move clean.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 12/59] config.txt: move color.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 13/59] config.txt: move column.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 14/59] config.txt: move commit.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 15/59] config.txt: move credential.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 16/59] config.txt: move completion.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 17/59] config.txt: move difftool.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 18/59] config.txt: move fastimport.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 19/59] config.txt: move filter.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 20/59] config.txt: move fsck.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 21/59] config.txt: move gc.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 22/59] config.txt: move gitweb.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 23/59] config.txt: move grep.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 24/59] config.txt: move gpg.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 25/59] config.txt: move guitool.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 26/59] config.txt: move help.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 27/59] config.txt: move ssh.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 28/59] config.txt: move i18n.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 29/59] " Nguyễn Thái Ngọc Duy
2018-10-21 19:40 ` Andrei Rybak
2018-10-20 12:38 ` [PATCH 30/59] config.txt: move init.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 31/59] config.txt: move instaweb.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 32/59] config.txt: move interactive.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 33/59] config.txt: move log.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 34/59] config.txt: move mailinfo.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 35/59] config.txt: move mailmap.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 36/59] config.txt: move man.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 37/59] config.txt: move mergetool.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 38/59] config.txt: move notes.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 39/59] config.txt: move pack.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 40/59] config.txt: move pager.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 41/59] config.txt: move pretty.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 42/59] config.txt: move protocol.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 43/59] config.txt: move remote.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 44/59] config.txt: move remotes.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 45/59] config.txt: move repack.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 46/59] config.txt: move rerere.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 47/59] config.txt: move sequencer.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 48/59] config.txt: move showBranch.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 49/59] config.txt: move splitIndex.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 50/59] config.txt: move status.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 51/59] config.txt: move tag.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 52/59] config.txt: move transfer.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 53/59] config.txt: move uploadarchive.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 54/59] config.txt: move uploadpack.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 55/59] config.txt: move url.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 56/59] config.txt: move user.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 57/59] config.txt: move versionsort.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 58/59] config.txt: move web.* " Nguyễn Thái Ngọc Duy
2018-10-20 12:38 ` [PATCH 59/59] config.txt: move worktree.* " Nguyễn Thái Ngọc Duy
2018-10-20 19:25 ` [PATCH 00/59] Split config.txt Ævar Arnfjörð Bjarmason
2018-10-20 19:30 ` Duy Nguyen
2018-10-20 19:41 ` Ævar Arnfjörð Bjarmason
2018-10-20 23:29 ` Junio C Hamano
2018-10-22 14:42 ` Duy Nguyen
2018-10-22 19:22 ` Stefan Beller [this message]
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='CAGZ79kZyjjG9oRzC4jJ4ZqD2efkzR5GyFHUkZKf=811emE=tHA@mail.gmail.com' \
--to=sbeller@google.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=pclouds@gmail.com \
/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).