Re: [Warzone-dev] Attached fix for VPATH builds (VERSION 2)

2008-01-08 Thread Gerard Krol
Kelly Anderson wrote:
 Hmmm, amazing, as soon as I saw the patch on the mailing list I 
 realized I could use a variable that's already set by 
 autoconf/automake instead of the PWD thing. :-D

 Here's the version that properly embraces autoconf/automake.

Seems to work fine here. Committed in r3406. Thanks for the patch!

- Gerard

___
Warzone-dev mailing list
Warzone-dev@gna.org
https://mail.gna.org/listinfo/warzone-dev


Re: [Warzone-dev] Attached fix for VPATH builds (VERSION 2)

2008-01-07 Thread Kelly Anderson
Hmmm, amazing, as soon as I saw the patch on the mailing list I realized 
I could use a variable that's already set by autoconf/automake instead 
of the PWD thing. :-D


Here's the version that properly embraces autoconf/automake.


Kelly Anderson wrote:
I've attached some subtle fixes to get VPATH builds working 
correctly.  I'm a firm believer in building outside the source tree 
(when possible).


Regards,

Kelly


___
Warzone-dev mailing list
Warzone-dev@gna.org
https://mail.gna.org/listinfo/warzone-dev


--- ./data/Makefile.am.orig 2008-01-07 13:43:35.573533866 -0700
+++ ./data/Makefile.am  2008-01-07 13:44:53.484548212 -0700
@@ -15,17 +15,17 @@ data: $(BASEARCHIVE) $(PATCHARCHIVE)
$(MAKE) $(AM_MAKEFLAGS) -C mods data
 
 $(BASEARCHIVE): $(BASELIST)
-   zip -ru0 $@ $(BASELIST) -x *svn*
+   (cd $(srcdir)  zip -ru0 $(abs_builddir)/$@ $(BASELIST) -x *svn*)
zip -T $@
 
 $(PATCHARCHIVE): $(PATCHLIST:%=mp/%)
-   cd mp  zip -ru0 ../$@ $(PATCHLIST) -x *svn*
+   (cd $(srcdir)/mp  zip -ru0 $(abs_builddir)/$@ $(PATCHLIST) -x *svn*)
zip -T $@
 
 install-data-local: $(BASEARCHIVE) $(PATCHARCHIVE)
$(MKDIR_P) $(DESTDIR)$(pkgdatadir)
-   $(INSTALL_DATA) $(srcdir)/$(BASEARCHIVE) 
$(DESTDIR)$(pkgdatadir)/$(BASEARCHIVE)
-   $(INSTALL_DATA) $(srcdir)/$(PATCHARCHIVE) 
$(DESTDIR)$(pkgdatadir)/$(PATCHARCHIVE)
+   $(INSTALL_DATA) $(BASEARCHIVE) $(DESTDIR)$(pkgdatadir)/$(BASEARCHIVE)
+   $(INSTALL_DATA) $(PATCHARCHIVE) $(DESTDIR)$(pkgdatadir)/$(PATCHARCHIVE)
 
 uninstall-local:
rm -f $(DESTDIR)$(pkgdatadir)/$(BASEARCHIVE)
--- ./data/mods/global/Makefile.am.orig 2008-01-07 13:43:32.996638143 -0700
+++ ./data/mods/global/Makefile.am  2008-01-07 13:52:09.821515289 -0700
@@ -13,22 +13,22 @@ CLEANFILES=$(AIVOLUTIONARCHIVE) $(GRIMAR
 data: $(AIVOLUTIONARCHIVE) $(GRIMARCHIVE) $(NEWST_TILESARCHIVE)
 
 $(AIVOLUTIONARCHIVE): $(AIVOLUTIONLIST:%=aivolution/%)
-   cd aivolution  zip -ru0 ../$@ $(AIVOLUTIONLIST) -x *svn*
+   (cd $(srcdir)/aivolution  zip -ru0 $(abs_builddir)/$@ 
$(AIVOLUTIONLIST) -x *svn*)
zip -T $@
 
 $(GRIMARCHIVE): $(GRIMLIST:%=grim/%)
-   cd grim  zip -ru0 ../$@ $(GRIMLIST) -x *svn*
+   (cd $(srcdir)/grim  zip -ru0 $(abs_builddir)/$@ $(GRIMLIST) -x *svn*)
zip -T $@
 
 $(NEWST_TILESARCHIVE): $(NEWST_TILESLIST:%=newst_tiles/%)
-   cd newst_tiles  zip -ru0 ../$@ $(NEWST_TILESLIST) -x *svn*
+   (cd $(srcdir)/newst_tiles  zip -ru0 $(abs_builddir)/$@ 
$(NEWST_TILESLIST) -x *svn*)
zip -T $@
 
 install-data-local: $(AIVOLUTIONARCHIVE) $(GRIMARCHIVE) $(NEWST_TILESARCHIVE)
$(MKDIR_P) $(DESTDIR)$(pkgdatadir)/mods/global
-   $(INSTALL_DATA) $(srcdir)/$(AIVOLUTIONARCHIVE) 
$(DESTDIR)$(pkgdatadir)/mods/global/$(AIVOLUTIONARCHIVE)
-   $(INSTALL_DATA) $(srcdir)/$(GRIMARCHIVE) 
$(DESTDIR)$(pkgdatadir)/mods/global/$(GRIMARCHIVE)
-   $(INSTALL_DATA) $(srcdir)/$(NEWST_TILESARCHIVE) 
$(DESTDIR)$(pkgdatadir)/mods/global/$(NEWST_TILESARCHIVE)
+   $(INSTALL_DATA) $(AIVOLUTIONARCHIVE) 
$(DESTDIR)$(pkgdatadir)/mods/global/$(AIVOLUTIONARCHIVE)
+   $(INSTALL_DATA) $(GRIMARCHIVE) 
$(DESTDIR)$(pkgdatadir)/mods/global/$(GRIMARCHIVE)
+   $(INSTALL_DATA) $(NEWST_TILESARCHIVE) 
$(DESTDIR)$(pkgdatadir)/mods/global/$(NEWST_TILESARCHIVE)
 
 uninstall-local:
rm -f $(DESTDIR)$(pkgdatadir)/mods/global/$(AIVOLUTIONARCHIVE)
___
Warzone-dev mailing list
Warzone-dev@gna.org
https://mail.gna.org/listinfo/warzone-dev