On Mon, Jan 12, 2009 at 6:57 AM, Julien Cristau <[email protected]> wrote: > On Mon, Jan 12, 2009 at 06:49:42 -0800, Dan Nicholson wrote: > >> Looks good. Oh, it looks like the binary [[ ]] shell operators slipped >> in there at some point. Those are bash only, I think, and should be >> fixed. But that's not part of this patch. >> > It's the usual [ ], except this is m4 so there's an extra pair of [ ].
Looks like you're right. Still, it's forking to cut get the first character, when the whole thing could be done in a simple case statement. -XKBOUTPUT_FIRSTCHAR=`echo $XKBOUTPUT | cut -b 1` - -if [[ x$XKBOUTPUT_FIRSTCHAR != x/ ]] ; then - XKBOUTPUT="$XKB_BASE_DIRECTORY/$XKBOUTPUT" -fi +case "$XKBOUTPUT" in +/*) ;; +*) XKBOUTPUT="$XKB_BASE_DIRECTORY/$XKBOUTPUT" ;; +esac -- Dan _______________________________________________ xorg mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/xorg
