[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#464999: kdesdk-scripts: bashism in /bin/sh script



Package: kdesdk-scripts
Severity: important
Version: 3.5.8-1
User: debian-release@lists.debian.org
Usertags: goal-dash

Hello maintainer,

While performing an archive wide checkbashisms (from the 'devscripts' package) 
check I've found your package containing several /bin/sh scripts making 
use of a bashism.

checkbashisms' output:
> possible bashism in ./usr/bin/create_svnignore line 28 (trap with signal
> numbers): trap "rm svnignore.tmp" 1 2 15
> possible bashism in ./usr/bin/create_svnignore line 52
> (${parm/?/pat[/str]}): addignore ${ui/.ui/.h}
> possible bashism in ./usr/bin/create_svnignore line 53
> (${parm/?/pat[/str]}): addignore ${ui/.ui/.cpp}
> possible bashism in ./usr/bin/svnaddcurrentdir line 20 (should be 'b = a'):
> if [ "$answer" == "y" ]; then svn add $file; fi
> possible bashism in ./usr/bin/adddebug line 13 (echo -e):
>     *) echo -e "Usage: adddebug [-k] [-r] [-n]\n  -k: keep optimizations
> (removed by default)\n  -r: recursive (process all subdirectories)\n  -n:
> compile without NDEBUG and NO_DEBUG being defined (makes kdDebug calls
> work)"; exit 1;;


Not using bash as /bin/sh would lead to errors. Please be aware that although 
bash is currently the default /bin/sh there's a release goal for Lenny to 
make dash the default /bin/sh[1].

If you want more information about dash as /bin/sh, you can read:
http://lists.debian.org/debian-release/2008/01/msg00189.html
https://wiki.ubuntu.com/DashAsBinSh

[1]http://release.debian.org/lenny/goals.txt

Thank you,
-- 
Atomo64 - Raphael

Please avoid sending me Word, PowerPoint or Excel attachments.
See http://www.gnu.org/philosophy/no-word-attachments.html



Reply to: