Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
unaccent: Tweak value of PYTHON when building without Python support
authorMichael Paquier <michael@paquier.xyz>
Wed, 27 Sep 2023 05:40:23 +0000 (14:40 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 27 Sep 2023 05:40:23 +0000 (14:40 +0900)
As coded, the module's Makefile would fail to set a value for PYTHON as
it checked if the variable is defined.  When compiling without
--with-python, PYTHON is defined and set to an empty value, so the
existing check is not able to do its work.

This commit switches the rule to check if the value is empty rather than
defined, allowing the generation of unaccent.rules even if --with-python
is not used as long as "python" exists.  BISON and FLEX do the same in
pgxs.mk, for instance.

Thinko in f85a485f89e2.

Author: Japin Li
Discussion: https://postgr.es/m/MEYP282MB1669F86C0DC7B4DC48489CB0B6C3A@MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM
Backpatch-through: 13

contrib/unaccent/Makefile

index 652a3e774c072ef7934d74fb7d1eed279ccb09b3..3ff49ba1e93cd2fb99e4a5fa63db2566ce686b91 100644 (file)
@@ -26,7 +26,9 @@ endif
 update-unicode: $(srcdir)/unaccent.rules
 
 # Allow running this even without --with-python
-PYTHON ?= python
+ifeq ($(PYTHON),)
+PYTHON = python
+endif
 
 $(srcdir)/unaccent.rules: generate_unaccent_rules.py ../../src/common/unicode/UnicodeData.txt Latin-ASCII.xml
    $(PYTHON) $< --unicode-data-file $(word 2,$^) --latin-ascii-file $(word 3,$^) >$@