From: Aaron Schrab <aaron@schrab.com>
To: "Kyle J. McKay" <mackyle@gmail.com>
Cc: "Jeff King" <peff@peff.net>,
git@vger.kernel.org, "David Aguilar" <davvid@gmail.com>,
"Petr Baudis" <pasky@ucw.cz>,
"Richard Hartmann" <richih.mailinglist@gmail.com>,
"Daniel Knittl-Frank" <knittl89@googlemail.com>,
"Jan Krüger" <jk@jk.gs>, "Alejandro Mery" <amery@geeks.cl>
Subject: Re: [PATCH v3] config: add support for http.<url>.* settings
Date: Fri, 12 Jul 2013 16:58:44 -0400 [thread overview]
Message-ID: <20130712205843.GJ4604@pug.qqx.org> (raw)
In-Reply-To: <F5272E14-188E-4199-9523-D2ED66574D91@gmail.com>
At 06:07 -0700 12 Jul 2013, "Kyle J. McKay" <mackyle@gmail.com> wrote:
>I don't think it's necessary to split the URL apart though. Whatever
>URL the user gave to git on the command line (at some point even if
>it's now stored as a remote setting in config) complete with URL-
>encoding, user names, port names, etc. is the same url, possibly
>shortened, that needs to be used for the http.<url>.option setting.
This seems to be assuming that the configuration is done after the URL
is entered and that URLs are always entered manually. I don't think
either of those assumptions is valid. A user may want to specify http
settings for all repositories on a specified host and so add settings
for that host to ~/.gitconfig expecting those settings to be used later.
A URL in a slightly different format may later be copy+pasted without
the user realizing that it won't use that config due to one of the
versions being in a non-canonical form.
>I think that's simple and very easy to explain and avoids user
>confusion and surprise while still allowing a default to be set for a
>site but easily overridden for a portion of that site without needing
>to worry about the order config files are processed or the order of the
>[http "<url>"] sections within them.
I agree that the method is easy to explain, but I think a user may very
well be surprised and confused in a scenario like I described above.
And having the order not matter (in some cases) for these configuration
items deviates from how other configuration values are handled.
next prev parent reply other threads:[~2013-07-12 20:58 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-11 21:50 [PATCH v3] config: add support for http.<url>.* settings Kyle J. McKay
2013-07-11 23:12 ` Junio C Hamano
[not found] ` <455666C5-7663-4361-BF34-378D3EAE2891@gmail.com>
[not found] ` <7vsizjn390.fsf@alter.siamese.dyndns.org>
[not found] ` <7v4nbyic57.fsf@alter.siamese.dyndns.org>
2013-07-13 19:46 ` Kyle J. McKay
2013-07-15 4:02 ` Junio C Hamano
2013-07-15 5:12 ` Jeff King
2013-07-15 9:50 ` Kyle J. McKay
2013-07-15 5:06 ` Jeff King
2013-07-12 9:59 ` Jeff King
2013-07-12 13:07 ` Kyle J. McKay
2013-07-12 20:58 ` Aaron Schrab [this message]
2013-07-15 4:43 ` Jeff King
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=20130712205843.GJ4604@pug.qqx.org \
--to=aaron@schrab.com \
--cc=amery@geeks.cl \
--cc=davvid@gmail.com \
--cc=git@vger.kernel.org \
--cc=jk@jk.gs \
--cc=knittl89@googlemail.com \
--cc=mackyle@gmail.com \
--cc=pasky@ucw.cz \
--cc=peff@peff.net \
--cc=richih.mailinglist@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).