Re: [ptxdist] [APPLIED] ptxd_make_world_compile_commands_filter: Fix grep for uptodate check

[Michael Olbrich <m.olbrich@pengutronix.de>]

Thanks, applied as 609521efdc7d55d0fccca2c881f34adf23d22516.

Michael

[sent from post-receive hook]

On Fri, 17 Feb 2023 16:35:56 +0100, Uwe Kleine-König <u.kleine-koenig@pengutronix.de> wrote:
> The intend here is to check the return code of grep, no need to call [
> (i.e. test). This fixes an error message:
> 
> 	environment: line 10: [: syntax error: `-q' unexpected
> 
> Additionally add -e to force interpretation of
> "${PTXDIST_CROSS_CPPFLAGS}" as pattern. For me
> ${PTXDIST_CROSS_CPPFLAGS} starts with -isystem; without -e this is
> interpreted as option to grep instead of pattern which makes grep emit
> 
> 	grep: invalid option -- 't'
> 
> .)
> 
> Also add -F to make grep use a fixed-string compare (instead of
> interpreting ${PTXDIST_CROSS_CPPFLAGS} as regexp).
> 
> Fixes: ac0aac7fc727 ("impove compile_commands.json handling")
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> Acked-by: Alexander Dahl <post@lespocky.de>
> Message-Id: <20230210160258.12558-1-u.kleine-koenig@pengutronix.de>
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
> 
> diff --git a/scripts/lib/ptxd_make_world_compile_commands.sh b/scripts/lib/ptxd_make_world_compile_commands.sh
> index 56d7378fa28b..2703b5497799 100644
> --- a/scripts/lib/ptxd_make_world_compile_commands.sh
> +++ b/scripts/lib/ptxd_make_world_compile_commands.sh
> @@ -21,7 +21,7 @@ ptxd_make_world_compile_commands_filter() {
>      fi
>  
>      if [ "${src_cmds}" = "${dst_cmds}" ]; then
> -	if [ grep -q "${PTXDIST_CROSS_CPPFLAGS}" "${src_cmds}" ]; then
> +	if grep -q -F -e "${PTXDIST_CROSS_CPPFLAGS}" "${src_cmds}"; then
>  	    ptxd_warning "compile_commands.json is already up to date!"
>  	    return
>  	fi