Bug#489558: cyrus-doc-2.2: bashism in /bin/sh script
Sven Mueller
sven at debian.org
Thu Jul 17 10:19:30 UTC 2008
Package cyrus-doc-2.2
Severity 489558 wishlist
thanks
Raphael Geissert schrieb:
> Package: cyrus-doc-2.2
> Severity: important
> Version: 2.2.13-14
> User: debian-release at lists.debian.org
> Usertags: goal-dash
>
> While performing an archive wide checkbashisms (from the 'devscripts' package)
> check I've found your package containing one or more /bin/sh scripts making
> use of bashisms.
>
> checkbashisms' output:
>> possible bashism in ./usr/share/doc/cyrus-doc-2.2/examples/mboxtocyrus line
>> 57 (should be 'b = a'): if [ "$MAILBOX" == "" ]; then
>> possible bashism in ./usr/share/doc/cyrus-doc-2.2/examples/mboxtocyrus line
>> 61 (should be 'b = a'): if [ "$CYRUSFOLDER" == "" ]; then
Note two things here:
1) This is a _possible_ bashisms. I'm not sure wether it really is one
or not. All instances of [ (aka. test) I tried (bash, tcsh, dash)
supported the syntax used.
2) This is an example script in a doc package, which is unlikely to work
unmodified on any package users system anyway.
Severity important is therefor highly exagerated.
I won't be marking this as wontfix for now, but I intend to do so.
Please check your bug reports before marking them with such a high severity.
Thanks.
Sven
More information about the Pkg-Cyrus-imapd-Debian-devel
mailing list