Closed
Bug 819399
Opened 12 years ago
Closed 12 years ago
Add b2g OS_TARGET to source/config/writemozinfo.py
Categories
(Testing :: XPCShell Harness, defect)
Testing
XPCShell Harness
Tracking
(firefox19 fixed, firefox20 fixed, b2g18 fixed)
RESOLVED
FIXED
mozilla20
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(1 file)
1.03 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
For B2G we need to pass in {'os': 'gonk'} to the manifests since mozinfo isn't smart enough to handle emulators/devices yet.
Comment 1•12 years ago
|
||
We already have this:
http://mxr.mozilla.org/mozilla-central/source/config/writemozinfo.py
This is the script that writes out the mozinfo.json we use in the xpcshell harness. You should just make it write the correct value.
Assignee | ||
Comment 2•12 years ago
|
||
Ah cool. I was wondering where the "android" was being defined. Looks like we can disable tests without any patch using 'toolkit == "gonk"'
Though we should probably add the OS_TARGET for b2g to the known_os list anyway.
Assignee | ||
Comment 3•12 years ago
|
||
<ted> what's OS_TARGET in a b2g build, gonk?
<ahal> ted: I have no idea
<ted> $ make -sC objdir-gecko/ echo-variable-OS_TARGET
<ted> Android
<ted> that's not fantastic
<ahal> ouch
<ahal> ted: yeah, we'll need to do something about that
<ted> ahal: i'd be fine with setting os="b2g" if toolkit==gonk
<ahal> ted: yeah, that's what I ended up doing for reftests too
Assignee | ||
Updated•12 years ago
|
Summary: Add ability to pass arbitrary metadata to xpcshell manifests → Add b2g OS_TARGET to source/config/writemozinfo.py
Assignee | ||
Comment 4•12 years ago
|
||
Attachment #690890 -
Flags: review?(mdas)
Assignee | ||
Updated•12 years ago
|
Attachment #690890 -
Flags: review?(mdas) → review?(ted)
Updated•12 years ago
|
Attachment #690890 -
Flags: review?(ted) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta]
Comment 6•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
Assignee | ||
Updated•12 years ago
|
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta] → [automation-needed-in-aurora][automation-needed-in-beta][automation-needed-in-b2g18]
Assignee | ||
Updated•12 years ago
|
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta][automation-needed-in-b2g18] → [automation-needed-in-aurora][automation-needed-in-b2g18]
Assignee | ||
Comment 7•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/c335d671fc17
https://hg.mozilla.org/releases/mozilla-b2g18/rev/cbe115a9a488
status-firefox19:
--- → fixed
status-firefox20:
--- → fixed
Whiteboard: [automation-needed-in-aurora][automation-needed-in-b2g18]
Updated•12 years ago
|
Whiteboard: [status-b2g18:fixed]
Updated•12 years ago
|
status-b2g18:
--- → fixed
Whiteboard: [status-b2g18:fixed]
You need to log in
before you can comment on or make changes to this bug.
Description
•