From: Peter Eisentraut <peter_e@gmx.net>
Date: Thu, 14 Oct 2010 19:15:46 +0000 (+0300)
Subject: Fix makefile logic to not break the build when xgettext is missing
X-Git-Tag: REL9_1_ALPHA2~84
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=30e749dece0e6502d4dd0a3b2892eab61f8c073b;p=postgresql

Fix makefile logic to not break the build when xgettext is missing

xgettext is only required when make init-po is run manually; it is not
required for a build.  The intent to handle that was already there, but
the ifdef's were in the wrong place.
---

diff --git a/src/nls-global.mk b/src/nls-global.mk
index 9ba2cfd94e..32b3c0fbd6 100644
--- a/src/nls-global.mk
+++ b/src/nls-global.mk
@@ -45,22 +45,26 @@ all-po: $(MO_FILES)
 %.mo: %.po
 	$(MSGFMT) -o $@ $<
 
-ifdef XGETTEXT
 ifeq ($(word 1,$(GETTEXT_FILES)),+)
 po/$(CATALOG_NAME).pot: $(word 2, $(GETTEXT_FILES)) $(MAKEFILE_LIST)
+ifdef XGETTEXT
 	$(XGETTEXT) -D $(srcdir) -n $(addprefix -k, $(GETTEXT_TRIGGERS)) -f $<
 else
+	@echo "You don't have 'xgettext'."; exit 1
+endif
+else # GETTEXT_FILES
 po/$(CATALOG_NAME).pot: $(GETTEXT_FILES) $(MAKEFILE_LIST)
 # Change to srcdir explicitly, don't rely on $^.  That way we get
 # consistent #: file references in the po files.
+ifdef XGETTEXT
 	$(XGETTEXT) -D $(srcdir) -n $(addprefix -k, $(GETTEXT_TRIGGERS)) $(GETTEXT_FILES)
+else
+	@echo "You don't have 'xgettext'."; exit 1
 endif
+endif # GETTEXT_FILES
 	@$(MKDIR_P) $(dir $@)
 	sed -e '1,18 { s/SOME DESCRIPTIVE TITLE./LANGUAGE message translation file for $(CATALOG_NAME)/;s/PACKAGE/PostgreSQL/g;s/VERSION/$(MAJORVERSION)/g;s/YEAR/'`date +%Y`'/g; }' messages.po >$@
 	rm messages.po
-else # not XGETTEXT
-	@echo "You don't have 'xgettext'."; exit 1
-endif # not XGETTEXT
 
 
 # catalog name extentions must match behavior of PG_TEXTDOMAIN() in c.h