Am Donnerstag, den 09.03.2017, 16:19 +0100 schrieb Michael Olbrich: > On Thu, Mar 09, 2017 at 10:37:06AM +0100, Markus Niebel wrote: > > From: Markus Niebel <markus.nie...@tq-group.com> > > > > the libhogweed part requires libgmp. Let's make it configurable. > > > > This seems wrong. My understanding is, that the --enable-mini-gmp options > exists to avoid the libgmp dependency by providing a minimalistic version > of libgmp. > If we want to use the 'real' libgmp (Are there any compelling reasons to do > so?) then the --enable-mini-gmp must be _disabled_ if libhogweed is > installed not the other way around. > > Also, the '--with-nettle-mini' of gnutls must be changed to 'without'. And > all other nettle users must be reviewed to check if they need libhogweed. >
I did't noticed that in gnutls. But I came up with this because I noticed a build break during a clean rebuild on our buildserver using ptxdist 2017.01: 8<-------------------------------------------------- configure: summary of build options: Version: nettle 3.3 Host type: arm-v7a-linux-gnueabihf ABI: standard Assembly files: arm Install prefix: /usr Library directory: /usr/lib Compiler: arm-v7a-linux-gnueabihf-gcc Static libraries: no Shared libraries: yes Public key crypto: yes Using mini-gmp: yes Documentation: no finished target nettle.prepare ... ---------------------- target: gnutls.prepare ---------------------- checking build system type... x86_64-host-linux-gnu checking host system type... arm-v7a-linux-gnueabihf checking for a BSD-compatible install... /usr/local/lib/ptxdist-2017.01.0/bin/install -c checking whether build environment is sane... yes checking for arm-v7a-linux-gnueabihf-strip... arm-v7a-linux-gnueabihf-strip checking for a thread-safe mkdir -p... /usr/local/lib/ptxdist-2017.01.0/bin/mkdir -p ... checking for gawk... gawk checking whether it is safe to define __EXTENSIONS__... yes checking whether _XOPEN_SOURCE should be defined... no checking for _LARGEFILE_SOURCE value needed for large files... no checking for special C compiler options needed for large files... no checking for _FILE_OFFSET_BITS value needed for large files... 64 checking dependency style of arm-v7a-linux-gnueabihf-gcc... gcc3 checking for arm-v7a-linux-gnueabihf-ar... arm-v7a-linux-gnueabihf-ar checking the archiver (arm-v7a-linux-gnueabihf-ar) interface... ar checking whether we are using the GNU C++ compiler... yes checking whether arm-v7a-linux-gnueabihf-g++ accepts -g... yes checking dependency style of arm-v7a-linux-gnueabihf-g++... gcc3 checking for bison... bison -y checking for a sed that does not truncate output... /usr/local/lib/ptxdist-2017.01.0/bin/sed checking for autogen... /bin/true configure: WARNING: *** *** autogen not found. Will not link against libopts. *** checking for inline... inline checking for ANSI C header files... (cached) yes checking for NETTLE... yes checking for HOGWEED... yes checking for __gmpz_cmp in -lgmp... no configure: error: *** *** gmp was not found. 8<------------------------------------------------------------------------ This says nettle incl. hogweed were found but a gmp feature was missing for some reason ... So something seems wrong. > Michael > Markus > > Signed-off-by: Markus Niebel <markus.nie...@tq-group.com> > > > > Conflicts: > > rules/nettle.in > > --- > > rules/nettle.in | 14 +++++++++++++- > > rules/nettle.make | 11 ++++++++++- > > 2 files changed, 23 insertions(+), 2 deletions(-) > > > > diff --git a/rules/nettle.in b/rules/nettle.in > > index fcacb96..f931994 100644 > > --- a/rules/nettle.in > > +++ b/rules/nettle.in > > @@ -1,8 +1,20 @@ > > ## SECTION=networking > > > > -config NETTLE > > +menuconfig NETTLE > > tristate > > prompt "nettle" > > + select LIBGMP if NETTLE_LIBHOGWEED > > help > > Nettle is a cryptographic library. > > > > +if NETTLE > > + > > +config NETTLE_LIBHOGWEED > > + bool > > + prompt "Build libhogweed" > > + help > > + The libhogweed library contains those functions of Nettle that > > + uses bignum operations, and depends on the GMP library > > + > > +endif > > + > > diff --git a/rules/nettle.make b/rules/nettle.make > > index 59df66e..23d6cb8 100644 > > --- a/rules/nettle.make > > +++ b/rules/nettle.make > > @@ -48,8 +48,15 @@ NETTLE_CONF_OPT := \ > > --disable-documentation \ > > --disable-fat \ > > --$(call ptx/endis,PTXCONF_ARCH_ARM_NEON)-arm-neon \ > > - --disable-x86-aesni \ > > + --disable-x86-aesni > > + > > +ifdef PTXCONF_NETTLE_LIBHOGWEED > > +NETTLE_CONF_OPT += \ > > --enable-mini-gmp > > +else > > +NETTLE_CONF_OPT += \ > > + --disable-mini-gmp > > +endif > > > > # > > ---------------------------------------------------------------------------- > > # Target-Install > > @@ -65,7 +72,9 @@ $(STATEDIR)/nettle.targetinstall: > > @$(call install_fixup, nettle,DESCRIPTION,missing) > > > > @$(call install_lib, nettle, 0, 0, 0644, libnettle) > > +ifdef PTXCONF_NETTLE_LIBHOGWEED > > @$(call install_lib, nettle, 0, 0, 0644, libhogweed) > > +endif > > @$(call install_copy, nettle, 0, 0, 0755, -, /usr/bin/nettle-hash) > > @$(call install_copy, nettle, 0, 0, 0755, -, /usr/bin/sexp-conv) > > @$(call install_copy, nettle, 0, 0, 0755, -, > > /usr/bin/nettle-lfib-stream) > > -- > > 1.9.1 > > > > > > _______________________________________________ > > ptxdist mailing list > > ptxdist@pengutronix.de > _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de