[Buildroot] [git commit] package/pkg-autotools: drop support for FOO_GETTEXTIZE

Arnout Vandecappelle arnout at mind.be
Sat Mar 25 20:55:00 UTC 2023


commit: https://git.buildroot.net/buildroot/commit/?id=e32123b866442a1ccf65627b98147c91128b5912
branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master

Remove all support for FOO_GETTEXTIZE and alert users to FOO_AUTOPOINT
as the recommended solution in its stead. We can use the standard
check-deprecated-variable for this, because from a user perspective
FOO_AUTOPOINT is pretty-much a drop-in replacement.

The warnings about FOO_GETTEXTIZE_OPTS are no longer relevant, because
they will only make sense if FOO_GETTEXTIZE was already set.

Signed-off-by: Vincent Fazio <vfazio at gmail.com>
[Arnout:
 - use check-deprecated-variable;
 - remove FOO_GETTEXTIZE_OPTS warnings;
 - remove definition of GETTEXTIZE]
Signed-off-by: Arnout Vandecappelle <arnout at mind.be>
---
 package/pkg-autotools.mk | 23 +----------------------
 package/pkg-generic.mk   |  9 +--------
 2 files changed, 2 insertions(+), 30 deletions(-)

diff --git a/package/pkg-autotools.mk b/package/pkg-autotools.mk
index 48485567d7..f17186dfe4 100644
--- a/package/pkg-autotools.mk
+++ b/package/pkg-autotools.mk
@@ -87,14 +87,6 @@ define CONFIGURE_FIX_POWERPC64_HOOK
 	support/scripts/fix-configure-powerpc64.sh $($(PKG)_DIR)
 endef
 
-#
-# Hook to gettextize the package if needed
-#
-define GETTEXTIZE_HOOK
-	@$(call MESSAGE,"Gettextizing")
-	$(Q)cd $($(PKG)_SRCDIR) && $(GETTEXTIZE)
-endef
-
 #
 # Hook to autoreconf the package if needed
 #
@@ -144,14 +136,6 @@ ifndef $(2)_AUTORECONF
  endif
 endif
 
-ifndef $(2)_GETTEXTIZE
- ifdef $(3)_GETTEXTIZE
-  $(2)_GETTEXTIZE = $$($(3)_GETTEXTIZE)
- else
-  $(2)_GETTEXTIZE ?= NO
- endif
-endif
-
 ifndef $(2)_AUTOPOINT
  ifdef $(3)_AUTOPOINT
   $(2)_AUTOPOINT = $$($(3)_AUTOPOINT)
@@ -245,13 +229,8 @@ $(2)_POST_PATCH_HOOKS += UPDATE_CONFIG_HOOK
 
 ifeq ($$($(2)_AUTORECONF),YES)
 
-# This has to come before autoreconf
-ifeq ($$($(2)_GETTEXTIZE),YES)
-$(2)_PRE_CONFIGURE_HOOKS += GETTEXTIZE_HOOK
-$(2)_DEPENDENCIES += host-gettext
-$(2)_AUTORECONF_ENV += AUTOPOINT=/bin/true
 # autopoint is provided by gettext
-else ifeq ($$($(2)_AUTOPOINT),YES)
+ifeq ($$($(2)_AUTOPOINT),YES)
 $(2)_DEPENDENCIES += host-gettext
 $(2)_AUTORECONF_ENV += AUTOPOINT=$$(HOST_DIR)/bin/autopoint
 else
diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
index fc873c045a..5d1c1da128 100644
--- a/package/pkg-generic.mk
+++ b/package/pkg-generic.mk
@@ -1207,14 +1207,7 @@ $(eval $(call check-deprecated-variable,$(2)_AUTORECONF_OPT,$(2)_AUTORECONF_OPTS
 $(eval $(call check-deprecated-variable,$(2)_CONF_OPT,$(2)_CONF_OPTS))
 $(eval $(call check-deprecated-variable,$(2)_BUILD_OPT,$(2)_BUILD_OPTS))
 $(eval $(call check-deprecated-variable,$(2)_KCONFIG_OPT,$(2)_KCONFIG_OPTS))
-
-# Check for deprecated variables that have no replacement
-ifneq ($$($(2)_GETTEXTIZE_OPT),)
-$$(error "$(2)_GETTEXTIZE_OPT is no longer supported. Please fix your .mk file.")
-endif
-ifneq ($$($(2)_GETTEXTIZE_OPTS),)
-$$(error "$(2)_GETTEXTIZE_OPTS is no longer supported. Please fix your .mk file.")
-endif
+$(eval $(call check-deprecated-variable,$(2)_GETTEXTIZE,$(2)_AUTOPOINT))
 
 PACKAGES += $(1)
 



More information about the buildroot mailing list