Link NSPR dynamic shared libraries with the -headerpad_max_install_names option

RESOLVED FIXED in 4.4.1

Status

NSPR
NSPR
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: Wan-Teh Chang, Assigned: Wan-Teh Chang)

Tracking

4.4.1
PowerPC
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

766 bytes, patch
Conrad Carlen (not reading bugmail)
: review+
Simon Fraser
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

14 years ago
We should link the NSPR dynamic shared libraries with
the -headerpad_max_install_names linker option so that
their install names can be changed with install_name_tool.

See the ld(1) man page on Mac OS X for documentation on
-headerpad_max_install_names.
(Assignee)

Comment 1

14 years ago
Created attachment 135199 [details] [diff] [review]
Proposed patch

Note that $(DSO_LDOPTS) is passed to the compiler $(CC) in
the MKSHLIB command, but it seems that I don't need to use
"-Wl," in front of -headerpad_max_install_names.
(Assignee)

Updated

14 years ago
Attachment #135199 - Flags: superreview?(sfraser)
Attachment #135199 - Flags: review?(ccarlen)

Comment 2

14 years ago
Comment on attachment 135199 [details] [diff] [review]
Proposed patch

second time lucky i guess
Attachment #135199 - Flags: superreview?(sfraser) → superreview+
Comment on attachment 135199 [details] [diff] [review]
Proposed patch

r=ccarlen
Attachment #135199 - Flags: review?(ccarlen) → review+
(Assignee)

Comment 4

14 years ago
Patch checked into the NSPR tip (NSPR 4.5), NSPR_4_4_BRANCH (NSPR
4.4.1), and NSPR_PRE_4_2_CLIENT_BRANCH (Mozilla 1.6 Beta).
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → 4.4.1
You need to log in before you can comment on or make changes to this bug.