Closed
Bug 826627
Opened 12 years ago
Closed 12 years ago
NSS cert tests fail, beause Android doesn't have /tmp directory
Categories
(NSS :: Libraries, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
3.14.2
People
(Reporter: KaiE, Assigned: KaiE)
References
Details
Attachments
(1 file)
1.05 KB,
patch
|
rrelyea
:
review+
|
Details | Diff | Splinter Review |
Test script cert.sh has a hardcoded filename, it attempts to use file /tmp/cert but there is no /tmp directory on a Android 4.2.x device. cert_extensions_test()
Assignee | ||
Comment 1•12 years ago
|
||
And we shouldn't create files outside our area, the use of a global filename has the risk for a race condition if a machine runs multiple tests in parallel...
Assignee: nobody → kaie
Attachment #697837 -
Flags: review?(rrelyea)
Assignee | ||
Comment 2•12 years ago
|
||
This is a bustage-level problem. Because I'm trying to run "tinderbox" and "buildbot" driven builds on shared machines, using different user accounts, and stale files stay around, which cannot be overwritten by another user account. I'll check it in, pending review.
Assignee | ||
Comment 3•12 years ago
|
||
Checking in cert.sh; /cvsroot/mozilla/security/nss/tests/cert/cert.sh,v <-- cert.sh new revision: 1.62; previous revision: 1.61 done
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.14.2
Comment 4•11 years ago
|
||
Comment on attachment 697837 [details] [diff] [review] patch v1 r+ rrelyea
Attachment #697837 -
Flags: review?(rrelyea) → review+
You need to log in
before you can comment on or make changes to this bug.
Description
•