[PULL] Server build fixes
Keith Packard
keithp at keithp.com
Sat Feb 26 10:31:50 PST 2011
On Sat, 26 Feb 2011 15:33:20 +0100, Cyril Brulebois <kibi at debian.org> wrote:
> I fear that the sdksyms part is broken. If one builds with some
> options different than the ones used to generate the tarball, the file
> isn't generated again, leading to such issues.
Looks like sdksyms.c should be stuck in nodist_libloader_la_SOURCES so
that it doesn't get distributed. Nothing has changed here in a long
time, so I'm a bit confused why this cropped up so soon with 1.10.0;
the sdksyms.c stuff was added over two years ago.
A 'make clean' before 'make' will resolve the issue for the existing
1.10 package.
> A wild guess would be the “touch” added to configure leading to an
> up-to-date target. Also, I'm not sure it's a good idea to ship
> sdksyms.c in the tarball? (It was shipped already in rc3 and before,
> but that didn't lead to any issues.)
I read through the automake docs and they suggest placing a dependency
on the generated configuration header file. We've got several, but
there's the auto-generated do-not-use-config.h which holds all of the
cpp defines which sdksyms.c may use.
I think this is what we want:
diff --git a/hw/xfree86/loader/Makefile.am b/hw/xfree86/loader/Makefile.am
index 7f386cc..0e5b304 100644
--- a/hw/xfree86/loader/Makefile.am
+++ b/hw/xfree86/loader/Makefile.am
@@ -12,18 +12,21 @@ EXTRA_DIST = \
loaderProcs.h \
sdksyms.sh
+nodist_libloader_la_SOURCES = \
+ sdksyms.c
+
libloader_la_SOURCES = \
loader.c \
loaderProcs.h \
loadext.c \
loadmod.c \
- os.c \
- sdksyms.c
+ os.c
+
libloader_la_LIBADD = $(DLOPEN_LIBS)
CLEANFILES = sdksyms.c sdksyms.dep
-sdksyms.dep sdksyms.c: sdksyms.sh
+sdksyms.dep sdksyms.c: sdksyms.sh $(top_builddir)/include/do-not-use-config.h
CPP='$(CPP)' AWK='$(AWK)' $(srcdir)/sdksyms.sh $(top_srcdir) $(AM_CFLAGS) $(CFLAGS) $(INCLUDES)
SDKSYMS_DEP = sdksyms.dep
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110226/74f10a66/attachment.pgp>
More information about the xorg-devel
mailing list