call nsIFile.remove() only if it exists in build/media-resolution.js

RESOLVED FIXED

Status

Firefox OS
Gaia::Build
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: yurenju, Assigned: yurenju)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

we have intermittent build failure on try server on bug 1022192 and one of the root cause is we remove some files which does not exist. we should check file state before remove it.
Created attachment 8476506 [details] [review]
github PR: https://github.com/mozilla-b2g/gaia/pull/23134
Attachment #8476506 - Flags: review?(gduan)
https://tbpl.mozilla.org/?rev=943a40e6fd10291015c8edb2d08a6942de360336&tree=Gaia-Try
Assignee: nobody → yurenju.mozilla
Comment on attachment 8476506 [details] [review]
github PR: https://github.com/mozilla-b2g/gaia/pull/23134

This failure is so interesting . utils.ls should only show all existing files.
r=gduan
Attachment #8476506 - Flags: review?(gduan) → review+
merged.

https://github.com/mozilla-b2g/gaia/commit/2c5d0f5aa5ae65311c7f3bf715bc4f97ab8f6a13
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.