On Fri, Feb 20, 2009 at 5:34 PM, Marc Branchaud <marcnarc@xxxxxxxxxxx> wrote: > All valid points. But, dang it, I really think intelligently adding that > extra space the right thing to do. Already coded. :-) > It wouldn't be all that tedious -- you could detect the need for the extra > space in config_read_branches() the first time type is set to REBASE, no? What is that about everything being easy for the man who doesn't have to do it? :-) (It was no big deal, I just needed to rework a few things cause I'd painted myself into a corner.) >> I left that out on purpose. The only folks with push refspecs put them >> their manually, and the raw refspec is clearer and more concise than >> any English words can convey. That was my reasoning anyway. > > Fair enough. Yeah, I'm fixing this too by properly expanded out the refspecs the same way that it's done by {builtin-send-pack, http-push}.c. Hopefully I can have a patch out later today. j. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html