{"thread":{"id":"5467","subject":"[PATCH] git-repack: clear out tmp packfiles from $PACKDIR instead of cwd","startedAt":"2006-09-04T05:42:16Z","lastAt":"2006-09-04T12:16:07Z","messageCount":4,"participants":["Martin Langhoff","Jakub Narebski"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"26282","messageId":"11573485362686-git-send-email-martin@catalyst.net.nz","threadId":"5467","inReplyTo":null,"subject":"[PATCH] git-repack: clear out tmp packfiles from $PACKDIR instead of cwd","fromName":"Martin Langhoff","fromEmail":"martin@catalyst.net.nz","sentAt":"2006-09-04T05:42:16Z","receivedAt":"2006-09-04T05:42:16Z","isPatch":true,"sender":{"key":"martin@laptop.org","avatar":null},"body":"Temp packfiles should never be created in cwd anyway ;-)\n\nSigned-off-by: Martin Langhoff <martin@catalyst.net.nz>\n---\n git-repack.sh |    4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/git-repack.sh b/git-repack.sh\nindex ccc8e43..eee0d72 100755\n--- a/git-repack.sh\n+++ b/git-repack.sh\n@@ -24,8 +24,9 @@ do\n \tshift\n done\n \n-rm -f .tmp-pack-*\n PACKDIR=\"$GIT_OBJECT_DIRECTORY/pack\"\n+mkdir -p \"$PACKDIR\" || exit\n+rm -f \"$PACKDIR/.tmp-pack-*\"\n \n # There will be more repacking strategies to come...\n case \",$all_into_one,\" in\n@@ -43,7 +44,6 @@ case \",$all_into_one,\" in\n \t;;\n esac\n \n-mkdir -p \"$PACKDIR\" || exit\n pack_objects=\"$pack_objects $local $quiet $no_reuse_delta$extra\"\n name=$( { git-rev-list --objects --all $rev_list ||\n \t  echo \"git-rev-list died with exit code $?\"\n-- \n1.4.2.gdfe7\n\n\n-- \nVGER BF report: U 0.983488\n"},{"id":"26284","messageId":"46a038f90609032248h5eda61dft4150e13e49f8e59e@mail.gmail.com","threadId":"5467","inReplyTo":"11573485362686-git-send-email-martin@catalyst.net.nz","subject":"Re: [PATCH] git-repack: clear out tmp packfiles from $PACKDIR instead of cwd","fromName":"Martin Langhoff","fromEmail":"martin.langhoff@gmail.com","sentAt":"2006-09-04T05:48:58Z","receivedAt":"2006-09-04T05:48:58Z","isPatch":true,"sender":{"key":"martin.langhoff@gmail.com","avatar":"https://gravatar.com/avatar/1e3f311b6c4c15836501901ca58f8c0b0667246488084ba524d8bc9867e22fd9?d=mp&s=160"},"body":"On 9/4/06, Martin Langhoff <martin@catalyst.net.nz> wrote:\n> Temp packfiles should never be created in cwd anyway ;-)\n\nNote! Applies after \"git-repack: create new packs inside $PACKDIR, not cwd\"\n\n> VGER BF report: U 0.983488\n\nHilarious!\n\n\nm\n\n-- \nVGER BF report: U 0.569396\n"},{"id":"26286","messageId":"edgndv$mtv$1@sea.gmane.org","threadId":"5467","inReplyTo":"46a038f90609032248h5eda61dft4150e13e49f8e59e@mail.gmail.com","subject":"Re: [PATCH] git-repack: clear out tmp packfiles from $PACKDIR instead of cwd","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2006-09-04T08:17:11Z","receivedAt":"2006-09-04T08:17:11Z","isPatch":true,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Martin Langhoff wrote:\n\n> On 9/4/06, Martin Langhoff <martin@catalyst.net.nz> wrote:\n>> Temp packfiles should never be created in cwd anyway ;-)\n> \n> Note! Applies after \"git-repack: create new packs inside $PACKDIR, not cwd\"\n> \n>> VGER BF report: U 0.983488\n> \n> Hilarious!\n\nIsn't 'U' for Undecided? (and 'H' for Ham)?\n\n-- \nJakub Narebski\nWarsaw, Poland\nShadeHawk on #git\n\n\n\n-- \nVGER BF report: S 0.999453\n"},{"id":"26301","messageId":"46a038f90609040516y42d0250an85eea3c66bee31b6@mail.gmail.com","threadId":"5467","inReplyTo":"edgndv$mtv$1@sea.gmane.org","subject":"Re: [PATCH] git-repack: clear out tmp packfiles from $PACKDIR instead of cwd","fromName":"Martin Langhoff","fromEmail":"martin.langhoff@gmail.com","sentAt":"2006-09-04T12:16:07Z","receivedAt":"2006-09-04T12:16:07Z","isPatch":true,"sender":{"key":"martin.langhoff@gmail.com","avatar":"https://gravatar.com/avatar/1e3f311b6c4c15836501901ca58f8c0b0667246488084ba524d8bc9867e22fd9?d=mp&s=160"},"body":"On 9/4/06, Jakub Narebski <jnareb@gmail.com> wrote:\n> Martin Langhoff wrote:\n>\n> > On 9/4/06, Martin Langhoff <martin@catalyst.net.nz> wrote:\n> >> Temp packfiles should never be created in cwd anyway ;-)\n> >\n> > Note! Applies after \"git-repack: create new packs inside $PACKDIR, not cwd\"\n> >\n> >> VGER BF report: U 0.983488\n> >\n> > Hilarious!\n>\n> Isn't 'U' for Undecided? (and 'H' for Ham)?\n\nMaybe -- but the score is really high. This patch was part of a series\nof 2 actually. The other one had a score of 1.6 and got bounced. :-/\n\n\n\nmartin\n\n-- \nVGER BF report: U 0.558927\n"}]}