netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* small iputils build patch
@ 2013-01-06 16:42 Arjan van de Ven
  2013-01-15 18:24 ` YOSHIFUJI Hideaki
  0 siblings, 1 reply; 2+ messages in thread
From: Arjan van de Ven @ 2013-01-06 16:42 UTC (permalink / raw)
  To: netdev

[-- Attachment #1: Type: text/plain, Size: 404 bytes --]

The ping6 command can use either openssl or gnutls...
and the Makefile has a bunch of setup for defining which of the two to use.

Unfortunately, the final -D define on the commandline to enable gnutls
inside the ping6.c file didn't actually make it onto the gcc
commandline.
This patch adds the $(DEF_CRYPTO) Makefile variable to fix this gap.

Signed-off-by: Arjan van de Ven <arjanvandeven@gmail.com>

[-- Attachment #2: build.patch --]
[-- Type: application/octet-stream, Size: 962 bytes --]

The ping6 command can use either openssl or gnutls...
and the Makefile has a bunch of setup for defining which of the two to use.

Unfortunately, the final -D define on the commandline to enable gnutls inside the ping6.c file didn't actually make it onto the gcc commandline.
This patch adds the $(DEF_CRYPTO) Makefile variable to fix this gap.

Signed-off-by: Arjan van de Ven <arjanvandeven@gmail.com>

--- iputils-s20121221/Makefile~	2012-12-21 06:01:07.000000000 -0800
+++ iputils-s20121221/Makefile	2013-01-06 08:27:56.014891846 -0800
@@ -149,7 +149,7 @@
 DEF_ping_common = $(DEF_CAP) $(DEF_IDN)
 DEF_ping  = $(DEF_CAP) $(DEF_IDN) $(DEF_WITHOUT_IFADDRS)
 LIB_ping  = $(LIB_CAP) $(LIB_IDN)
-DEF_ping6 = $(DEF_CAP) $(DEF_IDN) $(DEF_WITHOUT_IFADDRS) $(DEF_ENABLE_PING6_RTHDR)
+DEF_ping6 = $(DEF_CAP) $(DEF_IDN) $(DEF_WITHOUT_IFADDRS) $(DEF_ENABLE_PING6_RTHDR) $(DEF_CRYPTO)
 LIB_ping6 = $(LIB_CAP) $(LIB_IDN) $(LIB_RESOLV) $(LIB_CRYPTO)
 
 ping: ping_common.o

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: small iputils build patch
  2013-01-06 16:42 small iputils build patch Arjan van de Ven
@ 2013-01-15 18:24 ` YOSHIFUJI Hideaki
  0 siblings, 0 replies; 2+ messages in thread
From: YOSHIFUJI Hideaki @ 2013-01-15 18:24 UTC (permalink / raw)
  To: Arjan van de Ven; +Cc: netdev

Arjan van de Ven wrote:
> The ping6 command can use either openssl or gnutls...
> and the Makefile has a bunch of setup for defining which of the two to use.
> 
> Unfortunately, the final -D define on the commandline to enable gnutls
> inside the ping6.c file didn't actually make it onto the gcc
> commandline.
> This patch adds the $(DEF_CRYPTO) Makefile variable to fix this gap.
> 
> Signed-off-by: Arjan van de Ven <arjanvandeven@gmail.com>
> 

Applied, thank you.

--yoshfuji

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-01-15 18:24 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-06 16:42 small iputils build patch Arjan van de Ven
2013-01-15 18:24 ` YOSHIFUJI Hideaki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).