[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