summaryrefslogtreecommitdiffstats
path: root/3rdparty/local.mk
diff options
context:
space:
mode:
authorP. J. McDermott <pj@pehjota.net>2023-02-24 07:38:19 (EST)
committer P. J. McDermott <pj@pehjota.net>2023-02-24 08:21:08 (EST)
commit50e8259639936c867c486917aba8d9f748a35d2c (patch)
tree35604a6128063849e63dae61c86ae37579e7e047 /3rdparty/local.mk
parentbfe96dd587b1133220e24f123f500379db1bef8b (diff)
downloadoverworld-rpg-50e8259639936c867c486917aba8d9f748a35d2c.zip
overworld-rpg-50e8259639936c867c486917aba8d9f748a35d2c.tar.gz
overworld-rpg-50e8259639936c867c486917aba8d9f748a35d2c.tar.bz2
build: Use AC_CONFIG_SUBDIRS()
But explain why Automake's SUBDIRS can't be used. Also, exclude instructions to generate files from generated files.
Diffstat (limited to '3rdparty/local.mk')
-rw-r--r--3rdparty/local.mk11
1 files changed, 11 insertions, 0 deletions
diff --git a/3rdparty/local.mk b/3rdparty/local.mk
index 2fcd50b..40a8490 100644
--- a/3rdparty/local.mk
+++ b/3rdparty/local.mk
@@ -1,8 +1,19 @@
if EMBEDDED_XCF2PNG
+## Yes, SUBDIRS should be used instead, except that these lines are still needed
+## in case boukengine's default verbosity differs from that of xcftools:
VV = $(VV_$(V))
VV_ = $(AM_DEFAULT_VERBOSITY)
VV_1 = 1
VV_0 = 0
+## Plus this is needed:
+## AM_MAKEFLAGS = V=$(VV)
+## But this is still not enough, for two reasons:
+## 1. xcftools needs a "distdir" target [1] in its makefile, which would mean
+## copying the source tree, which xcftools currently avoids with a symbolic
+## link and manifest file, and
+## 2. Installing boukengine also installs xcftools, which is unnecessary.
+## [1]:
+## https://gnu.org/s/automake/manual/html_node/Third_002dParty-Makefiles.html
$(XCF2PNG):
$(AM_V_at)(cd %reldir%/xcftools/ && $(MAKE) V=$(VV))
clean-local: