* [PATCH] Makefile: Fix grep warning @ 2023-08-10 11:02 Andrew Jones 2023-08-10 11:32 ` Andreas Schwab 0 siblings, 1 reply; 5+ messages in thread From: Andrew Jones @ 2023-08-10 11:02 UTC (permalink / raw) To: opensbi grep (at least my version, grep-3.8-3.fc38.x86_64) warns with "grep: warning: stray \ before -". Fix the warning by making the command line input to grep less ambiguous. Signed-off-by: Andrew Jones <ajones@ventanamicro.com> --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 730dbd910e51..03768a51f710 100644 --- a/Makefile +++ b/Makefile @@ -168,7 +168,7 @@ endif OPENSBI_LD_PIE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) $(USE_LD_FLAG) -fPIE -nostdlib -Wl,-pie -x c /dev/null -o /dev/null >/dev/null 2>&1 && echo y || echo n) # Check whether the compiler supports -m(no-)save-restore -CC_SUPPORT_SAVE_RESTORE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -mno-save-restore -x c /dev/null -o /dev/null 2>&1 | grep "\-save\-restore" >/dev/null && echo n || echo y) +CC_SUPPORT_SAVE_RESTORE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -mno-save-restore -x c /dev/null -o /dev/null 2>&1 | grep -- "-save-restore" >/dev/null && echo n || echo y) # Check whether the assembler and the compiler support the Zicsr and Zifencei extensions CC_SUPPORT_ZICSR_ZIFENCEI := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -march=rv$(OPENSBI_CC_XLEN)imafd_zicsr_zifencei -x c /dev/null -o /dev/null 2>&1 | grep "zicsr\|zifencei" > /dev/null && echo n || echo y) -- 2.41.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] Makefile: Fix grep warning 2023-08-10 11:02 [PATCH] Makefile: Fix grep warning Andrew Jones @ 2023-08-10 11:32 ` Andreas Schwab 2023-08-10 11:40 ` Andrew Jones 0 siblings, 1 reply; 5+ messages in thread From: Andreas Schwab @ 2023-08-10 11:32 UTC (permalink / raw) To: opensbi On Aug 10 2023, Andrew Jones wrote: > grep (at least my version, grep-3.8-3.fc38.x86_64) warns with > "grep: warning: stray \ before -". Fix the warning by making > the command line input to grep less ambiguous. > > Signed-off-by: Andrew Jones <ajones@ventanamicro.com> > --- > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index 730dbd910e51..03768a51f710 100644 > --- a/Makefile > +++ b/Makefile > @@ -168,7 +168,7 @@ endif > OPENSBI_LD_PIE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) $(USE_LD_FLAG) -fPIE -nostdlib -Wl,-pie -x c /dev/null -o /dev/null >/dev/null 2>&1 && echo y || echo n) > > # Check whether the compiler supports -m(no-)save-restore > -CC_SUPPORT_SAVE_RESTORE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -mno-save-restore -x c /dev/null -o /dev/null 2>&1 | grep "\-save\-restore" >/dev/null && echo n || echo y) > +CC_SUPPORT_SAVE_RESTORE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -mno-save-restore -x c /dev/null -o /dev/null 2>&1 | grep -- "-save-restore" >/dev/null && echo n || echo y) Please use grep -e ... instead. -- Andreas Schwab, SUSE Labs, schwab at suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 "And now for something completely different." ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] Makefile: Fix grep warning 2023-08-10 11:32 ` Andreas Schwab @ 2023-08-10 11:40 ` Andrew Jones 2023-08-10 11:46 ` Andreas Schwab 0 siblings, 1 reply; 5+ messages in thread From: Andrew Jones @ 2023-08-10 11:40 UTC (permalink / raw) To: opensbi On Thu, Aug 10, 2023 at 01:32:49PM +0200, Andreas Schwab wrote: > On Aug 10 2023, Andrew Jones wrote: > > > grep (at least my version, grep-3.8-3.fc38.x86_64) warns with > > "grep: warning: stray \ before -". Fix the warning by making > > the command line input to grep less ambiguous. > > > > Signed-off-by: Andrew Jones <ajones@ventanamicro.com> > > --- > > Makefile | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/Makefile b/Makefile > > index 730dbd910e51..03768a51f710 100644 > > --- a/Makefile > > +++ b/Makefile > > @@ -168,7 +168,7 @@ endif > > OPENSBI_LD_PIE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) $(USE_LD_FLAG) -fPIE -nostdlib -Wl,-pie -x c /dev/null -o /dev/null >/dev/null 2>&1 && echo y || echo n) > > > > # Check whether the compiler supports -m(no-)save-restore > > -CC_SUPPORT_SAVE_RESTORE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -mno-save-restore -x c /dev/null -o /dev/null 2>&1 | grep "\-save\-restore" >/dev/null && echo n || echo y) > > +CC_SUPPORT_SAVE_RESTORE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -mno-save-restore -x c /dev/null -o /dev/null 2>&1 | grep -- "-save-restore" >/dev/null && echo n || echo y) > > Please use grep -e ... instead. Sure, but I'd like to put a justification in the commit message, something like, Use -e, as opposed to --, because... Thanks, drew ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] Makefile: Fix grep warning 2023-08-10 11:40 ` Andrew Jones @ 2023-08-10 11:46 ` Andreas Schwab 2023-08-10 12:11 ` Andrew Jones 0 siblings, 1 reply; 5+ messages in thread From: Andreas Schwab @ 2023-08-10 11:46 UTC (permalink / raw) To: opensbi On Aug 10 2023, Andrew Jones wrote: > On Thu, Aug 10, 2023 at 01:32:49PM +0200, Andreas Schwab wrote: >> On Aug 10 2023, Andrew Jones wrote: >> >> > grep (at least my version, grep-3.8-3.fc38.x86_64) warns with >> > "grep: warning: stray \ before -". Fix the warning by making >> > the command line input to grep less ambiguous. >> > >> > Signed-off-by: Andrew Jones <ajones@ventanamicro.com> >> > --- >> > Makefile | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/Makefile b/Makefile >> > index 730dbd910e51..03768a51f710 100644 >> > --- a/Makefile >> > +++ b/Makefile >> > @@ -168,7 +168,7 @@ endif >> > OPENSBI_LD_PIE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) $(USE_LD_FLAG) -fPIE -nostdlib -Wl,-pie -x c /dev/null -o /dev/null >/dev/null 2>&1 && echo y || echo n) >> > >> > # Check whether the compiler supports -m(no-)save-restore >> > -CC_SUPPORT_SAVE_RESTORE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -mno-save-restore -x c /dev/null -o /dev/null 2>&1 | grep "\-save\-restore" >/dev/null && echo n || echo y) >> > +CC_SUPPORT_SAVE_RESTORE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -mno-save-restore -x c /dev/null -o /dev/null 2>&1 | grep -- "-save-restore" >/dev/null && echo n || echo y) >> >> Please use grep -e ... instead. > > Sure, but I'd like to put a justification in the commit message, something > like, Use -e, as opposed to --, because... It's portable. -- Andreas Schwab, SUSE Labs, schwab at suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 "And now for something completely different." ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] Makefile: Fix grep warning 2023-08-10 11:46 ` Andreas Schwab @ 2023-08-10 12:11 ` Andrew Jones 0 siblings, 0 replies; 5+ messages in thread From: Andrew Jones @ 2023-08-10 12:11 UTC (permalink / raw) To: opensbi On Thu, Aug 10, 2023 at 01:46:19PM +0200, Andreas Schwab wrote: > On Aug 10 2023, Andrew Jones wrote: > > > On Thu, Aug 10, 2023 at 01:32:49PM +0200, Andreas Schwab wrote: > >> On Aug 10 2023, Andrew Jones wrote: > >> > >> > grep (at least my version, grep-3.8-3.fc38.x86_64) warns with > >> > "grep: warning: stray \ before -". Fix the warning by making > >> > the command line input to grep less ambiguous. > >> > > >> > Signed-off-by: Andrew Jones <ajones@ventanamicro.com> > >> > --- > >> > Makefile | 2 +- > >> > 1 file changed, 1 insertion(+), 1 deletion(-) > >> > > >> > diff --git a/Makefile b/Makefile > >> > index 730dbd910e51..03768a51f710 100644 > >> > --- a/Makefile > >> > +++ b/Makefile > >> > @@ -168,7 +168,7 @@ endif > >> > OPENSBI_LD_PIE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) $(USE_LD_FLAG) -fPIE -nostdlib -Wl,-pie -x c /dev/null -o /dev/null >/dev/null 2>&1 && echo y || echo n) > >> > > >> > # Check whether the compiler supports -m(no-)save-restore > >> > -CC_SUPPORT_SAVE_RESTORE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -mno-save-restore -x c /dev/null -o /dev/null 2>&1 | grep "\-save\-restore" >/dev/null && echo n || echo y) > >> > +CC_SUPPORT_SAVE_RESTORE := $(shell $(CC) $(CLANG_TARGET) $(RELAX_FLAG) -nostdlib -mno-save-restore -x c /dev/null -o /dev/null 2>&1 | grep -- "-save-restore" >/dev/null && echo n || echo y) > >> > >> Please use grep -e ... instead. > > > > Sure, but I'd like to put a justification in the commit message, something > > like, Use -e, as opposed to --, because... > > It's portable. Hmm, shellcheck doesn't complain, shellcheck -s sh <(echo "echo foo | grep -- foo") but maybe the concern is that plopping the "grep -- foo" string into random shells may not always work, since a random shell could assume the -- is for itself to interpret, rather than for grep? Anyway, v2 coming up, but without a justification for -e vs. --, since it's not clear to me. Thanks, drew ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-08-10 12:11 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-08-10 11:02 [PATCH] Makefile: Fix grep warning Andrew Jones 2023-08-10 11:32 ` Andreas Schwab 2023-08-10 11:40 ` Andrew Jones 2023-08-10 11:46 ` Andreas Schwab 2023-08-10 12:11 ` Andrew Jones
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.