Mon Aug 15 14:56:57 2005 Søren Sandmann <sandmann@redhat.com>

Remove the xkb* files.
Add xkbVT.o and friends as the last things on the link line so they will
    override the symbols in libxkb.a. Add a comment on how this situation
    might not be considered ideal.

hw/xfree86/Makefile.am

@@ -12,6 +12,9 @@ DIST_SUBDIRS = common ddc dummylib i2c x86emu int10 fbdevhw os-support \
 
 bin_PROGRAMS = Xorg
 
+AM_CFLAGS = @XORG_CFLAGS@
+INCLUDES = @XORG_INCS@ 
+
 Xorg_SOURCES = xorg.c
 
 DISTCLEANFILES = xorg.c
@@ -41,11 +44,32 @@ XORG_LIBS = \
             parser/libparser.a \
             $(OS_LIBS) \
             dummylib/libdummy.a \
-            dixmods/libdixmods.la \
+	    dixmods/libdixmods.la \
             @XORG_LIBS@
 
+#
+#  The xkbVT.c xkbKillSrv.c and xkbPrivate.c files used below are really
+#  the same files as ddxVT.c ddxKillSrv.c and ddxPrivate.c from
+#  $(top_srcdir)/xkb. However they are compiled with the flag 
+#  XF86DDXACTIONS which means they'll do stuff like Ctrl-Alt-Fx etc.
+#  They have to be last on the link line so that their symbols will
+#  override the ones from libxkb.a.
+#
+#  Unfortunately, those symbols can't be removed from libxkb because that
+#  causes other ddx'es to break. One possible fix would be to turn libxkb
+#  into shared a shared library and add API that would people linking to
+#  it to install callbacks for those actions.
+#  
+
+xkb%.c:
+	echo "#define XF86DDXACTIONS" > $@
+	echo "#include \"$(top_srcdir)/xkb/ddx$*.c\"" >> $@
+
 Xorg_LDADD = $(XORG_LIBS) \
              @XSERVER_LIBS@ \
+	     xkbVT.o \
+	     xkbKillSrv.o \
+	     xkbPrivate.o \
              -ldl
 
 Xorg_LDFLAGS = $(LD_EXPORT_SYMBOLS_FLAG)