[Buildroot] [PATCH 1/1] package/igmpproxy: fix musl build

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Tue May 17 20:31:47 UTC 2016


Hello,

On Sun, 15 May 2016 16:41:11 -0500, Matt Weber wrote:

> diff --git a/package/igmpproxy/0002-Use-standard-unsigned-integer-types.patch b/package/igmpproxy/0002-Use-standard-unsigned-integer-types.patch
> new file mode 100644
> index 0000000..200bf23
> --- /dev/null
> +++ b/package/igmpproxy/0002-Use-standard-unsigned-integer-types.patch
> @@ -0,0 +1,213 @@
> +Fix musl build
> +
> +Downloaded from:
> +https://github.com/alpinelinux/aports/blob/master/main/igmpproxy/0001-Use-standard-unsigned-integer-types.patch
> +
> +Patch was reported upstream:
> +http://sourceforge.net/p/igmpproxy/bugs/22/
> +
> +Signed-off-by: Bernd Kuhls <bernd.kuhls at t-online.de>

This should be below the commit log of the original patch, and the "Fix
musl build" is not good. We don't care about "Fixing the musl build",
we want to make the source code better, which has the consequence of
fixing the musl build. So the original patch title "Use standard
unsigned integer types" is good.

> +From ed3dff27f360239910310be6706fd54572398992 Mon Sep 17 00:00:00 2001
> +From: Natanael Copa <ncopa at alpinelinux.org>
> +Date: Mon, 3 Mar 2014 12:11:26 +0000
> +Subject: [PATCH] Use standard unsigned integer types
> +
> +Use the stanard unsigned integer types instead of the non-standard
> +u_char, u_short and u_int.
> +
> +This fixes build with musl libc.
> +
> +(cherry picked from commit b914bcf882c6189be7a0ce6fceb34422372c3606)

From where is this commit b914bcf882c6189be7a0ce6fceb34422372c3606
coming? Upstream seems to be https://github.com/pali/igmpproxy, and I
don't see any commit identified as
b914bcf882c6189be7a0ce6fceb34422372c3606.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com



More information about the buildroot mailing list