Re: [PATCH 19/22] t/t8*: merge "grep | sed" pipelines
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Mar 6, 2024, 00:59 UTC
- Message-ID
- <xmqq8r2ww4xg.fsf@gitster.g>
- In-Reply-To
- <20240305212533.12947-20-dev+git@drbeat.li>
"Beat Bolli" <bb@drbeat.li> writes:
Show 15 quoted lines
> Signed-off-by: Beat Bolli <dev+git@drbeat.li> > --- > t/t8013-blame-ignore-revs.sh | 28 ++++++++++++++-------------- > 1 file changed, 14 insertions(+), 14 deletions(-) > > diff --git a/t/t8013-blame-ignore-revs.sh b/t/t8013-blame-ignore-revs.sh > index 9a03b0f361ff..05213d13f30f 100755 > --- a/t/t8013-blame-ignore-revs.sh > +++ b/t/t8013-blame-ignore-revs.sh > @@ -25,11 +25,11 @@ test_expect_success setup ' > > git blame --line-porcelain file >blame_raw && > > - grep -E "^[0-9a-f]+ [0-9]+ 1" blame_raw | sed -e "s/ .*//" >actual && > + sed -Ene "/^[0-9a-f]+ [0-9]+ 1/s/ .*//p" blame_raw >actual &&
Isn't -E a GNUism?
At least,
https://pubs.opengroup.org/onlinepubs/9699919799/utilities/sed.html
does not seem to have it (we may need to fix t6030 to rid its only existing use).