[Buildroot] [PATCH v2] wireshark: new package

[Thomas Petazzoni <thomas.petazzoni@free-electrons.com>]

On Wed, 24 Oct 2012 16:13:33 +0200, Baruch Siach wrote:
> diff --git a/package/wireshark/wireshark-dont-include-prefix.patch b/package/wireshark/wireshark-dont-include-prefix.patch
> new file mode 100644
> index 0000000..6f3c946
> --- /dev/null
> +++ b/package/wireshark/wireshark-dont-include-prefix.patch
> @@ -0,0 +1,18 @@
> +configure.in: don't add the build host's /usr/include and /usr/lib to the 
> +search path of include files and libraries. This is not what you want when 
> +cross compiling.
> +
> +Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> +---
> +diff -Nuar wireshark-1.8.3.orig/configure.in wireshark-1.8.3/configure.in
> +--- wireshark-1.8.3.orig/configure.in	2012-08-15 23:33:32.000000000 +0300
> ++++ wireshark-1.8.3/configure.in	2012-10-24 09:54:15.259853331 +0200
> +@@ -639,7 +639,7 @@
> + # If using $prefix we add "$prefix/include" to the include search path
> + # and "$prefix/lib" to the library search path.
> + #
> +-if test "x$prefix" != "x" ; then
> ++if false ; then
> + 	AC_MSG_CHECKING(whether to use $prefix for headers and libraries)
> + 	if test -d $prefix/include ; then
> + 		AC_MSG_RESULT(yes)

This works, but cannot be submitted upstream. Do you think you can come
up with a patch suitable for upstream inclusion?

Note that the OE patch is not better than your proposal:
http://cgit.openembedded.org/openembedded/tree/recipes/wireshark/files/fix-configure.patch.
It is also not suitable for upstream.

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com