threads / patch / 31932

patchconfigure: fix some output message

Subject: [PATCH] configure: fix some output message

## tl;dr

2 messages between Oct 24, 2012 and Oct 25, 2012. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Stefano Lattarini· Oct 24, 2012, 15:34 UTC · lore

Before this change, output from ./configure could contain botched wording like this:

    checking Checking for POSIX Threads with '-pthread'... yes
instead of the intended:
    checking for POSIX Threads with '-pthread'... yes
Signed-off-by: Stefano Lattarini <stefano.lattarini@gmail.com>
---
 configure.ac | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
Show changes to configure.ac +2 −2
diff --git a/configure.ac b/configure.ac
index c85888c..ad215cc 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1024,7 +1024,7 @@ elif test -z "$PTHREAD_CFLAGS"; then
   for opt in -mt -pthread -lpthread; do
      old_CFLAGS="$CFLAGS"
      CFLAGS="$opt $CFLAGS"
-     AC_MSG_CHECKING([Checking for POSIX Threads with '$opt'])
+     AC_MSG_CHECKING([for POSIX Threads with '$opt'])
      AC_LINK_IFELSE([PTHREADTEST_SRC],
 	[AC_MSG_RESULT([yes])
 		NO_PTHREADS=
@@ -1044,7 +1044,7 @@ elif test -z "$PTHREAD_CFLAGS"; then
 else
   old_CFLAGS="$CFLAGS"
   CFLAGS="$PTHREAD_CFLAGS $CFLAGS"
-  AC_MSG_CHECKING([Checking for POSIX Threads with '$PTHREAD_CFLAGS'])
+  AC_MSG_CHECKING([for POSIX Threads with '$PTHREAD_CFLAGS'])
   AC_LINK_IFELSE([PTHREADTEST_SRC],
 	[AC_MSG_RESULT([yes])
 		NO_PTHREADS=
-- 
1.8.0.rc2.11.gd25c58c
Jeff King· Oct 25, 2012, 09:11 UTC · re: Stefano Lattarini · lore

Re: [PATCH] configure: fix some output message

On Wed, Oct 24, 2012 at 05:34:27PM +0200, Stefano Lattarini wrote:
Show 10 quoted lines
> Before this change, output from ./configure could contain
> botched wording like this:
> 
>     checking Checking for POSIX Threads with '-pthread'... yes
> 
> instead of the intended:
> 
>     checking for POSIX Threads with '-pthread'... yes
> 
> Signed-off-by: Stefano Lattarini <stefano.lattarini@gmail.com>
Thanks, looks obviously correct.
-Peff

← back to recent threads