Bug 229095 - HP-UX build not using pthread properly
HP-UX build not using pthread properly
Product: 389
Classification: Community
Component: Directory Server (Show other bugs)
All Other
medium Severity medium
: ---
: ---
Assigned To: Nathan Kinder
Viktor Ashirov
Depends On:
Blocks: 152373 240316 FDS1.1.0
  Show dependency treegraph
Reported: 2007-02-16 18:11 EST by Nathan Kinder
Modified: 2015-12-07 11:44 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2015-12-07 11:44:51 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Proposed Fix (2.22 KB, patch)
2007-02-16 18:11 EST, Nathan Kinder
no flags Details | Diff
Revised Diffs (2.74 KB, patch)
2007-02-16 18:18 EST, Nathan Kinder
no flags Details | Diff
CVS Commit Message (1.59 KB, text/plain)
2007-02-16 18:26 EST, Nathan Kinder
no flags Details

  None (edit)
Description Nathan Kinder 2007-02-16 18:11:51 EST
The autotools build-system is not using libpthread properly.  I attempted to
test a build, and while the compile worked, ds_newinst-bin immediately core
dumped with an assertion from NSPR.

According to the man page for pthread on HP-UX, we need to build with
"-D_POSIX_C_SOURCE=199506L -D_HPUX_SOURCE" when using libpthread.  We also need
to ensure that libpthread is linked to our programs before libc.  I added
AC_DEFINE macros as well as adding "-lpthread" to AM_LDFLAGS for the HP-UX build.
Comment 1 Nathan Kinder 2007-02-16 18:11:51 EST
Created attachment 148252 [details]
Proposed Fix
Comment 2 Nathan Kinder 2007-02-16 18:18:57 EST
Created attachment 148253 [details]
Revised Diffs

I forgot to include the changes to config.h.in in the last set of diffs.  These
diffs have them included.
Comment 3 Nathan Kinder 2007-02-16 18:26:19 EST
Created attachment 148254 [details]
CVS Commit Message

Checked into ldapserver (HEAD).  Thanks to Rich for the review!

Note You need to log in before you can comment on or make changes to this bug.