{"thread":{"id":"11320","subject":"Re: The Reposithon! Take 2 [URL FIX]","startedAt":"2007-12-17T12:00:43Z","lastAt":"2007-12-17T18:03:41Z","messageCount":3,"participants":["Sam Vilain","Junio C Hamano","Nicholas Clark"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"63416","messageId":"476664EB.5030500@vilain.net","threadId":"11320","inReplyTo":"20071217115450.GR23703@plum.flirble.org","subject":"Re: The Reposithon! Take 2 [URL FIX]","fromName":"Sam Vilain","fromEmail":"sam@vilain.net","sentAt":"2007-12-17T12:00:43Z","receivedAt":"2007-12-17T12:00:43Z","isPatch":false,"sender":{"key":"sam@vilain.net","avatar":"https://gravatar.com/avatar/8fc840ca854dbf6f7065b4335e3b934951c1dca3b11db688e95e471901f8f4a8?d=mp&s=160"},"body":"Nicholas Clark wrote:\n> On Tue, Dec 18, 2007 at 12:48:26AM +1300, Sam Vilain wrote:\n>> Nicholas Clark wrote:\n>>> error: Could not interpret The page was, like, not found and stuff. as something to pull\n>>>\n>>>\n>>>                            ^ random capitalisation or missing punctuation\n>>>                                          ^ slang interjection\n>>>                                                          ^ slang interjection\n>>>                                                                     ^ missing capitalisation\n>>>\n>> I've never had my server 404 message\n>>\n>>   http://utsl.gen.nz/err/404.html\n>>\n>> So well critiqued!  Thanks!  :-)\n>>\n>> Hopefully all the pull problems should be sorted now.\n> \n> I remain unhappy that git does not tell me that it's reporting a 404 error.\n> 404 makes sense in any language. What the server admin chooses as the\n> accompanying page may well not, although usually it would be because he or\n> she chooses to make it report errors in the local language.\n\nmeh.  look, actually tell you what my little web server is returning the\nwrong error HTTP response code.  Which is a pretty stupid\nmisconfiguration and not one you'd see very often.  So the data gets a\nlittle further in before it is noticed to be wrong.\n\nThe error message isn't the best, no.\n\nJunio, any chance of this going in?\n\nSubject: Clarify error response from 'git fetch' for bad responses\n\nThis error message prints the reponse from the server at this point.\nLabel it as such in the output.\n---\n walker.c |    2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/walker.c b/walker.c\nindex 397b80d..adc3e80 100644\n--- a/walker.c\n+++ b/walker.c\n@@ -274,7 +274,7 @@ int walker_fetch(struct walker *walker, int targets, char **target,\n \n \tfor (i = 0; i < targets; i++) {\n \t\tif (interpret_target(walker, target[i], &sha1[20 * i])) {\n-\t\t\terror(\"Could not interpret %s as something to pull\", target[i]);\n+\t\t\terror(\"Could not interpret response from server '%s' as something to pull\", target[i]);\n \t\t\tgoto unlock_and_fail;\n \t\t}\n \t\tif (process(walker, lookup_unknown_object(&sha1[20 * i])))\n"},{"id":"63445","messageId":"7vmys98ali.fsf@gitster.siamese.dyndns.org","threadId":"11320","inReplyTo":"476664EB.5030500@vilain.net","subject":"Re: The Reposithon! Take 2 [URL FIX]","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2007-12-17T18:00:57Z","receivedAt":"2007-12-17T18:00:57Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sam Vilain <sam@vilain.net> writes:\n\n> The error message isn't the best, no.\n>\n> Junio, any chance of this going in?\n\nLooks innocuous and an obvious improvement to me.\n"},{"id":"63446","messageId":"20071217180341.GZ23703@plum.flirble.org","threadId":"11320","inReplyTo":"7vmys98ali.fsf@gitster.siamese.dyndns.org","subject":"Re: The Reposithon! Take 2 [URL FIX]","fromName":"Nicholas Clark","fromEmail":"nick@ccl4.org","sentAt":"2007-12-17T18:03:41Z","receivedAt":"2007-12-17T18:03:41Z","isPatch":false,"sender":{"key":"nick@ccl4.org","avatar":null},"body":"On Mon, Dec 17, 2007 at 10:00:57AM -0800, Junio C Hamano wrote:\n> Sam Vilain <sam@vilain.net> writes:\n> \n> > The error message isn't the best, no.\n> >\n> > Junio, any chance of this going in?\n> \n> Looks innocuous and an obvious improvement to me.\n\nCool. Thanks to you both for addressing this concern of mine so rapidly.\n\nNicholas Clark\n"}]}