Thanks to a hint from Saul, I discovered that there are tests like this in portable.h but sunos5 is not defined: #if defined(sunos5) ... As it turns out, most of these are not causing harm in the current (trunk) code. But this should be cleaned. up.
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → 5.13
Created attachment 119328 [details] [diff] [review] proposed fix Removed all occurrences of sunos5. We no longer provide defines for these things at all (they are not used anywhere): HAVE_GETPWNAM_R HAVE_GETGRNAM_R SIG_FN WAIT_FLAGS
Fix committed to the trunk: mozilla/directory/c-sdk/ldap/include/portable.h new revision: 5.8; previous revision: 5.7 Fix 200528 - ldap/include/portable.h is confused about Solaris. Avoid using sunos5 (not defined). Removed definitions of these macros that are not used anwhere: HAVE_GETPWNAM_R, HAVE_GETGRNAM_R, SIG_FN, WAIT_FLAGS.
Status: ASSIGNED → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
TM -> 5.12
Target Milestone: 5.13 → 5.12
Created attachment 130481 [details] [diff] [review] fix for the fix (Linux build problem) I had to back out part of the fix... older Linux releases do not define in_addr_t.
fix for the fix committed to the trunk: mozilla/directory/c-sdk/ldap/include/portable.h new revision: 5.9; previous revision: 5.8 Follow up fix for 200528 - ldap/include/portable.h is confused about Solaris: older Linux versions do not have in_addr_t, so don't assume they do.
You need to log in before you can comment on or make changes to this bug.