Bug 229095 - HP-UX build not using pthread properly
Summary: HP-UX build not using pthread properly
Alias: None
Product: 389
Classification: Retired
Component: Directory Server   
(Show other bugs)
Version: 1.0.4
Hardware: All Other
Target Milestone: ---
Assignee: Nathan Kinder
QA Contact: Viktor Ashirov
Depends On:
Blocks: 152373 240316 FDS1.1.0
TreeView+ depends on / blocked
Reported: 2007-02-16 23:11 UTC by Nathan Kinder
Modified: 2015-12-07 16:44 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2015-12-07 16:44:51 UTC
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 23:11 UTC, Nathan Kinder
no flags Details | Diff
Revised Diffs (2.74 KB, patch)
2007-02-16 23:18 UTC, Nathan Kinder
no flags Details | Diff
CVS Commit Message (1.59 KB, text/plain)
2007-02-16 23:26 UTC, Nathan Kinder
no flags Details

Description Nathan Kinder 2007-02-16 23:11:51 UTC
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 23:11:51 UTC
Created attachment 148252 [details]
Proposed Fix

Comment 2 Nathan Kinder 2007-02-16 23:18:57 UTC
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 23:26:19 UTC
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.