On Wednesday 25 February 2009 11:10:01 Timothy Redaelli wrote: > what do you think about checking for bashism on install_qa_check? > Obviously only for scripts with #!/bin/sh and #!/sbin/runscript as first > line. > > I think checkbashisms.pl [1] could be a good start point. > > [1] http://svn.debian.org/viewsvn/devscripts/trunk/scripts/checkbashisms.pl
i recall it being incorrect in some cases (it checked for what dash supports, not what POSIX supports), but that was a while ago, so maybe my experience is dated at this point. otherwise, integrating it sounds sane to me, and if we can push fixes back to debian, even better. a repoman check on files in files/ would be useful as well i'd think -mike