Closed Bug 332633 Opened 19 years ago Closed 19 years ago

remove duplicate header files in nss/cmd/sslsample

Categories

(NSS :: Tools, defect, P2)

Product:
NSS
Component:
Tools
Version:
3.11
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: nelson, Assigned: nelson)

Details

Attachments

(1 file)

remove duplicate files, change manifests (checked in)
43.05 KB, patch
julien.pierre
: review+
Details | Diff | Splinter Review
NSS's header files that define the error strings are in nss/cmd/lib but they are also duplicated in nss/cmd/sslsample. They should not be duplicated there. Patch forthcoming.
Comment on attachment 217096 [details] [diff] [review] remove duplicate files, change manifests (checked in) r+, but REQUIRES = dbm should be removed from client.mn/server.mn rather than just commented .
Attachment #217096 - Flags: review?(julien.pierre.bugs) → review+
Checking in lib/manifest.mn; new revision: 1.10; previous revision: 1.9 Checking in SSLsample/client.mn; new revision: 1.4; previous revision: 1.3 Checking in SSLsample/server.mn; new revision: 1.4; previous revision: 1.3 Removing SSLsample/NSPRerrs.h; new revision: delete; previous revision: 1.2 Removing SSLsample/SECerrs.h; new revision: delete; previous revision: 1.2 Removing SSLsample/SSLerrs.h; new revision: delete; previous revision: 1.3
Status: NEW → RESOLVED
Closed: 19 years ago
Priority: -- → P2
Resolution: --- → FIXED
Target Milestone: --- → 3.11.1
I only fixed this on the trunk. If you think I should fix this on the 3.11 branch also, please reopen, change the target to 3.11.1 and request second review.
Target Milestone: 3.11.1 → 3.12
Comment on attachment 217096 [details] [diff] [review] remove duplicate files, change manifests (checked in) Checked in on branch Removing SSLsample/NSPRerrs.h; new revision: delete; previous revision: 1.2 Removing SSLsample/SECerrs.h; new revision: delete; previous revision: 1.2 Removing SSLsample/SSLerrs.h; new revision: delete; previous revision: 1.2.28.1 Checking in SSLsample/client.mn; new revision: 1.3.28.1; previous revision: 1.3 Checking in SSLsample/server.mn; new revision: 1.3.28.1; previous revision: 1.3 Checking in lib/manifest.mn; new revision: 1.9.28.1; previous revision: 1.9
Attachment #217096 - Attachment description: remove duplicate files, change manifests → remove duplicate files, change manifests (checked in)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: