{"thread":{"id":"1084","subject":"[PATCH] Trivial sed fix up in t/t6001","startedAt":"2005-07-01T13:55:16Z","lastAt":"2005-07-01T15:51:06Z","messageCount":2,"participants":["Mark Allen","Jon Seymour"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"5532","messageId":"20050701135516.87892.qmail@web41208.mail.yahoo.com","threadId":"1084","inReplyTo":null,"subject":"[PATCH] Trivial sed fix up in t/t6001","fromName":"Mark Allen","fromEmail":"mrallen1@yahoo.com","sentAt":"2005-07-01T13:55:16Z","receivedAt":"2005-07-01T13:55:16Z","isPatch":true,"sender":{"key":"mrallen1@yahoo.com","avatar":null},"body":"Change the sed seperator in t/t6001.\n\nThis trivial patch removes the semicolon as the sed seperator in the t/t6001 test script\nand replaces it with white space.  This makes BSD sed(1) much happier.\n\nSigned-off-by: Mark Allen <mrallen1@yahoo.com>\n\n===\n\nJon,\n\nIf you could ack this patch or incorporate it into your tree, I'd be very obliged.\n\nThanks,\n\n--Mark\n\n\n\ndiff --git a/t/t6001-rev-list-merge-order.sh b/t/t6001-rev-list-merge-order.sh\n--- a/t/t6001-rev-list-merge-order.sh\n+++ b/t/t6001-rev-list-merge-order.sh\n@@ -40,7 +40,8 @@ save_tag()\n \t[ -n \"$_tag\" ] || error \"usage: save_tag tag commit-args ...\"\n \tshift 1\n     \t\"$@\" >.git/refs/tags/$_tag\n-    \tsed_script=\"s/$(tag $_tag)/$_tag/g${sed_script+;}$sed_script\"\n+    \tsed_script=\"s/$(tag $_tag)/$_tag/g\n+$sed_script\"\n }\n \n # Replace unhelpful sha1 hashses with their symbolic equivalents \n"},{"id":"5539","messageId":"2cfc40320507010851259a7e85@mail.gmail.com","threadId":"1084","inReplyTo":"20050701135516.87892.qmail@web41208.mail.yahoo.com","subject":"Re: [PATCH] Trivial sed fix up in t/t6001","fromName":"Jon Seymour","fromEmail":"jon.seymour@gmail.com","sentAt":"2005-07-01T15:51:06Z","receivedAt":"2005-07-01T15:51:06Z","isPatch":true,"sender":{"key":"jon.seymour@gmail.com","avatar":"https://avatars.githubusercontent.com/u/207131?v=4"},"body":"On 7/1/05, Mark Allen <mrallen1@yahoo.com> wrote:\n> Change the sed seperator in t/t6001.\n> \n> This trivial patch removes the semicolon as the sed seperator in the t/t6001 test script\n> and replaces it with white space.  This makes BSD sed(1) much happier.\n> \n> Signed-off-by: Mark Allen <mrallen1@yahoo.com>\n> \n> ===\n> \n> Jon,\n> \n> If you could ack this patch or incorporate it into your tree, I'd be very obliged.\n> \n> Thanks,\n> \n> --Mark\n\nAck'd. I have also re-applied it to the code I moved into\nt/t6000-lib.sh in my recent patch series, so it is fit to go in one\nway or another!\n\njon.\n"}]}