> But I'd like to stay strict matching the filenames.
> 
> +     for _liba in /usr/lib/lib{c,crypto}; do
> +             _libas="$_libas $(ls $_liba.so.+([0-9.]).a | sort -V | tail -1)"
> +     done

Yes, this is indeed better.

So Klemens, can you write a patch that uses this and removes the then
unused _l. We can think about the hoisting in a second step.

> > +   _libas=${_libas# }
> 
> This would be another way of suppressing the blank right away.
> But it's not really necessary anyway, because the leading blank is
> removed in the list of the other for-loop.
> 
>       _libas="${_libas:+$_libas }$(ls $_liba.so.+([0-9.]).a | sort -V | tail 
> -1)"

While this is is clever, it does a bit too much in one line for my taste.

I'm fine with Klemens's version of zapping the blank after the for loop,
but we can also omit this, I don't really care.

>   
> >     # Remount read-write, if /usr/lib is on a read-only ffs filesystem.
> >     if [[ $_mp == *' type ffs '*'read-only'* ]]; then
> > 
> 
> -- 
> -=[rpe]=-
> 

Reply via email to