[PATCH] scripts: dtc-version: Don't show error messages

Dragan Simic dsimic at manjaro.org
Tue Feb 6 11:37:27 CET 2024


Hello Quentin,

On 2024-02-06 11:25, Quentin Schulz wrote:
> On 2/6/24 04:56, Dragan Simic wrote:
>> Prevent the error messages produced by which(1), such as the one 
>> quoted
>> below, from being visible in the build outputs.
>> 
>>      which: no dtc in (./scripts/dtc)
>> 
>> This makes the build outputs look a tiny bit cleaner.
>> 
>> Signed-off-by: Dragan Simic <dsimic at manjaro.org>
>> ---
>>   scripts/dtc-version.sh | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/scripts/dtc-version.sh b/scripts/dtc-version.sh
>> index 53ff868bcdce..6d3d7b68fdfa 100755
>> --- a/scripts/dtc-version.sh
>> +++ b/scripts/dtc-version.sh
>> @@ -15,7 +15,7 @@ if [ ${#dtc} -eq 0 ]; then
>>          exit 1
>>   fi
>> 
>> -if ! which $dtc >/dev/null ; then
>> +if ! which $dtc >/dev/null 2>/dev/null ; then
> 
> What about
> 
> which $dtc > /dev/null 2>&1
> 
> instead?

That's exactly what I intended to use, but IIRC some shells
or some older versions of some shells may not recognize that,
so I went with a more verbose, but slightly safer option.

> Reviewed-by: Quentin Schulz <quentin.schulz at theobroma-systems.com>

Thanks for your review.


More information about the U-Boot mailing list