Hi, Torsten Bögershausen wrote: > - /^\s*[^#]\s*which\s/ and err 'which is not portable (please use type)'; > + /^\s*[^#]\s*which\s+[-a-zA-Z0-9]+$/ and err 'which is not portable (please use type)'; Hmm. Neither the old version nor the new one matches what seem to be typical uses of 'which', based on a quick code search: if which sl >/dev/null 2>&1 then sl -l ... fi or if test -x "$(which sl 2>/dev/null)" then sl -l ... fi -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html