Thread (4 messages) flat view 4 messages, 2 authors, 2016-06-15

Re: [PATCH] Remove "bashism" from contrib/thunderbird-patch-inline/appp.sh

From: Ángel González <hidden>
Date: 2016-06-15 22:50:55

Maxin john wrote:
Remove "bashism" and minor corrections for
contrib/thunderbird-patch-inline/appp.sh

Signed-off-by: Maxin B. John <redacted>
This is wrong.

You are replacing bash with sh:
-#!/bin/bash
+#!/bin/sh
but the script still uses bash-specific syntax (aka. bashishms):
+
 PATCH=$(zenity --file-selection)

 if [ "$?" != "0" ] ; then
So with your change the script won't be able to run on systems which
don't have bash as /bin/sh
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help