{"thread":{"id":"17889","subject":"[PATCH 1/6] Modify description file to say what this file is","startedAt":"2009-02-19T07:36:00Z","lastAt":"2009-03-04T10:01:54Z","messageCount":6,"participants":["John Tapsell","Junio C Hamano","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":6},"messages":[{"id":"105400","messageId":"200902190736.00462.johnflux@gmail.com","threadId":"17889","inReplyTo":null,"subject":"[PATCH 1/6] Modify description file to say what this file is","fromName":"John Tapsell","fromEmail":"johnflux@gmail.com","sentAt":"2009-02-19T07:36:00Z","receivedAt":"2009-02-19T07:36:00Z","isPatch":true,"sender":{"key":"johnflux@gmail.com","avatar":"https://gravatar.com/avatar/25f70d4c0f96396b84a2e34bcd9bdc233462c7b4be29b5fdca8266fc53f30b0c?d=mp&s=160"},"body":"A lot of people see this message for the first time on the gitweb\ninterface, where there is no clue as to what 'this file' means.\n\nSigned-off-by: John Tapsell \n---\n templates/hooks--update.sample | 5 +++--\n templates/this--description | 2 +-\n 2 files changed, 4 insertions(+), 3 deletions(-)\n\ndiff --git a/templates/hooks--update.sample b/templates/hooks--update.sample\nindex 93c6055..f753d28 100755\n--- a/templates/hooks--update.sample\n+++ b/templates/hooks--update.sample\n@@ -43,10 +43,11 @@ allowdeletetag=$(git config --bool hooks.allowdeletetag)\n \n # check for no description\n projectdesc=$(sed -e '1q' \"$GIT_DIR/description\")\n-if [ -z \"$projectdesc\" -o \"$projectdesc\" = \"Unnamed repository; edit this file to name it for gitweb.\" ]; then\n+case $projectdesc in \"Unnamed repository;\"*|'')\n \techo \"*** Project description file hasn't been set\" >&2\n \texit 1\n-fi\n+\t;;\n+esac\n \n # --- Check types\n # if $newrev is 0000...0000, it's a commit to delete a ref.\ndiff --git a/templates/this--description b/templates/this--description\nindex c6f25e8..dcde3c5 100644\n--- a/templates/this--description\n+++ b/templates/this--description\n@@ -1 +1 @@\n-Unnamed repository; edit this file to name it for gitweb.\n+Unnamed repository; edit this file 'description' to name the repository\n-- \n1.6.2.rc1.3.g7d31b.dirty\n"},{"id":"105428","messageId":"7vy6w2n3cz.fsf@gitster.siamese.dyndns.org","threadId":"17889","inReplyTo":"200902190736.00462.johnflux@gmail.com","subject":"Re: [PATCH 1/6] Modify description file to say what this file is","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-02-19T10:11:08Z","receivedAt":"2009-02-19T10:11:08Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"[PATCH 1/6] Modify description file to say what this file is\n\nLooks good.\n\n[PATCH 2/6] Google has renamed the imap folder\n\nJeff already pointed out an obvious thinko; I could fix-up locally (just\nask).\n\n[PATCH 3/6] Improve error message for branching an existing branch\n\nThe extra sentence is useless noise to annoy users and make them shout\n\"none of your business!\" back to git.\n\nI would probably get this error message \"already exists.\" more from\nforgetting to say \"-f\" in this sequence:\n\n $ git branch -f pu next\n $ git checkout pu\n $ sh rebuild-pu-script\n\nto rebuild pu on top of updated next, and \"did you mean to checkout?\"\nmisses the mark by a kilometer.\n\n[PATCH 4/6] Improve error message for git-filter-branch\n\nLooks good, with Sverre's rewording would be better, which I could locally\nsquash in. Needs signoff, which I could locally forge (just ask to fix-up\nand forge).\n\n[PATCH 5/6] Change output \"error: \" to \"Error: \" etc\n\nJeff is right, and the patch is wrong.\n\n[PATCH 6/6] Mention to the user that they can reorder commits\n\nThe placement of the new message does not feel right, as adding anything\nnear \"If you remove ... WILL BE LOST\" will cloud out that message which is\nmore important.\n\nI think it should come near or perhaps even before Commands, if we were to\nadd anything here.\n\nBut I am afraid that the proposed new message will hurt the clueless users\nmore than it would help them.\n\nThe cheat-sheet at the top is not for learning what the command can do for\nthe first time. It is there to remind people (who already have general\nidea on what can be done) how exactly the commands are spelled. If\nsomebody does not even know that the purpose of rebase-i is to amend and\nresequence, he will more likely destroy his history by blindly using the\ncommand without knowing what is going on, than making a lucky guess.\n\nFor that reason, a more appropriate line to add, if we were to add\nanything, might be:\n\n # s, squash = use commit, but meld into previous commit\n #\n+# If you do not know what is going on, remove everything and exit the editor!\n+#\n # If you remove a line here THAT COMMIT WILL BE LOST.\n # However, if you remove everything, the rebase will be aborted.\n"},{"id":"105432","messageId":"43d8ce650902190218v513b8f58n8338086af41b79d9@mail.gmail.com","threadId":"17889","inReplyTo":"7vy6w2n3cz.fsf@gitster.siamese.dyndns.org","subject":"Re: [PATCH 1/6] Modify description file to say what this file is","fromName":"John Tapsell","fromEmail":"johnflux@gmail.com","sentAt":"2009-02-19T10:18:33Z","receivedAt":"2009-02-19T10:18:33Z","isPatch":true,"sender":{"key":"johnflux@gmail.com","avatar":"https://gravatar.com/avatar/25f70d4c0f96396b84a2e34bcd9bdc233462c7b4be29b5fdca8266fc53f30b0c?d=mp&s=160"},"body":"2009/2/19 Junio C Hamano :\n> [PATCH 1/6] Modify description file to say what this file is\n>\n> Looks good.\n>\n> [PATCH 2/6] Google has renamed the imap folder\n>\n> Jeff already pointed out an obvious thinko; I could fix-up locally (just\n> ask).\n>\n> [PATCH 3/6] Improve error message for branching an existing branch\n>\n> The extra sentence is useless noise to annoy users and make them shout\n> \"none of your business!\" back to git.\n>\n> I would probably get this error message \"already exists.\" more from\n> forgetting to say \"-f\" in this sequence:\n>\n> $ git branch -f pu next\n> $ git checkout pu\n> $ sh rebuild-pu-script\n>\n> to rebuild pu on top of updated next, and \"did you mean to checkout?\"\n> misses the mark by a kilometer.\n>\n> [PATCH 4/6] Improve error message for git-filter-branch\n>\n> Looks good, with Sverre's rewording would be better, which I could locally\n> squash in. Needs signoff, which I could locally forge (just ask to fix-up\n> and forge).\n>\n> [PATCH 5/6] Change output \"error: \" to \"Error: \" etc\n>\n> Jeff is right, and the patch is wrong.\n>\n> [PATCH 6/6] Mention to the user that they can reorder commits\n>\n> The placement of the new message does not feel right, as adding anything\n> near \"If you remove ... WILL BE LOST\" will cloud out that message which is\n> more important.\n>\n> I think it should come near or perhaps even before Commands, if we were to\n> add anything here.\n>\n> But I am afraid that the proposed new message will hurt the clueless users\n> more than it would help them.\n>\n> The cheat-sheet at the top is not for learning what the command can do for\n> the first time. It is there to remind people (who already have general\n> idea on what can be done) how exactly the commands are spelled. If\n> somebody does not even know that the purpose of rebase-i is to amend and\n> resequence, he will more likely destroy his history by blindly using the\n> command without knowing what is going on, than making a lucky guess.\n>\n> For that reason, a more appropriate line to add, if we were to add\n> anything, might be:\n>\n> # s, squash = use commit, but meld into previous commit\n> #\n> +# If you do not know what is going on, remove everything and exit the editor!\n> +#\n> # If you remove a line here THAT COMMIT WILL BE LOST.\n> # However, if you remove everything, the rebase will be aborted.\n>\n>\n\nJunio,\n\n Thanks - I like everything you said. Could you go ahead and commit\nthe accepted ones, with all the fix ups mentioned?\n\nThanks!\n\nJohnFlux\n"},{"id":"106949","messageId":"43d8ce650903032307i512268f4sa3240c517b51d0b4@mail.gmail.com","threadId":"17889","inReplyTo":"7vy6w2n3cz.fsf@gitster.siamese.dyndns.org","subject":"Re: [PATCH 1/6] Modify description file to say what this file is","fromName":"John Tapsell","fromEmail":"johnflux@gmail.com","sentAt":"2009-03-04T07:07:14Z","receivedAt":"2009-03-04T07:07:14Z","isPatch":true,"sender":{"key":"johnflux@gmail.com","avatar":"https://gravatar.com/avatar/25f70d4c0f96396b84a2e34bcd9bdc233462c7b4be29b5fdca8266fc53f30b0c?d=mp&s=160"},"body":"Hi Junio,\n Did you commit my patches? I don't see them yet in git.git ?\n\nJohn Tapsell\n\n2009/2/19 Junio C Hamano :\n> [PATCH 1/6] Modify description file to say what this file is\n>\n> Looks good.\n>\n> [PATCH 2/6] Google has renamed the imap folder\n>\n> Jeff already pointed out an obvious thinko; I could fix-up locally (just\n> ask).\n>\n> [PATCH 3/6] Improve error message for branching an existing branch\n>\n> The extra sentence is useless noise to annoy users and make them shout\n> \"none of your business!\" back to git.\n>\n> I would probably get this error message \"already exists.\" more from\n> forgetting to say \"-f\" in this sequence:\n>\n>    $ git branch -f pu next\n>    $ git checkout pu\n>    $ sh rebuild-pu-script\n>\n> to rebuild pu on top of updated next, and \"did you mean to checkout?\"\n> misses the mark by a kilometer.\n>\n> [PATCH 4/6] Improve error message for git-filter-branch\n>\n> Looks good, with Sverre's rewording would be better, which I could locally\n> squash in.  Needs signoff, which I could locally forge (just ask to fix-up\n> and forge).\n>\n> [PATCH 5/6] Change output \"error: \" to \"Error: \" etc\n>\n> Jeff is right, and the patch is wrong.\n>\n> [PATCH 6/6] Mention to the user that they can reorder commits\n>\n> The placement of the new message does not feel right, as adding anything\n> near \"If you remove ... WILL BE LOST\" will cloud out that message which is\n> more important.\n>\n> I think it should come near or perhaps even before Commands, if we were to\n> add anything here.\n>\n> But I am afraid that the proposed new message will hurt the clueless users\n> more than it would help them.\n>\n> The cheat-sheet at the top is not for learning what the command can do for\n> the first time.  It is there to remind people (who already have general\n> idea on what can be done) how exactly the commands are spelled.  If\n> somebody does not even know that the purpose of rebase-i is to amend and\n> resequence, he will more likely destroy his history by blindly using the\n> command without knowing what is going on, than making a lucky guess.\n>\n> For that reason, a more appropriate line to add, if we were to add\n> anything, might be:\n>\n>  #  s, squash = use commit, but meld into previous commit\n>  #\n> +# If you do not know what is going on, remove everything and exit the editor!\n> +#\n>  # If you remove a line here THAT COMMIT WILL BE LOST.\n>  # However, if you remove everything, the rebase will be aborted.\n>\n>\n"},{"id":"106957","messageId":"20090304100000.GB32433@coredump.intra.peff.net","threadId":"17889","inReplyTo":"43d8ce650903032307i512268f4sa3240c517b51d0b4@mail.gmail.com","subject":"Re: [PATCH 1/6] Modify description file to say what this file is","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-03-04T10:00:00Z","receivedAt":"2009-03-04T10:00:00Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Wed, Mar 04, 2009 at 07:07:14AM +0000, John Tapsell wrote:\n\n> Hi Junio,\n> Did you commit my patches? I don't see them yet in git.git ?\n\nTry looking in 'next', which is where new patches cook before going to\nmaster (especially now, as we are right in the middle of release freeze\non master).\n\n git log --author=John.Tapsell origin/next\n\n-Peff\n"},{"id":"106958","messageId":"7v7i357gj1.fsf@gitster.siamese.dyndns.org","threadId":"17889","inReplyTo":"43d8ce650903032307i512268f4sa3240c517b51d0b4@mail.gmail.com","subject":"Re: [PATCH 1/6] Modify description file to say what this file is","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-03-04T10:01:54Z","receivedAt":"2009-03-04T10:01:54Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"John Tapsell writes:\n\n> Hi Junio,\n> Did you commit my patches? I don't see them yet in git.git ?\n\nPlease look for them in the 'next' branch.\n\nPatches that came after -rc1 are at best queued in 'next' for post 1.6.2.\nThe only exceptions are critical and obvious fixes, and fixes to\nregressions introduced since 1.6.1.\n"}]}