Closed
Bug 924569
Opened 11 years ago
Closed 11 years ago
b2g desktop mochitests shouldn't use the xre.zip from tooltool
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(1 file)
(deleted),
patch
|
jgriffin
:
review+
|
Details | Diff | Splinter Review |
B2G desktop mochitests are currently running on Cedar and Ash, however they are failing because the certutil binary isn't found:
https://tbpl.mozilla.org/php/getParsedLog.php?id=28756410&tree=Cedar
This is because they are attempting to use the xre.zip from tooltool which they shouldn't be.
This patch already landed on Cedar and seems to clear up the issue:
https://tbpl.mozilla.org/php/getParsedLog.php?id=28835376&tree=Ash
The 'container is null' error shown there is bug 921180.
Attachment #814504 -
Flags: review?(jgriffin)
Assignee | ||
Comment 1•11 years ago
|
||
*Er, that patch landed on ash-mozharness, not Cedar.
Comment 2•11 years ago
|
||
Comment on attachment 814504 [details] [diff] [review]
Patch 1.0 - don't use tooltool xre.zip
Review of attachment 814504 [details] [diff] [review]:
-----------------------------------------------------------------
Cool, thanks for finding this.
::: scripts/b2g_desktop_unittest.py
@@ +101,1 @@
> dirs['abs_%s_dir' % d] = os.path.join(
We should simplify this logic now that there is only one value in 'd'.
Attachment #814504 -
Flags: review?(jgriffin) → review+
Assignee | ||
Comment 3•11 years ago
|
||
With nit fixed: https://hg.mozilla.org/build/mozharness/rev/09aa2fde9936
Comment 4•11 years ago
|
||
in production
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•