Closed Bug 544191 Opened 14 years ago Closed 14 years ago

Use system zlib on Mac OS X

Categories

(NSS :: Build, defect)

All
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
3.12.6

People

(Reporter: wtc, Assigned: wtc)

Details

Attachments

(1 file)

Attached patch Proposed patchSplinter Review
The attached patch allows us to use system zlib on Mac OS X.

All versions of Mac OS X that we support have zlib as a system library.
In Mac OS X 10.4 and later, the system zlib is version 1.2.3, which is
the latest release and the same version as the zlib in the NSS source
tree.

Firefox 3 requires Mac OS X 10.4 or later.

The reason we need to change the signtool and modutil makefiles
is that we can't add -lz to EXTRA_LIBS because $(PROGRAM) depends
on $(EXTRA_LIBS) in mozilla/security/coreconf/rules.mk:

$(PROGRAM): $(OBJS) $(EXTRA_LIBS)
        ...

gmake on Linux apparently has an extension that interprets
a dependency of -lz as /usr/lib/libz.so.  But gmake on Mac
OS X tries to make -lz (as expected) and fails.  So we have to
add -lz to OS_LIBS instead.
Attachment #425169 - Flags: review?(rrelyea)
Comment on attachment 425169 [details] [diff] [review]
Proposed patch

r+, but make sure that either mozilla wants this change, or if not, that they have an easy way to turn it off.

bob
Attachment #425169 - Flags: review?(rrelyea) → review+
This change by itself doesn't affect Mozilla because Mozilla doesn't build
modutil and signtool.  When I check in the makefile patch in bug 275744,
this change will affect Mozilla because lib/ssl will depend on zlib, but
Mozilla can build NSS with NSS_ENABLE_ZLIB= (unsetting NSS_ENABLE_ZLIB)
to turn it off.

I checked in the patch on the NSS trunk (NSS 3.12.6).

Checking in coreconf/Darwin.mk;
/cvsroot/mozilla/security/coreconf/Darwin.mk,v  <--  Darwin.mk
new revision: 1.24; previous revision: 1.23
done
Checking in nss/cmd/platlibs.mk;
/cvsroot/mozilla/security/nss/cmd/platlibs.mk,v  <--  platlibs.mk
new revision: 1.66; previous revision: 1.65
done
Checking in nss/cmd/modutil/Makefile;
/cvsroot/mozilla/security/nss/cmd/modutil/Makefile,v  <--  Makefile
new revision: 1.8; previous revision: 1.7
done
Checking in nss/cmd/signtool/Makefile;
/cvsroot/mozilla/security/nss/cmd/signtool/Makefile,v  <--  Makefile
new revision: 1.7; previous revision: 1.6
done
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.12.6
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: