NSS cert tests fail, beause Android doesn't have /tmp directory

RESOLVED FIXED in 3.14.2

Status

NSS
Libraries
P2
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: kaie, Assigned: kaie)

Tracking

3.14.2
3.14.2
ARM
Android

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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

5 years ago
Created attachment 697837 [details] [diff] [review]
patch v1

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)

Updated

5 years ago
Depends on: 440617
(Assignee)

Comment 2

5 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

5 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
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.14.2

Comment 4

5 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.