> 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]=- >