tag 486050 patch
thanks

On Thu, Jun 12, 2008 at 11:41:00PM -0500, Raphael Geissert wrote:

||  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/bin/tex2eps.sh line 16 (unsafe echo with
||  > backslash):
||  > echo '\nopagenumbers' > $tmp.tex

tex2eps.sh will indeed fail when interpreted by dash: it replaces the \n
with a newline when echoing.

The following patch should fix the problem:

diff -rud mgp-1.11b.orig/contrib/tex2eps.sh mgp-1.11b/contrib/tex2eps.sh
--- mgp-1.11b.orig/contrib/tex2eps.sh   1998-11-30 14:21:55.000000000 +0100
+++ mgp-1.11b/contrib/tex2eps.sh        2008-06-15 19:05:43.000000000 +0200
@@ -13,7 +13,7 @@
 # temporary filename (without .eps suffix)
 tmp=$1
 
-echo '\nopagenumbers' > $tmp.tex
+printf '\\nopagenumbers\n' > $tmp.tex
 cat >> $tmp.tex
 echo '\end' >> $tmp.tex
 tex $tmp.tex > /dev/null 2> /dev/null

Ciao. Vincent.



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to