Jim Seymour wrote:
> Bruce Momjian <[EMAIL PROTECTED]> wrote:
> > 
> [snip]
> > 
> > OK, patch attached and applied.  It adds configure tests for the 5-arg
> > version of getpwuid_r() and properly falls back to the Posix draft
> > version you have on Solaris.  Seems Solaris 9 also still has the draft
> > version.
> [snip]
> 
> Well, yes and no.  If you define _POSIX_PTHREAD_SEMANTICS, you get the
> 5-arg version.  It looks like this has been the case at least back to
> Solaris 2.5.1.
> 
> I didn't really expect anything prior to 2.5.1 to be an issue, so I
> didn't bother looking into fixes for anything beyond that.

Oh, very interesting.  CVS HEAD has in template/solaris:
        
        # tools/thread/thread_test must be run
        if test "$GCC" = yes
        then    THREAD_CPPFLAGS="-D_POSIX_PTHREAD_SEMANTICS"
                THREAD_LIBS="-pthread"
        else    THREAD_CPPFLAGS="-mt -D_POSIX_PTHREAD_SEMANTICS"
                THREAD_LIBS="-lpthread"
        fi

I added the "-D_POSIX_PTHREAD_SEMANTICS"  flags into post-7.4.2, so it
seems this platform would work even without checking for the 4-arg
getpwuid_r version.  However, I noticed that 'man getpwuid_r' only
mentions the 4-arg version.

I will leave the 4-arg check in. The original author thought it might be
needed, and the Solaris manual mentions it, so odds are some other
platforms will hit it too, and perhaps not have the 5-arg version.

Thanks for the research.  I will add a mention in the solaris template
file.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  [EMAIL PROTECTED]               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

---------------------------(end of broadcast)---------------------------
TIP 5: Have you checked our extensive FAQ?

               http://www.postgresql.org/docs/faqs/FAQ.html

Reply via email to