Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Baruch Siach <baruch@tkos.co.il>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2] libpcap: bump to version 1.7.2
Date: Wed, 11 Mar 2015 16:34:50 +0200	[thread overview]
Message-ID: <20150311143450.GG2177@tarshish> (raw)
In-Reply-To: <1426081428-32041-1-git-send-email-gustavo@zacarias.com.ar>

Hi Gustavo,

On Wed, Mar 11, 2015 at 10:43:47AM -0300, Gustavo Zacarias wrote:
> Drop upstream patch (uclinux), adapt libnl patch to new version.
> And add hash file.
> 
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> ---
>  .../libpcap/0001-remove-libnl-include-path.patch   | 17 ++++++----
>  .../0002-configure.in-uclinux-is-also-linux.patch  | 39 ----------------------
>  package/libpcap/libpcap.hash                       |  2 ++
>  package/libpcap/libpcap.mk                         |  2 +-
>  4 files changed, 13 insertions(+), 47 deletions(-)
>  delete mode 100644 package/libpcap/0002-configure.in-uclinux-is-also-linux.patch
>  create mode 100644 package/libpcap/libpcap.hash
> 
> diff --git a/package/libpcap/0001-remove-libnl-include-path.patch b/package/libpcap/0001-remove-libnl-include-path.patch
> index 7b7106b..a4039ef 100644
> --- a/package/libpcap/0001-remove-libnl-include-path.patch
> +++ b/package/libpcap/0001-remove-libnl-include-path.patch
> @@ -1,16 +1,19 @@
>  Remove hardcoded path to libnl3 include directory
>  
>  Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> +[Gustavo: update for 1.7.2]
>  
> -Index: b/configure.in
> -===================================================================
> ---- a/configure.in
> -+++ b/configure.in
> -@@ -445,7 +445,6 @@
> +diff -Nura libpcap-1.7.2.orig/configure.in libpcap-1.7.2/configure.in
> +--- libpcap-1.7.2.orig/configure.in	2015-03-11 10:02:44.416189921 -0300
> ++++ libpcap-1.7.2/configure.in	2015-03-11 10:03:03.967633673 -0300
> +@@ -466,9 +466,8 @@
>   			AC_DEFINE(HAVE_LIBNL_3_x,1,[if libnl exists and is version 3.x])
>   			AC_DEFINE(HAVE_LIBNL_NLE,1,[libnl has NLE_FAILURE])
>   			AC_DEFINE(HAVE_LIBNL_SOCKETS,1,[libnl has new-style socket api])
> --			V_INCLS="$V_INCLS -I/usr/include/libnl3"
> +-			V_INCLS="$V_INCLS ${incdir}"
>   			have_any_nl="yes"
> - 		])
> +-		],[], ${incdir} ${libnldir} -lnl-genl-3 -lnl-3 )
> ++		],[], ${libnldir} -lnl-genl-3 -lnl-3 )

That's disappointing. Upstream actually merged this patch (commit 
157330dcac5), but another commit (ee3296275f) broke it in another way. Is it 
possible to pass --with-libnl=/dev/null so that ${incdir} and ${libnldir} 
point to a non-existing place? That should allow removing this patch.

Thanks,
baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

  parent reply	other threads:[~2015-03-11 14:34 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-11 13:43 [Buildroot] [PATCH 1/2] libpcap: bump to version 1.7.2 Gustavo Zacarias
2015-03-11 13:43 ` [Buildroot] [PATCH 2/2] tcpdump: security bump to version 4.7.3 Gustavo Zacarias
2015-03-11 16:09   ` Peter Korsgaard
2015-03-11 14:34 ` Baruch Siach [this message]
2015-03-11 15:43   ` [Buildroot] [PATCH 1/2] libpcap: bump to version 1.7.2 Gustavo Zacarias
2015-03-11 16:34     ` Gustavo Zacarias
2015-03-11 16:39       ` Peter Korsgaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150311143450.GG2177@tarshish \
    --to=baruch@tkos.co.il \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox