Re: [PATCH] fetch/pull: support Cogito-style remote branch information.
- From
Linus Torvalds <torvalds@osdl.org>
- Date
- Jul 15, 2005, 22:42 UTC
- Message-ID
- <Pine.LNX.4.58.0507151529590.19183@g5.osdl.org>
- In-Reply-To
- <7voe93rbmu.fsf_-_@assigned-by-dhcp.cox.net>
Hmm..
This patch actually brings up two different issues
- I actually prefer code and documentation to be separated. Finding the actual changes to code in this patch is made much harder by the fact that most of the changes are documentation updates. In many ways it would have been nicer to separate the two out - first the actual change, then the docs updates.
Maybe that's just me.
- I'd much rather have a generic "address rewriting layer" than a "-b" flag.
I don't mind the shorthand at all, but I don't think it should be that special. It's not "worthy" of a flag - if you have a shorthand that says "linus == rsync://kernel.org/pub/scm/git/git.git", then I think it should just work, and
git pull linus
should end up not needing a "-b" flag. It's not like there is any real ambiguity.
Now, I'd like the address rewriting to actually be fairly capable, so it should be a script of its own.
And it's not necessarily just the branch handling, but more of a generic shorthand: I'd love to be able to mix something like
git pull jgarzik/misc-2.6 upstream
and "jgarzik" would be expanded (through something like .git/branches) to "master.kernel.org:/pub/scm/linux/kernel/git/jgarzik/"), resulting in the _full_ path being expanded to
master.kernel.org:/pub/scm/linux/kernel/git/jgarzik/misc-2.6 upstream
which I have to write out in full (or, more commonly, cut-and-paste) right now.
Hmm?
Linus