From: Jon Loeliger <jdl@freescale.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: "Jay Soffian" <jaysoffian@gmail.com>,
"Junio C Hamano" <gitster@pobox.com>,
"Johannes Schindelin" <Johannes.Schindelin@gmx.de>,
しらいしななこ <nanako3@bluebottle.com>,
git@vger.kernel.org, "Daniel Barkalow" <barkalow@iabervon.org>
Subject: Re: [PATCH 2/2] Add support for url aliases in config files
Date: Thu, 21 Feb 2008 10:31:09 -0600 [thread overview]
Message-ID: <47BDA74D.8080608@freescale.com> (raw)
In-Reply-To: <alpine.LFD.1.00.0802201735030.7833@woody.linux-foundation.org>
Linus Torvalds wrote:
>
> The only really sane use of this is the one that Daniel started out with:
> you may have a "external" representation of a name, but for your own
> purely local configuration reasons, you may want to map that name into
> another one that works for you.
>
[ snip ]
> So it's generally *not* that it's an "alias" for another site, since it
> isn't that in general - it may just be a very local configuration thing.
> It's also not really that you would call it one thing and others would
> call it another thing: you want to call it the *same* thing as others call
> it, but you want to work around some specific site issue (or just use a
> cache that is closer without having to think about it).
I totally agree with all that.
> [access "git://git.kernel.org/*"]
> proxy = proxy-program
> url = "ssh://master.kernel.org/*"
>
> which admittedly looks rather strange too, but at the same time it does
> make sense from a "what do we really want to do?" standpoint.
>
> Of course, in this case Daniel didn't actually do that "proxy" part, but I
> think the argument that we should try to make the config file syntax
> describe what the user wants to do is still very true. So skip that
> "proxy" part (maybe somebody wants to do that too some day), and leave the
>
> [access "original"]
> url = "rewritten"
>
> kind of syntax.
>
> (And no, I'm not at all married to the "access" and "url" parts - I don't
> care. I'm just saying that syntax should be tied to what people want to
> do, not anything else.
Right. And I think part of the issue is, which of the two forms will
be present in the user's git requests? He'd _like_ to use the same,
canonical name everyone else does, but for whatever reason, it has to
be rewritten for the actual transportation implementation to work.
Sort of like virtual/physical address definitions here?
[virtual_url "git://git.kernel.org/"]
physical_url = "ssh://master.kernel.org/"
But maybe it needs to be:
physical_url ["ssh://master.kernel.org/"]
virtual_url = "git://git.kernel.org/"
virtual_url = "git://kernel.org/"
The git user would use the virtual_url, but it would be translated
to the physical_url for the underlying implementation to work.
I think it is more than paint coloring at work here. The important
part is to convey the concepts properly with the names. The reader
of the config file has to be able to tell which way the mapping is run.
jdl
next prev parent reply other threads:[~2008-02-21 16:47 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-20 18:43 [PATCH 2/2] Add support for url aliases in config files Daniel Barkalow
2008-02-20 19:15 ` Junio C Hamano
2008-02-20 19:24 ` Daniel Barkalow
2008-02-20 19:42 ` Junio C Hamano
2008-02-20 19:54 ` Jay Soffian
2008-02-20 20:42 ` Daniel Barkalow
2008-02-20 22:02 ` しらいしななこ
2008-02-20 22:22 ` Johannes Schindelin
2008-02-21 0:47 ` Junio C Hamano
2008-02-21 1:26 ` Jay Soffian
2008-02-21 1:47 ` Linus Torvalds
2008-02-21 2:19 ` Jay Soffian
2008-02-21 5:04 ` Daniel Barkalow
2008-02-21 5:34 ` Junio C Hamano
2008-02-25 4:08 ` Junio C Hamano
2008-02-25 4:30 ` Daniel Barkalow
2008-02-25 6:22 ` Junio C Hamano
2008-02-25 6:35 ` Daniel Barkalow
2008-02-21 16:31 ` Jon Loeliger [this message]
2008-02-20 19:49 ` Jay Soffian
2008-02-20 19:58 ` Junio C Hamano
2008-02-20 20:22 ` Jay Soffian
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=47BDA74D.8080608@freescale.com \
--to=jdl@freescale.com \
--cc=Johannes.Schindelin@gmx.de \
--cc=barkalow@iabervon.org \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=jaysoffian@gmail.com \
--cc=nanako3@bluebottle.com \
--cc=torvalds@linux-foundation.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).