Closed
Bug 1035624
Opened 10 years ago
Closed 10 years ago
Promise constructor does not work on gaia try server
Categories
(Infrastructure & Operations Graveyard :: CIDuty, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: yurenju, Assigned: jgriffin)
References
Details
Attachments
(2 files)
1.48 KB,
patch
|
Details | Diff | Splinter Review | |
1.44 KB,
patch
|
nthomas
:
review+
jgriffin
:
checked-in+
|
Details | Diff | Splinter Review |
we try to create Promise by constructor on bug 1029967 and it work well on local xulrunner, but it does not work on try server, attachment is a test cast to trigger failure. now we use Promise.defer() as an workaround for this issue but hopely we can use promise constructor since it will be in es6 draft.[1] [1] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise#Specification
Reporter | ||
Comment 1•10 years ago
|
||
try server: https://tbpl.mozilla.org/?rev=122277dfe95e29e2e0eea924b47a85b599b9288c&tree=Gaia-Try
Reporter | ||
Comment 2•10 years ago
|
||
and you can apply the patch and execute |make promise|, it works on local xulrunner.
Comment 3•10 years ago
|
||
Hmm, I would have thought that bug 1025731 should have fixed this. jgriffin, missed configs/b2g/gaia_unit_production_config.py perhaps ?
Blocks: 1025731
Assignee | ||
Comment 4•10 years ago
|
||
(In reply to Nick Thomas [:nthomas] from comment #3) > Hmm, I would have thought that bug 1025731 should have fixed this. > > jgriffin, missed configs/b2g/gaia_unit_production_config.py perhaps ? Right you are, I'll update it now.
Assignee | ||
Comment 5•10 years ago
|
||
Attachment #8452413 -
Flags: review?(nthomas)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jgriffin
Comment 6•10 years ago
|
||
Comment on attachment 8452413 [details] [diff] [review] Update xre for gaia_unit tests, lgtm
Attachment #8452413 -
Flags: review?(nthomas) → review+
Assignee | ||
Comment 7•10 years ago
|
||
Comment on attachment 8452413 [details] [diff] [review] Update xre for gaia_unit tests, https://hg.mozilla.org/build/mozharness/rev/12ddabf24210
Attachment #8452413 -
Flags: checked-in+
Assignee | ||
Comment 8•10 years ago
|
||
(In reply to Jonathan Griffin (:jgriffin) from comment #7) > Comment on attachment 8452413 [details] [diff] [review] > Update xre for gaia_unit tests, > > https://hg.mozilla.org/build/mozharness/rev/12ddabf24210 Pushed to production. https://hg.mozilla.org/build/mozharness/rev/497dd135e8d7 John, can you merge this to your version of mozharness so it works on gaia-try?
Flags: needinfo?(jhford)
Comment 9•10 years ago
|
||
done: http://hg.mozilla.org/users/jford_mozilla.com/mozharness/rev/8a90cc9ed444
Flags: needinfo?(jhford)
Assignee | ||
Comment 10•10 years ago
|
||
Thanks.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 11•10 years ago
|
||
Mozharness patch merged to production branch (https://hg.mozilla.org/build/mozharness/rev/497dd135e8d7)
Updated•6 years ago
|
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Updated•4 years ago
|
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•