[PATCHv2 3/4] scripts/clean-package.sh: remove useless cat

Alberto Bursi bobafetthotmail at gmail.com
Sat Jul 11 20:12:58 EDT 2020



On 12/07/20 01:38, Paul Spooren wrote:
> Hi,
> 
> On 11.07.20 08:45, Rosen Penev wrote:
>> Found with shellcheck.
>>
>> Signed-off-by: Rosen Penev <rosenp at gmail.com>
>> ---
>>   v2: rebased
>>   scripts/clean-package.sh | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/scripts/clean-package.sh b/scripts/clean-package.sh
>> index 4b536c8a48..03e49df3e7 100755
>> --- a/scripts/clean-package.sh
>> +++ b/scripts/clean-package.sh
>> @@ -8,13 +8,13 @@ IFS=$'\n'
>>       echo "File/directory not found"
>>       exit 1
>>   }
>> -cat "$1" | (
>> +(
>>       cd "$2" || exit 1
>>       while read -r entry; do
>>           [ -n "$entry" ] || break
>>           [ ! -d "$entry" ] || [ -L "$entry" ] && rm -f "$entry"
>>       done
>> -)
>> +) < "$1"
>>   sort -r "$1" | (
>>       cd "$2" || exit 1
>>       while read -r entry; do
> I'd be curious whats wrong here, do you mind adding the SC numbers to 
> future improvements?
> 

It's said in the commit title, "remove useless cat"
which is a kind of well-known topic for shell experts.

"The purpose of cat is to concatenate (or "catenate") files. If it's 
only one file, concatenating it with nothing at all is a waste of time, 
and costs you a process."

 From "Useless Use of Cat Award"
http://porkmail.org/era/unix/award.html

-Alberto



More information about the openwrt-devel mailing list