[30265] in CVS-changelog-for-Kerberos-V5

home help back first fref pref prev next nref lref last post

krb5 commit: Use PTHREAD_CFLAGS when testing for getpwnam_r()

daemon@ATHENA.MIT.EDU (Greg Hudson)
Sun Aug 5 09:47:25 2018

Date: Sun, 5 Aug 2018 09:47:12 -0400
From: Greg Hudson <ghudson@mit.edu>
Message-Id: <201808051347.w75DlCKL009141@drugstore.mit.edu>
To: cvs-krb5@mit.edu
Reply-To: krbdev@mit.edu
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cvs-krb5-bounces@mit.edu

https://github.com/krb5/krb5/commit/013de5c54eafebf5aa3fae82821c21ca2072040d
commit 013de5c54eafebf5aa3fae82821c21ca2072040d
Author: sashan <anedvedicky@gmail.com>
Date:   Mon Jul 23 12:56:15 2018 +0000

    Use PTHREAD_CFLAGS when testing for getpwnam_r()
    
    On Solaris, AX_PTHREAD can include -D_POSIX_PTHREAD_SEMANTICS in
    PTHREAD_CFLAGS, which affects which variants of getpwnam_r() and
    getpwuid_r() are declared by system headers.  In configure.in, use
    PTHREAD_CFLAGS when testing for those functions to make sure that the
    tests see the same variants as the build will.
    
    [ghudson@mit.edu: rewrote commit message and comment]
    
    ticket: 8723 (new)

 src/configure.in |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/src/configure.in b/src/configure.in
index bafacf8..cfa26eb 100644
--- a/src/configure.in
+++ b/src/configure.in
@@ -576,6 +576,10 @@ fi
 ])
 
 
+# PTHREAD_CFLAGS changes which variant of these functions is declared
+# on Solaris 11, so use it for these tests.
+old_CFLAGS=$CFLAGS
+CFLAGS="$CFLAGS $PTHREAD_CFLAGS"
 AC_CHECK_FUNC(getpwnam_r,ac_cv_func_getpwnam_r=yes,ac_cv_func_getpwnam_r=no)
 AC_CHECK_FUNC(getpwuid_r,ac_cv_func_getpwuid_r=yes,ac_cv_func_getpwuid_r=no)
 if test "$ac_cv_func_getpwnam_r" = yes; then
@@ -625,6 +629,7 @@ if test "$ac_cv_func_getpwnam_r" = yes; then
     fi
   fi
 fi
+CFLAGS=$old_CFLAGS
 
 if test "$ac_cv_func_getpwnam_r" = no && test "$ac_cv_func_getpwuid_r" = yes; then
   # Actually, we could do this check, and the corresponding checks
_______________________________________________
cvs-krb5 mailing list
cvs-krb5@mit.edu
https://mailman.mit.edu/mailman/listinfo/cvs-krb5

home help back first fref pref prev next nref lref last post