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)

3.14.2
ARM
Android
defect

Tracking

(Not tracked)

RESOLVED FIXED
3.14.2

People

(Reporter: KaiE, Assigned: KaiE)

References

Details

Attachments

(1 file)

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()
Attached patch patch v1Splinter Review
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)
Depends on: 440617
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.
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 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.

Attachment

General

Created:
Updated:
Size: