Attachment 8571 Details for Bug 7825 – patch that resolves the stdout/stderr mess up

[patch] patch that resolves the stdout/stderr mess up

0001-build-autoconf-fix-check-for-GNU-ld-version.patch (text/plain), 1.51 KB, created by Björn Jacke on 2013-02-19 14:36:33 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Björn Jacke

Created: 2013-02-19 14:36:33 UTC

Size: 1.51 KB

patch

obsolete

>From f4e00a2a31a8ef2e4ad855825fecb07e6a63888e Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Bj=C3=B6rn=20Jacke?= <bj@sernet.de>
>Date: Tue, 19 Feb 2013 15:30:34 +0100
>Subject: [PATCH] build/autoconf: fix check for GNU ld version
>
>we need to look for the version once in the stdout and once in the stderr
>output. Some version of ld output to stdout, some output to stderr. redirecting
>stderr to stdout messes the output up in our case, that's why we have to do two
>runs. See also bug #7825.
>---
> source3/configure.in | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
>diff --git a/source3/configure.in b/source3/configure.in
>index 1b24ad6..594f4b9 100644
>--- a/source3/configure.in
>+++ b/source3/configure.in
>@@ -227,7 +227,11 @@ dnl Certain versions of GNU ld the default is not to have the
> dnl --allow-shlib-undefined flag defined.  This causes a stackload of
> dnl warnings when building modules.
> if test "$ac_cv_prog_gnu_ld" = "yes"; then
>-	ac_cv_gnu_ld_version=`$CC -Wl,-v /dev/null 2>&1 < /dev/null | grep "GNU ld"`
>+	ac_cv_gnu_ld_version=`$CC -Wl,-v /dev/null 2> /dev/null < /dev/null | grep "GNU ld"`
>+	# we need to make a 2nd (separate!) check on the output of stderr, see bug #7825:
>+	if test -z ${ac_cv_gnu_ld_version} ; then
>+	  ac_cv_gnu_ld_version=`$CC -Wl,-v /dev/null 2>&1 > /dev/null < /dev/null | grep "GNU ld"`
>+	fi
> 	AC_MSG_CHECKING(GNU ld release date)
> 	changequote(,)dnl
> 	ac_cv_gnu_ld_date=`echo $ac_cv_gnu_ld_version | sed -n 's,^.*$[2-9][0-9][0-9][0-9]$[-]*$[01][0-9]$[-]*$[0-3][0-9]$.*$,\1\2\3,p'`
>-- 
>1.8.0
>

Actions: View

Attachments on bug 7825: 8571 | 8572 | 8577 | 8578