diff mbox series

[1/1] erlang: always use Buildroot's zlib

Message ID 20181012141149.72489-1-fhunleth@troodon-software.com
State Accepted
Commit ec5378038f80f444ce7f5106283c7cccb8faae16
Headers show
Series [1/1] erlang: always use Buildroot's zlib | expand

Commit Message

Frank Hunleth Oct. 12, 2018, 2:11 p.m. UTC
While Erlang includes a version of zlib, it's intended for Windows and
there's an expectation that non-Windows platforms provide it. It's also
not as regularly updated as the one in Buildroot. This change makes
Erlang always use a Buildroot-provided zlib.

Fixes this compile error:

 CC	/home/buildroot/autobuild/run/instance-0/output/build/erlang-21.0/erts/emulator/zlib/obj/x86_64-buildroot-linux-musl/opt/adler32.o
In file included from zlib/adler32.c:11:0:
zlib/zutil.h:172:39: error: "_LFS64_LARGEFILE" is not defined [-Werror=undef]
     (!defined(_LARGEFILE64_SOURCE) || _LFS64_LARGEFILE-0 == 0)
                                       ^~~~~~~~~~~~~~~~

See http://autobuild.buildroot.net/results/fc633f80c7c36a90e641487f5a888fbb767c2a54/.

Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>
---
 package/erlang/Config.in | 1 +
 package/erlang/erlang.mk | 3 +--
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Peter Korsgaard Oct. 14, 2018, 8:48 p.m. UTC | #1
>>>>> "Frank" == Frank Hunleth <fhunleth@troodon-software.com> writes:

 > While Erlang includes a version of zlib, it's intended for Windows and
 > there's an expectation that non-Windows platforms provide it. It's also
 > not as regularly updated as the one in Buildroot. This change makes
 > Erlang always use a Buildroot-provided zlib.

 > Fixes this compile error:

 >  CC	/home/buildroot/autobuild/run/instance-0/output/build/erlang-21.0/erts/emulator/zlib/obj/x86_64-buildroot-linux-musl/opt/adler32.o
 > In file included from zlib/adler32.c:11:0:
 > zlib/zutil.h:172:39: error: "_LFS64_LARGEFILE" is not defined [-Werror=undef]
 >      (!defined(_LARGEFILE64_SOURCE) || _LFS64_LARGEFILE-0 == 0)
 >                                        ^~~~~~~~~~~~~~~~

 > See http://autobuild.buildroot.net/results/fc633f80c7c36a90e641487f5a888fbb767c2a54/.

 > Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>

Committed, thanks.
Peter Korsgaard Oct. 24, 2018, 11:58 a.m. UTC | #2
>>>>> "Frank" == Frank Hunleth <fhunleth@troodon-software.com> writes:

 > While Erlang includes a version of zlib, it's intended for Windows and
 > there's an expectation that non-Windows platforms provide it. It's also
 > not as regularly updated as the one in Buildroot. This change makes
 > Erlang always use a Buildroot-provided zlib.

 > Fixes this compile error:

 >  CC	/home/buildroot/autobuild/run/instance-0/output/build/erlang-21.0/erts/emulator/zlib/obj/x86_64-buildroot-linux-musl/opt/adler32.o
 > In file included from zlib/adler32.c:11:0:
 > zlib/zutil.h:172:39: error: "_LFS64_LARGEFILE" is not defined [-Werror=undef]
 >      (!defined(_LARGEFILE64_SOURCE) || _LFS64_LARGEFILE-0 == 0)
 >                                        ^~~~~~~~~~~~~~~~

 > See http://autobuild.buildroot.net/results/fc633f80c7c36a90e641487f5a888fbb767c2a54/.

 > Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>

Committed to 2018.02.x and 2018.08.x, thanks.
diff mbox series

Patch

diff --git a/package/erlang/Config.in b/package/erlang/Config.in
index 75ba17086a..ab87eab6ff 100644
--- a/package/erlang/Config.in
+++ b/package/erlang/Config.in
@@ -23,6 +23,7 @@  config BR2_PACKAGE_ERLANG
 	depends on !BR2_STATIC_LIBS
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_PACKAGE_ERLANG_ARCH_SUPPORTS
+	select BR2_PACKAGE_ZLIB
 	help
 	  Erlang is a programming language used to build massively
 	  scalable soft real-time systems with requirements on high
diff --git a/package/erlang/erlang.mk b/package/erlang/erlang.mk
index 2ac903ffea..ca0aa0b770 100644
--- a/package/erlang/erlang.mk
+++ b/package/erlang/erlang.mk
@@ -64,10 +64,9 @@  else
 ERLANG_CONF_OPTS += --without-odbc
 endif
 
-ifeq ($(BR2_PACKAGE_ZLIB),y)
+# Always use Buildroot's zlib
 ERLANG_CONF_OPTS += --enable-shared-zlib
 ERLANG_DEPENDENCIES += zlib
-endif
 
 # Remove source, example, gs and wx files from staging and target.
 ERLANG_REMOVE_PACKAGES = gs wx