eccert.sh QA test script fails

RESOLVED INVALID

Status

NSS
Test
RESOLVED INVALID
13 years ago
13 years ago

People

(Reporter: Nelson Bolyard (seldom reads bugmail), Assigned: Nelson Bolyard (seldom reads bugmail))

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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

13 years ago
Created attachment 209665 [details] [diff] [review]
patch to eccert.sh, v1

Please review.
Attachment #209665 - Flags: superreview?(wtchang)
Attachment #209665 - Flags: review?(vipul.gupta)

Comment 2

13 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

13 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

13 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
Last Resolved: 13 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.