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)

defect
Not set
normal

Tracking

(firefox19 fixed, firefox20 fixed, b2g18 fixed)

RESOLVED FIXED
mozilla20
Tracking Status
firefox19 --- fixed
firefox20 --- fixed
b2g18 --- fixed

People

(Reporter: ahal, Assigned: ahal)

References

Details

Attachments

(1 file)

For B2G we need to pass in {'os': 'gonk'} to the manifests since mozinfo isn't smart enough to handle emulators/devices yet.
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.
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.
<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
Summary: Add ability to pass arbitrary metadata to xpcshell manifests → Add b2g OS_TARGET to source/config/writemozinfo.py
Attachment #690890 - Flags: review?(mdas)
Attachment #690890 - Flags: review?(mdas) → review?(ted)
Attachment #690890 - Flags: review?(ted) → review+
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta]
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta] → [automation-needed-in-aurora][automation-needed-in-beta][automation-needed-in-b2g18]
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta][automation-needed-in-b2g18] → [automation-needed-in-aurora][automation-needed-in-b2g18]
Whiteboard: [status-b2g18:fixed]
Whiteboard: [status-b2g18:fixed]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: