Closed
Bug 46420
Opened 25 years ago
Closed 24 years ago
NSS port for OS/2
Categories
(NSS :: Tools, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
3.0.1
People
(Reporter: hctrinh, Assigned: rrelyea)
Details
Attachments
(11 files)
9.07 KB,
patch
|
Details | Diff | Splinter Review | |
5.11 KB,
text/plain
|
Details | |
845 bytes,
patch
|
Details | Diff | Splinter Review | |
580 bytes,
patch
|
Details | Diff | Splinter Review | |
1.73 KB,
patch
|
Details | Diff | Splinter Review | |
1.32 KB,
patch
|
Details | Diff | Splinter Review | |
8.50 KB,
text/plain
|
Details | |
4.03 KB,
text/plain
|
Details | |
351 bytes,
patch
|
Details | Diff | Splinter Review | |
291 bytes,
patch
|
Details | Diff | Splinter Review | |
5.37 KB,
text/plain
|
Details |
Via this bug, we will submit build/makefile/code changes required to
build NSS for OS/2, in both gcc/emx and VACPP builds.
Comment 1•25 years ago
|
||
Huynh, please use the "Create a new attachment"
link in the bug report to attach your patch and
we will review and test it. Thanks.
Reporter | ||
Comment 2•25 years ago
|
||
Wan-Teh, thanks for responding so soon.
One of the changes consists of replacing the explicit .a suffix to
$(LIB_SUFFIX) in the makefiles.
Do I have to submit individual diff for each makefile or can someone
with check-in right do a global change to the following makefiles:
- security/nss/lib/fortcrypt/swfort/pkcs11
- security/nss/cmd
- security/nss/cmd/swfort
- security/nss/cmd/swfort/newuser
- security/nss/cmd/signtool
- security/nss/cmd/pk12util
- security/nss/cmd/crmftest
- security/nss/cmd/crmf.cgi
Reporter | ||
Comment 3•25 years ago
|
||
corrected spelling of "port" in summary line
Summary: NSS prot for OS/2 → NSS port for OS/2
Comment 5•25 years ago
|
||
The patch is fine -- I assume you will add mozilla/security/coreconf/OS2.mk?
Reporter | ||
Comment 6•25 years ago
|
||
Yes, I have a prelim versin of os2.mk whicj I will submit shortly.
Comment 7•25 years ago
|
||
Do you want us to check these in?
If so, what branch/etc.
Thanks
Comment 8•25 years ago
|
||
We should check these in so that they get
checked into the tip and the appropriate
branches.
Could you attach your OS2.mk file?
Bob, could you check these in? I've already
reviewed their patches, so you just need to
review OS2.mk. Thanks.
Assignee: hctrinh → relyea
Reporter | ||
Comment 9•25 years ago
|
||
Reporter | ||
Comment 10•25 years ago
|
||
I attached a preliminary version of OS2.mk per your request. I am
expecting the BSAFE source code soon and will submit additional build
related changes to OS2.mk and other other files very soon.
Reporter | ||
Comment 11•25 years ago
|
||
Assignee | ||
Comment 12•25 years ago
|
||
Yes, I can check it in.
bob
Reporter | ||
Comment 13•25 years ago
|
||
Reporter | ||
Comment 14•25 years ago
|
||
Reporter | ||
Comment 15•25 years ago
|
||
Reporter | ||
Comment 16•25 years ago
|
||
Bob; Your comment on 8/17 indicates that you will check-in the changes
for Batch 01,02,03. I would appreciate if you could append a note
saying that the check-in is done. Thanks.
Wan-teh; Could you please review attached Batch04, 05. Thanks.
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Reporter | ||
Comment 17•25 years ago
|
||
Reporter | ||
Comment 18•25 years ago
|
||
Reporter | ||
Comment 19•25 years ago
|
||
Updated•25 years ago
|
Target Milestone: --- → 3.0.1
Reporter | ||
Comment 20•25 years ago
|
||
Reporter | ||
Comment 21•25 years ago
|
||
Assignee | ||
Comment 22•25 years ago
|
||
OK, this code is checked into the NSS_30_BRANCH. Please verify that it all still
works for OS/2. I merged the files in by hand.
I'll be merging the changes into the tip this morning...
Assignee | ||
Comment 23•24 years ago
|
||
patches are now in on both the tip and the branch.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•