Re: ANSWER

Alan Chandler [off-list ref] writes:

On Sunday 01 October 2006 11:38, Alan Chandler wrote:

quoted

I have a .git/remotes/public file which contains

URL:roo.home:/var/lib/git/akclib.git
Push:master

The URL: and Push: need spaces before the detail

This used to work, so somewhere along the line (when the builtin was 
introduced?) the need for the space has arisen.

True.  I did not notice it before but this indeed is a breakage
when builtin-push.c was done.

Totally untested, but it looks obvious enough...

-- >8 --
git-push: .git/remotes/ file does not require SP after colon

Although most people would have one after colon if only for
readability, we never required it in git-parse-remote, so let's
not require one only in git-push.

Signed-off-by: Junio C Hamano <redacted>
---

diff --git a/builtin-push.c b/builtin-push.c
index c43f256..f5150ed 100644
--- a/builtin-push.c
+++ b/builtin-push.c

@@ -78,12 +78,12 @@ static int get_remotes_uri(const char *r
 		int is_refspec;
 		char *s, *p;
 
-		if (!strncmp("URL: ", buffer, 5)) {
+		if (!strncmp("URL:", buffer, 4)) {
 			is_refspec = 0;
-			s = buffer + 5;
-		} else if (!strncmp("Push: ", buffer, 6)) {
+			s = buffer + 4;
+		} else if (!strncmp("Push:", buffer, 5)) {
 			is_refspec = 1;
-			s = buffer + 6;
+			s = buffer + 5;
 		} else
 			continue;