Re: [DTrace-devel] [PATCH] Adjust paths for common executables

Reviewed-by: Eugene Loh <[email protected]>
I'm okay with this, but I don't really know the underlying issues.  Why 
for bash and sed but not for other utilities (gawk or whatever)?

On 2/14/26 11:12, Kris Van Hees via DTrace-devel wrote:
> Favour /bin/* over /usr/bin/* for things like bash and sed.
>
> Signed-off-by: Kris Van Hees <[email protected]>
> ---
>   test/unittest/arrays/err.D_UNKNOWN.uregs_toobig.aarch64.x | 2 +-
>   test/unittest/arrays/err.D_UNKNOWN.uregs_toobig.x86_64.x  | 2 +-
>   test/unittest/codegen/err.deref_0.r.p                     | 2 +-
>   test/unittest/codegen/err.deref_1.r.p                     | 2 +-
>   test/unittest/codegen/err.deref_i0.r.p                    | 2 +-
>   test/unittest/codegen/err.deref_i1.r.p                    | 2 +-
>   test/unittest/codegen/err.deref_string-assoc.r.p          | 2 +-
>   test/unittest/codegen/err.deref_string-gvar.r.p           | 2 +-
>   test/unittest/codegen/err.deref_string-lvar.r.p           | 2 +-
>   test/unittest/codegen/err.deref_string-tvar.r.p           | 2 +-
>   test/unittest/preprocessor/err.ifdefincomp.r.p            | 2 +-
>   test/unittest/preprocessor/err.incompelse.r.p             | 2 +-
>   test/utils/clean_probes.sh                                | 2 +-
>   13 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/test/unittest/arrays/err.D_UNKNOWN.uregs_toobig.aarch64.x b/test/unittest/arrays/err.D_UNKNOWN.uregs_toobig.aarch64.x
> index 627beb32..8d60c971 100755
> --- a/test/unittest/arrays/err.D_UNKNOWN.uregs_toobig.aarch64.x
> +++ b/test/unittest/arrays/err.D_UNKNOWN.uregs_toobig.aarch64.x
> @@ -1,4 +1,4 @@
> -#!/usr/bin/bash
> +#!/bin/bash
>   
>   [ `uname -m` = "aarch64" ] && exit 0
>   exit 2
> diff --git a/test/unittest/arrays/err.D_UNKNOWN.uregs_toobig.x86_64.x b/test/unittest/arrays/err.D_UNKNOWN.uregs_toobig.x86_64.x
> index 12759d64..975b71f6 100755
> --- a/test/unittest/arrays/err.D_UNKNOWN.uregs_toobig.x86_64.x
> +++ b/test/unittest/arrays/err.D_UNKNOWN.uregs_toobig.x86_64.x
> @@ -1,4 +1,4 @@
> -#!/usr/bin/bash
> +#!/bin/bash
>   
>   [ `uname -m` = "x86_64" ] && exit 0
>   exit 2
> diff --git a/test/unittest/codegen/err.deref_0.r.p b/test/unittest/codegen/err.deref_0.r.p
> index 68ebc99b..f7dd7d99 100755
> --- a/test/unittest/codegen/err.deref_0.r.p
> +++ b/test/unittest/codegen/err.deref_0.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/sed -f
> +#!/bin/sed -f
>   
>   # runtest.sh looks for "0x" to filter out pointer values.
>   # Strip the 0x so that the illegal address will not be filtered out;
> diff --git a/test/unittest/codegen/err.deref_1.r.p b/test/unittest/codegen/err.deref_1.r.p
> index 68ebc99b..f7dd7d99 100755
> --- a/test/unittest/codegen/err.deref_1.r.p
> +++ b/test/unittest/codegen/err.deref_1.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/sed -f
> +#!/bin/sed -f
>   
>   # runtest.sh looks for "0x" to filter out pointer values.
>   # Strip the 0x so that the illegal address will not be filtered out;
> diff --git a/test/unittest/codegen/err.deref_i0.r.p b/test/unittest/codegen/err.deref_i0.r.p
> index 68ebc99b..f7dd7d99 100755
> --- a/test/unittest/codegen/err.deref_i0.r.p
> +++ b/test/unittest/codegen/err.deref_i0.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/sed -f
> +#!/bin/sed -f
>   
>   # runtest.sh looks for "0x" to filter out pointer values.
>   # Strip the 0x so that the illegal address will not be filtered out;
> diff --git a/test/unittest/codegen/err.deref_i1.r.p b/test/unittest/codegen/err.deref_i1.r.p
> index 68ebc99b..f7dd7d99 100755
> --- a/test/unittest/codegen/err.deref_i1.r.p
> +++ b/test/unittest/codegen/err.deref_i1.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/sed -f
> +#!/bin/sed -f
>   
>   # runtest.sh looks for "0x" to filter out pointer values.
>   # Strip the 0x so that the illegal address will not be filtered out;
> diff --git a/test/unittest/codegen/err.deref_string-assoc.r.p b/test/unittest/codegen/err.deref_string-assoc.r.p
> index 68ebc99b..f7dd7d99 100755
> --- a/test/unittest/codegen/err.deref_string-assoc.r.p
> +++ b/test/unittest/codegen/err.deref_string-assoc.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/sed -f
> +#!/bin/sed -f
>   
>   # runtest.sh looks for "0x" to filter out pointer values.
>   # Strip the 0x so that the illegal address will not be filtered out;
> diff --git a/test/unittest/codegen/err.deref_string-gvar.r.p b/test/unittest/codegen/err.deref_string-gvar.r.p
> index 68ebc99b..f7dd7d99 100755
> --- a/test/unittest/codegen/err.deref_string-gvar.r.p
> +++ b/test/unittest/codegen/err.deref_string-gvar.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/sed -f
> +#!/bin/sed -f
>   
>   # runtest.sh looks for "0x" to filter out pointer values.
>   # Strip the 0x so that the illegal address will not be filtered out;
> diff --git a/test/unittest/codegen/err.deref_string-lvar.r.p b/test/unittest/codegen/err.deref_string-lvar.r.p
> index 68ebc99b..f7dd7d99 100755
> --- a/test/unittest/codegen/err.deref_string-lvar.r.p
> +++ b/test/unittest/codegen/err.deref_string-lvar.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/sed -f
> +#!/bin/sed -f
>   
>   # runtest.sh looks for "0x" to filter out pointer values.
>   # Strip the 0x so that the illegal address will not be filtered out;
> diff --git a/test/unittest/codegen/err.deref_string-tvar.r.p b/test/unittest/codegen/err.deref_string-tvar.r.p
> index 68ebc99b..f7dd7d99 100755
> --- a/test/unittest/codegen/err.deref_string-tvar.r.p
> +++ b/test/unittest/codegen/err.deref_string-tvar.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/sed -f
> +#!/bin/sed -f
>   
>   # runtest.sh looks for "0x" to filter out pointer values.
>   # Strip the 0x so that the illegal address will not be filtered out;
> diff --git a/test/unittest/preprocessor/err.ifdefincomp.r.p b/test/unittest/preprocessor/err.ifdefincomp.r.p
> index 7629ac59..68cbfdbb 100755
> --- a/test/unittest/preprocessor/err.ifdefincomp.r.p
> +++ b/test/unittest/preprocessor/err.ifdefincomp.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/sed -f
> +#!/bin/sed -f
>   # GCC 16 improved diagnostics formatting.  Until GCC 16 is the minimum
>   # supported, we have to sed it back again.
>   s,[‘’],",g
> diff --git a/test/unittest/preprocessor/err.incompelse.r.p b/test/unittest/preprocessor/err.incompelse.r.p
> index ae9a6833..64ecab97 100755
> --- a/test/unittest/preprocessor/err.incompelse.r.p
> +++ b/test/unittest/preprocessor/err.incompelse.r.p
> @@ -1,4 +1,4 @@
> -#!/usr/bin/sed -f
> +#!/bin/sed -f
>   # GCC 16 improved diagnostics formatting.  Until GCC 16 is the minimum
>   # supported, we have to sed it back again.
>   s,[‘’],,g
> diff --git a/test/utils/clean_probes.sh b/test/utils/clean_probes.sh
> index 16052cbc..6cc176a4 100755
> --- a/test/utils/clean_probes.sh
> +++ b/test/utils/clean_probes.sh
> @@ -1,4 +1,4 @@
> -#!/usr/bin/bash
> +#!/bin/bash
>   
>   TRACEFS=/sys/kernel/debug/tracing
>   EVENTS=${TRACEFS}/available_events
>
> _______________________________________________
> DTrace-devel mailing list
> [email protected]
> https://oss.oracle.com/mailman/listinfo/dtrace-devel