httping: fix build without host-gettext
PATCH 1/1 httping: fix build without host-gettext
Since commit 654de6512554c (httping: use the new gettext logic), host-gettext is not a dependency of httping. This breaks the build because the httping Makefile calls msgfmt unconditionally. Add a patch that allows build without translation files generation.
Fixes:
http://autobuild.buildroot.net/results/73e/73e624775617d1b95da7d4ef666612af5bceef61/
http://autobuild.buildroot.net/results/2bf/2bff43d7b749b4daf32868bddbae94b510949bf8/
http://autobuild.buildroot.net/results/8a6/8a69659afa21f467ca5de7915363cebf0fc7aff2/
--- ...0002-Makefile-allow-build-without-gettext.patch | 48 ++++++++++++++++++++++ package/httping/httping.mk | 3 +- 2 files changed, 50 insertions(+), 1 deletion(-) create mode 100644 package/httping/0002-Makefile-allow-build-without-gettext.patch
package/httping/0002-Makefile-allow-build-without-gettext.patch new file +48 -0
@@ -0,0 +1,48 @@ +From fe7d6c5a0e5dfe129f228498037393d23d6ae890 Mon Sep 17 00:00:00 2001 +From: Baruch Siach <[email protected]> +Date: Tue, 18 Jul 2017 19:09:03 +0300 +Subject: [PATCH] Makefile: allow build without gettext + +The msgfmt command is part of the gettext package, and is used to generate +binary translation files. When gettext is not installed, build fails. +Translation files are not always needed on size constrained embedded targets. +Add an option to disable translation files generation using the NO_GETTEXT +variable. + +Signed-off-by: Baruch Siach <[email protected]> +--- +Upstream status: https://github.com/flok99/httping/pull/36 + + Makefile | 4 ++++ + 1 file changed, 4 insertions(+) + +diff --git a/Makefile b/Makefile +index 46127f4cdde1..160cc1794ec8 100644 +--- a/Makefile ++++ b/Makefile +@@ -55,7 +55,9 @@ MKDIR=/bin/mkdir + ARCHIVE=/bin/tar cf - + COMPRESS=/bin/gzip -9 + ++ifneq ($(NO_GETTEXT),yes) + TRANSLATIONS=nl.mo ru.mo ++endif + + OBJS=gen.o http.o io.o error.o utils.o main.o tcp.o res.o socks5.o kalman.o cookies.o help.o colors.o + +@@ -118,10 +120,12 @@ install: $(TARGET) $(TRANSLATIONS) + ifneq ($(DEBUG),yes) + $(STRIP) $(DESTDIR)/$(BINDIR)/$(TARGET) + endif ++ifneq ($(NO_GETTEXT),yes) + mkdir -p $(DESTDIR)/$(PREFIX)/share/locale/nl/LC_MESSAGES + cp nl.mo $(DESTDIR)/$(PREFIX)/share/locale/nl/LC_MESSAGES/httping.mo + mkdir -p $(DESTDIR)/$(PREFIX)/share/locale/ru/LC_MESSAGES + cp ru.mo $(DESTDIR)/$(PREFIX)/share/locale/ru/LC_MESSAGES/httping.mo ++endif + + + makefile.inc: +-- +2.13.2 +
package/httping/httping.mk +2 -1
@@ -22,7 +22,8 @@ HTTPING_MAKE_OPTS = $(TARGET_CONFIGURE_OPTS) \ FW=$(if $(BR2_PACKAGE_FFTW),yes,no) \ NC=$(if $(BR2_PACKAGE_NCURSES_WCHAR),yes,no) \ SSL=$(if $(BR2_PACKAGE_OPENSSL),yes,no) \ - TFO=$(if $(BR2_PACKAGE_HTTPING_TFO),yes,no) + TFO=$(if $(BR2_PACKAGE_HTTPING_TFO),yes,no) \ + NO_GETTEXT=$(if $(BR2_SYSTEM_ENABLE_NLS),no,yes) define HTTPING_BUILD_CMDS $(HTTPING_MAKE_OPTS) LDFLAGS="$(HTTPING_LDFLAGS)" \
Signature · 1 line
2.13.2
PATCH 1/1 httping: fix build without host-gettext
Hello,
Quoted 10 lines · Baruch Siach
Since commit
654de6512554c(httping: use the new gettext logic), host-gettext is not a dependency of httping. This breaks the build because the httpingMakefilecalls msgfmt unconditionally. Add a patch that allows build without translation files generation.Fixes:
http://autobuild.buildroot.net/results/73e/73e624775617d1b95da7d4ef666612af5bceef61/
http://autobuild.buildroot.net/results/2bf/2bff43d7b749b4daf32868bddbae94b510949bf8/
http://autobuild.buildroot.net/results/8a6/8a69659afa21f467ca5de7915363cebf0fc7aff2/
Quoted 8 lines
Cc Thomas Petazzoni <[email protected]>Cc Gilles Talis <[email protected]>Signed-off-by Baruch Siach <[email protected]>--- ...0002-Makefile-allow-build-without-gettext.patch | 48 ++++++++++++++++++++++ package/httping/httping.mk | 3 +- 2 files changed, 50 insertions(+), 1 deletion(-) create mode 100644 package/httping/0002-Makefile-allow-build-without-gettext.patch
Applied to master, thanks. To be honest, I wasn't sure if adding a patch was really worth it, compared to re-adding the host-gettext dependency. But since you've done the effort, and submitted the patch upstream, I applied. We'll say what upstream says :)
Thanks!
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com