Bug#445863: [cl-debian] Bug#445863: bashism in /usr/lib/emacsen-common/packages/install/slime

Pierre THIERRY nowhere.man at levallois.eu.org
Tue Oct 9 15:56:29 UTC 2007


tag 445863 + patch
thanks

Scribit Géraud Meyer dies 08/10/2007 hora 19:54:
> == is not an operator for /usr/bin/[. It is one for the bash keyword [[.
> [ after an if in bash probably also is interpreted directly by bash.

The == operator seems the only bashism to me. if [ "a" = $1 ]; then ...
is perfectly accepted by dash. So the correction should just be:

diff -r 4f0b69b9911b debian/emacsen-install
--- a/debian/emacsen-install    Tue Oct 09 17:48:14 2007 +0200
+++ b/debian/emacsen-install    Tue Oct 09 17:52:57 2007 +0200
@@ -32,7 +32,7 @@ fi

 # in case of a failed installation .el symlinks are left behind, remove them
 if [ -d "${ELCDIR}" ] ; then
-  if [ "${ELCDIR}" == "" ] ; then
+  if [ "${ELCDIR}" = "" ] ; then
     echo ELCDIR is empty, emergency stop
     exit 1
   else

Quickly,
Pierre
-- 
nowhere.man at levallois.eu.org
OpenPGP 0xD9D50D8A
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <https://mailman.common-lisp.net/pipermail/cl-debian/attachments/20071009/11cf6011/attachment.sig>


More information about the Cl-debian mailing list