[Buildroot] [PATCH v2 1/2] libfcgi (0003-link-against-math.patch): Do actually link when needed.
Arnout Vandecappelle
arnout at mind.be
Thu May 28 22:00:39 UTC 2015
On 05/28/15 17:36, Thomas Claveirole wrote:
> libfcgi uses frexp() and thus should link with libm, but fails to do
> so. Thus link errors with programs that use libfcgi (the bug does
> appear only with uClibc, because the GNU libc provides frexp()
> directly in libc.so). Modify 0003-link-against-math.patch so libfcgi
> links against math, instead of programs that need it.
>
> Signed-off-by: Thomas Claveirole <thomas.claveirole at green-communications.fr>
Looks good, but the patch should still have your Signed-off-by to the patch:
> ---
> package/libfcgi/0003-link-against-math.patch | 36 +++++++++-------------------
> 1 file changed, 11 insertions(+), 25 deletions(-)
>
> diff --git a/package/libfcgi/0003-link-against-math.patch b/package/libfcgi/0003-link-against-math.patch
> index 4c87ea6..ccf648a 100644
> --- a/package/libfcgi/0003-link-against-math.patch
> +++ b/package/libfcgi/0003-link-against-math.patch
> @@ -1,28 +1,14 @@
> -Link against math libraries when needed
> +Link libfcgi against the math library.
>
> -Signed-off-by: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
Here.
With that:
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
Regards,
Arnout
> -
> -Index: b/cgi-fcgi/Makefile.am
> -===================================================================
> ---- a/cgi-fcgi/Makefile.am
> -+++ b/cgi-fcgi/Makefile.am
> -@@ -11,5 +11,5 @@
> - LIBDIR = ../libfcgi
> - LIBFCGI = $(LIBDIR)/libfcgi.la
> -
> --LDADD = $(LIBFCGI)
> -+LDADD = $(LIBFCGI) -lm
> - cgi_fcgi_SOURCES = $(INCLUDE_FILES) cgi-fcgi.c
> -Index: b/examples/Makefile.am
> +Index: b/libfcgi/Makefile.am
> ===================================================================
> ---- a/examples/Makefile.am
> -+++ b/examples/Makefile.am
> -@@ -21,7 +21,7 @@
> - LIBDIR = ../libfcgi
> - LIBFCGI = $(LIBDIR)/libfcgi.la
> -
> --LDADD = $(LIBFCGI)
> -+LDADD = $(LIBFCGI) -lm
> +--- a/libfcgi/Makefile.am
> ++++ b/libfcgi/Makefile.am
> +@@ -18,6 +18,7 @@
> + os_ at SYSTEM@.c
> + libfcgi_la_CC = @PTHREAD_CC@
> + libfcgi_la_CFLAGS = @PTHREAD_CFLAGS@
> ++libfcgi_la_LIBADD = -lm
>
> - echo_SOURCES = $(INCLUDE_FILES) echo.c
> - echo_x_SOURCES = $(INCLUDE_FILES) echo-x.c
> + libfcgi___la_SOURCES = $(INCLUDE_FILES) \
> + $(INCLUDEDIR)/fcgio.h \
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot
mailing list