Closed
Bug 820564
Opened 12 years ago
Closed 12 years ago
Drop support for Solaris versions older than Solaris 8 (SunOS 5.8)
Categories
(NSS :: Build, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
3.14.2
People
(Reporter: wtc, Assigned: wtc)
Details
Attachments
(2 files)
7.49 KB,
patch
|
KaiE
:
review+
rrelyea
:
superreview+
|
Details | Diff | Splinter Review |
1.78 KB,
patch
|
Details | Diff | Splinter Review |
Solaris 8 (SunOS 5.8) was released at the same time as Windows XP, so it is safe to drop support for Solaris versions older than Solaris 8. In fact, I think it is fine to only support Solaris 10 and newer, but I'm willing to be more conservative.
Attachment #691053 -
Flags: superreview?(rrelyea)
Attachment #691053 -
Flags: review?(kaie)
Comment 1•12 years ago
|
||
Comment on attachment 691053 [details] [diff] [review] Remove SunOS5.x.mk for SunOS 5.3-5.7 I agree, Given that we no longer have test machines for any of these platforms. r=kaie
Attachment #691053 -
Flags: review?(kaie) → review+
Assignee | ||
Comment 2•12 years ago
|
||
Comment on attachment 691053 [details] [diff] [review] Remove SunOS5.x.mk for SunOS 5.3-5.7 Patch checked in on the NSS trunk (NSS 3.14.2). Removing SunOS5.3.mk; /cvsroot/mozilla/security/coreconf/SunOS5.3.mk,v <-- SunOS5.3.mk new revision: delete; previous revision: 1.3 done Removing SunOS5.4.mk; /cvsroot/mozilla/security/coreconf/SunOS5.4.mk,v <-- SunOS5.4.mk new revision: delete; previous revision: 1.3 done Removing SunOS5.4_i86pc.mk; /cvsroot/mozilla/security/coreconf/SunOS5.4_i86pc.mk,v <-- SunOS5.4_i86pc.mk new revision: delete; previous revision: 1.5 done Removing SunOS5.5.1.mk; /cvsroot/mozilla/security/coreconf/SunOS5.5.1.mk,v <-- SunOS5.5.1.mk new revision: delete; previous revision: 1.3 done Removing SunOS5.5.1_i86pc.mk; /cvsroot/mozilla/security/coreconf/SunOS5.5.1_i86pc.mk,v <-- SunOS5.5.1_i86pc.mk new revision: delete; previous revision: 1.4 done Removing SunOS5.5.mk; /cvsroot/mozilla/security/coreconf/SunOS5.5.mk,v <-- SunOS5.5.mk new revision: delete; previous revision: 1.3 done Removing SunOS5.6.mk; /cvsroot/mozilla/security/coreconf/SunOS5.6.mk,v <-- SunOS5.6.mk new revision: delete; previous revision: 1.3 done Removing SunOS5.6_i86pc.mk; /cvsroot/mozilla/security/coreconf/SunOS5.6_i86pc.mk,v <-- SunOS5.6_i86pc.mk new revision: delete; previous revision: 1.4 done Removing SunOS5.7.mk; /cvsroot/mozilla/security/coreconf/SunOS5.7.mk,v <-- SunOS5.7.mk new revision: delete; previous revision: 1.3 done Removing SunOS5.7_i86pc.mk; /cvsroot/mozilla/security/coreconf/SunOS5.7_i86pc.mk,v <-- SunOS5.7_i86pc.mk new revision: delete; previous revision: 1.4 done
Assignee | ||
Comment 3•12 years ago
|
||
Kai: feel free to review or ignore this patch. This patch should allow us to just use coreconf/SunOS5.mk for all Solaris versions and remove all of the SunOS5.x.mk files. Right now each of the SunOS5.x.mk file adds -DSOLARIS2_x to OS_DEFINES, where x = 8, 9, 10, 11. I verified that none of the SOLARIS2_x macros is being used, so I didn't copy that to SunOS5.mk. Ideally we should do a CVS copy of SunOS5.mk to SunOS.mk. This will allow us to avoid the special case in coreconf/config.mk.
Assignee | ||
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 4•12 years ago
|
||
Comment on attachment 691053 [details] [diff] [review] Remove SunOS5.x.mk for SunOS 5.3-5.7 r+ relyea
Attachment #691053 -
Flags: superreview?(rrelyea) → superreview+
You need to log in
before you can comment on or make changes to this bug.
Description
•