-lX11 is added in a wrong place

[Alexandre Oliva]

Pavel Roskin <Pavel.Roskin at ecsoft.co.uk> writes:

> AC_PATH_X [...] may return $have_x==yes but $x_libraries empty,
> [...] we still need -lX11

The patch is wrong, because, if x_libraries is *not* empty, it causes
X_LIBRARIES not to contain -lX11.  How about this one?  (I have just
edited yours, so it may be broken, but you get the idea)

--- kaffe-snap/configure.in.orig        Wed Jul 15 00:49:44 1998
+++ kaffe-snap/configure.in     Thu Jul 16 16:09:07 1998
@@ -412,6 +412,8 @@
 fi
 if test "${x_libraries}" != NONE && test -n "${x_libraries}"; then
        X_LIBRARIES=-L`echo ${x_libraries} | sed -e "s/:/ -L/g"`
+fi
+if test "$have_x" = "yes" ; then
        X_LIBRARIES="$X_LIBRARIES -lX11"
 fi
 if test "${x_includes}" != NONE && test -n "${x_includes}"; then

-- 
Alexandre Oliva
mailto:oliva at dcc.unicamp.br mailto:aoliva at acm.org
http://www.dcc.unicamp.br/~oliva
Universidade Estadual de Campinas, SP, Brasil