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