Re: [Freeciv-Dev] (PR#40349) else clauses in Makefile.am conditionals
URL: http://bugs.freeciv.org/Ticket/Display.html?id=40349 This is now handled in gna bug #16142. - ML ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
Re: [Freeciv-Dev] (PR#40349) else clauses in Makefile.am conditionals
URL: http://bugs.freeciv.org/Ticket/Display.html?id=40349 2008/7/13 Daniel Markstedt: [EMAIL PROTECTED] - Tue Jul 01 20:47:26 2008]: 2008/7/1 Marko Lindqvist: 2008/7/1 Jason Dorje Short: make dist assumes all conditionals pass to find out what needs to be distributed. I believe make dist collects files from both conditional pass and conditional fails possibilities. In the case of ftwl-directory, I suspect that old automake is not properly handling conditionally built library (even though it's in conditional pass branch) and subsequently not adding source files it would require. Attached patch removes conditionality from library and makes entering to the directory in the first place conditional instead. Conditional subdirs are already used in other places. This works for me (automake 1.10.1) at least. I can compile both ftwl-client and other clients, and 'make dist' adds ftwl-files no matter which client is enabled from configure. I cannot currently test this patch, as configuring S2_0 with --enable- ftwl fails both with and without it. The last messages are: checking for unistd.h... yes configure: error: bad value yes for --enable-ftwl Try --enable-ftwl=x11 or --enable-ftwl=sdl Please test also 'make dist' when ftwl is *not* selected as client at configure time. 'make dist' should make full tarball, no matter the configure time selections. - ML ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
Re: [Freeciv-Dev] (PR#40349) else clauses in Makefile.am conditionals
URL: http://bugs.freeciv.org/Ticket/Display.html?id=40349 2008/7/1 Jason Dorje Short: I haven't looked up documentation on this but my feeling is these are bad - and, in every case I could find, unnecessary. Sometimes automake considers not setting variable at all an error. At the very least you are unable to use '+=' for unset variable. make dist assumes all conditionals pass to find out what needs to be distributed. I believe make dist collects files from both conditional pass and conditional fails possibilities. - ML ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
Re: [Freeciv-Dev] (PR#40349) else clauses in Makefile.am conditionals
URL: http://bugs.freeciv.org/Ticket/Display.html?id=40349 2008/7/1 Marko Lindqvist: 2008/7/1 Jason Dorje Short: make dist assumes all conditionals pass to find out what needs to be distributed. I believe make dist collects files from both conditional pass and conditional fails possibilities. In the case of ftwl-directory, I suspect that old automake is not properly handling conditionally built library (even though it's in conditional pass branch) and subsequently not adding source files it would require. Attached patch removes conditionality from library and makes entering to the directory in the first place conditional instead. Conditional subdirs are already used in other places. This works for me (automake 1.10.1) at least. I can compile both ftwl-client and other clients, and 'make dist' adds ftwl-files no matter which client is enabled from configure. - ML diff -Nurd -X.diff_ignore freeciv/utility/ftwl/Makefile.am freeciv/utility/ftwl/Makefile.am --- freeciv/utility/ftwl/Makefile.am2007-03-05 19:15:19.0 +0200 +++ freeciv/utility/ftwl/Makefile.am2008-07-01 21:57:16.0 +0300 @@ -1,12 +1,6 @@ ## Process this file with automake to produce Makefile.in -if FTWL -FTWLLIB=libftwl.a -else -FTWLLIB= -endif - -noinst_LIBRARIES = $(FTWLLIB) +noinst_LIBRARIES = libftwl.a AM_CPPFLAGS = -I. -I$(top_srcdir)/utility -I../../intl -I/usr/include/freetype2 @CLIENT_CFLAGS@ @@ -17,6 +11,7 @@ ALL_BACKEND_X11_FILES=$(BACKEND_COMMON_FILES) be_x11_ximage.c ALL_BACKEND_SDL_FILES=$(BACKEND_COMMON_FILES) be_sdl.c + ALL_OTHER_FILES= \ back_end.h \ common_types.c \ @@ -43,11 +38,8 @@ if FTWL_SDL BACKEND_SDL_FILES=$(ALL_BACKEND_SDL_FILES) endif -if FTWL -OTHER_FILES=$(ALL_OTHER_FILES) -endif ## Above, note -I../../intl instead of -I$(top_srdir)/intl is deliberate. libftwl_a_SOURCES = $(BACKEND_X11_FILES) \ - $(BACKEND_SDL_FILES) $(OTHER_FILES) + $(BACKEND_SDL_FILES) $(ALL_OTHER_FILES) diff -Nurd -X.diff_ignore freeciv/utility/Makefile.am freeciv/utility/Makefile.am --- freeciv/utility/Makefile.am 2007-03-05 19:15:20.0 +0200 +++ freeciv/utility/Makefile.am 2008-07-01 21:54:42.0 +0300 @@ -1,6 +1,8 @@ ## Process this file with automake to produce Makefile.in -SUBDIRS=ftwl +if FTWL +SUBDIRS = ftwl +endif noinst_LIBRARIES = libcivutility.a ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev