Please fix your scripts and makefile script fragments!

Note the following well known behaviour of test(1):

        09:02 [252] $ if test -n ; then echo true; else echo false; fi
        true
        09:02 [253] $ if test -n ""; then echo true; else echo false; fi
        false
        09:02 [254] $ if test -n "blah"; then echo true; else echo false; fi
        true
        09:02 [255] $ 

As a result not that patches such as the following are necessary for
_ALL_ of the plugins makefiles, at least as of 0.99.7:

--- plugins/ethercat/Makefile.am.orig   Mon Dec 17 20:14:47 2007
+++ plugins/ethercat/Makefile.am        Wed Feb 27 08:57:33 2008
@@ -81,7 +81,7 @@ LIBS =
 #
 plugin.c: $(DISSECTOR_SRC) $(top_srcdir)/tools/make-dissector-reg \
     $(top_srcdir)/tools/make-dissector-reg.py
-       @if test -n $(PYTHON); then \
+       @if test -n "$(PYTHON)"; then \
                echo Making plugin.c with python ; \
                $(PYTHON) $(top_srcdir)/tools/make-dissector-reg.py $(srcdir) \
                    plugin $(DISSECTOR_SRC) ; \



-- 
                                                Greg A. Woods

H:+1 416 218-0098 W:+1 416 489-5852 x122 VE3TCP RoboHack <[EMAIL PROTECTED]>
Planix, Inc. <[EMAIL PROTECTED]>       Secrets of the Weird <[EMAIL PROTECTED]>

Attachment: pgp1NxoosK47Q.pgp
Description: PGP signature

_______________________________________________
Wireshark-dev mailing list
[email protected]
http://www.wireshark.org/mailman/listinfo/wireshark-dev

Reply via email to