{"thread":{"id":"46279","subject":"[PATCH] t1450: use egrep for regexp \"alternation\"","startedAt":"2017-06-28T17:17:10Z","lastAt":"2017-06-28T17:17:10Z","messageCount":1,"participants":["Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"323434","messageId":"xmqqinjgvxpr.fsf@gitster.mtv.corp.google.com","threadId":"46279","inReplyTo":null,"subject":"[PATCH] t1450: use egrep for regexp \"alternation\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-06-28T17:17:04Z","receivedAt":"2017-06-28T17:17:10Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"GNU grep allows \"\\(A\\|B\\)\" as alternation in BRE, but this is an\nextension not understood by some other implementations of grep\n(Michael Kebe reported an breakage on Solaris).\n\nRewrite the offending test to ERE and use egrep instead.\n\nNoticed-by: Michael Kebe <michael.kebe@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>\n---\n t/t1450-fsck.sh | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/t/t1450-fsck.sh b/t/t1450-fsck.sh\nindex 8f52da2771..30e217dea2 100755\n--- a/t/t1450-fsck.sh\n+++ b/t/t1450-fsck.sh\n@@ -541,7 +541,7 @@ test_expect_success 'fsck --name-objects' '\n \t\tremove_loose_object $(git rev-parse julius:caesar.t) &&\n \t\ttest_must_fail git fsck --name-objects >out &&\n \t\ttree=$(git rev-parse --verify julius:) &&\n-\t\tgrep \"$tree (\\(refs/heads/master\\|HEAD\\)@{[0-9]*}:\" out\n+\t\tegrep \"$tree \\((refs/heads/master|HEAD)@\\{[0-9]*\\}:\" out\n \t)\n '\n \n-- \n2.13.2-655-gbf99380098\n\n"}]}