Thanks Dominik.
Dominik Vogt wrote:
They changed something with quoting in autoconf/automake a while
ago. Unfortunately you have to test with the old and new auto*
versions to make sure some things in configure work fine.
Odd. It worked for me, too, using the same version of
autoconf that'd previously generated the configure. Guess
that must've changed on your system and not mine. Thanks
for fixing it!
There were a couple issues with your changes to libs/FScreen.c.
I don't know that I "agree" with the "all code should be
compiled" opinion, but that's certainly your call to make.
And the #ifdef'd code is now more sensible than it was, and
than I'd made it. Thanks.
But, there was one typo (capitalization error), and one
error in the #def's. Solaris' xinerama.h does *not* define
XineramaScreenInfo, since Solaris' xinerama system API is
*completely* different. I just moved the relevant #def out
one level. A patch to libs/FScreen.c is attached. It will
now link both with and without the Solaris xinerama.h
header.
Thanks again!
- Chris
Index: libs/FScreen.c
===================================================================
RCS file: /home/cvs/fvwm/fvwm/libs/FScreen.c,v
retrieving revision 1.24
diff -u -r1.24 FScreen.c
--- libs/FScreen.c 26 Aug 2004 09:40:42 -0000 1.24
+++ libs/FScreen.c 26 Aug 2004 15:52:53 -0000
@@ -73,10 +73,10 @@
# define FScreenHaveXinerama 1
# ifdef HAVE_SOLARIS_XINERAMA
# define FScreenHaveSolarisXinerama 1
+# define FSCREEN_NEED_SCREENINFO
# ifdef HAVE_SOLARIS_XINERAMA_H
# include <X11/extensions/xinerama.h>
# else
-# define FSCREEN_NEED_SCREENINFO
# define FSCREEN_NEED_SOLARIS_PROTOTYPES
# endif
# else
@@ -114,7 +114,7 @@
#if FScreenHaveXinerama
# if FScreenHaveSolarisXinerama
# define XineramaQueryExtension(d,b,c) 1 /* Lie, for now */
-# define XineramaIsActive(d) Xineramagetstate((d),0)
+# define XineramaIsActive(d) XineramaGetState((d),0)
# endif
#else
# define XineramaQueryExtension(da, b, c) 0
