{"thread":{"id":"7556","subject":"[RFC/PATCH 1/2] git-push reports the URL after failing.","startedAt":"2007-04-07T06:04:55Z","lastAt":"2007-04-07T06:04:55Z","messageCount":1,"participants":["Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"38840","messageId":"7vfy7cg1o8.fsf@assigned-by-dhcp.cox.net","threadId":"7556","inReplyTo":null,"subject":"[RFC/PATCH 1/2] git-push reports the URL after failing.","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2007-04-07T06:04:55Z","receivedAt":"2007-04-07T06:04:55Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"This came up on #git when somebody was getting 'unable to create\n./objects/tmp_oXXXX' but sweared he had write permission to that\ndirectory.  It turned out that the repository URL was changed\nand he was accessing a repository he does not have a write\npermission anymore.\n\nI am not sure how much this would have helped somebody who\nbelieved he was accessing location when the permission of that\nlocation was changed while he was looking the other way, though.\nBut giving more information on the error path would be better,\nand the next change would be helped with this as well.\n\nSigned-off-by: Junio C Hamano <junkio@cox.net>\n---\n\n builtin-push.c |    2 ++\n 1 files changed, 2 insertions(+), 0 deletions(-)\n\ndiff --git a/builtin-push.c b/builtin-push.c\nindex 70b1168..23143be 100644\n--- a/builtin-push.c\n+++ b/builtin-push.c\n@@ -339,6 +339,8 @@ static int do_push(const char *repo)\n \t\terr = run_command_v_opt(argv, RUN_GIT_CMD);\n \t\tif (!err)\n \t\t\tcontinue;\n+\n+\t\terror(\"failed to push to '%s'\", uri[i]);\n \t\tswitch (err) {\n \t\tcase -ERR_RUN_COMMAND_FORK:\n \t\t\tdie(\"unable to fork for %s\", sender);\n"}]}