Closed
Bug 324742
Opened 19 years ago
Closed 19 years ago
eccert.sh QA test script fails
Categories
(NSS :: Test, defect)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: nelson, Assigned: nelson)
References
Details
Attachments
(1 file)
3.99 KB,
patch
|
Details | Diff | Splinter Review |
Vipul and Christophe both reported that the script eccert.sh fails when attempting to generate the first self-signed cert. While investigating this today, I found several problems with eccert.sh that I had to fix before I could get the test to run. The problem was that eccert.sh creates two files in the wrong directory. Consequently, certutil cannot find the password and "noise" files it needs. Patch forthcoming.
Assignee | ||
Comment 1•19 years ago
|
||
Please review.
Attachment #209665 -
Flags: superreview?(wtchang)
Attachment #209665 -
Flags: review?(vipul.gupta)
Comment 2•19 years ago
|
||
Hi Nelson, I'm still confused about what changed. Why is it that cert.sh makes references to ${NOISE_FILE} and ${R_NOISE_FILE} instead of ${CADIR}/${R_NOISE_FILE}. Why do these scripts have to be different in this respect now? I looked through the CVS log for both files and couldn't figure it out. vipul (In reply to comment #1) > Created an attachment (id=209665) [edit] > patch to eccert.sh, v1 > > Please review. >
Assignee | ||
Comment 3•19 years ago
|
||
Comment on attachment 209665 [details] [diff] [review] patch to eccert.sh, v1 cancelling review requests.
Attachment #209665 -
Flags: superreview?(wtchang)
Attachment #209665 -
Flags: review?(vipul.gupta)
Assignee | ||
Comment 4•19 years ago
|
||
The ecc test shell scripts are being worked on in antoher bug, so this patch/bug isn't needed. Marking invalid.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•