Closed
Bug 935343
Opened 11 years ago
Closed 11 years ago
Convert content/media/tests to manifest format
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla28
People
(Reporter: kinetik, Assigned: kinetik)
References
Details
(Whiteboard: [qa-])
Attachments
(2 files)
1.33 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
21.68 KB,
patch
|
cajbir
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Comment 2•11 years ago
|
||
I dropped the MOZ_DASH tests since we're going to remove that code soon anyway (and was mainly worried that it'd be added to mozinfo.py and never removed).
Try push: https://tbpl.mozilla.org/?tree=Try&rev=9d2ab8fe1df7
Attachment #827778 -
Flags: review?(chris.double)
Updated•11 years ago
|
Attachment #827778 -
Flags: review?(chris.double) → review+
Comment 4•11 years ago
|
||
Comment on attachment 827776 [details] [diff] [review]
bug935343_mozinfo_v0.patch
Review of attachment 827776 [details] [diff] [review]:
-----------------------------------------------------------------
::: python/mozbuild/mozbuild/mozinfo.py
@@ +78,5 @@
> d['asan'] = substs.get('MOZ_ASAN') == '1'
> d['tests_enabled'] = substs.get('ENABLE_TESTS') == "1"
> d['bin_suffix'] = substs.get('BIN_SUFFIX', '')
>
> + d['suite'] = bool(substs.get('MOZ_SUITE'))
Let's not add this, you can just use appname=='seamonkey' already.
Attachment #827776 -
Flags: review?(ted) → review+
Assignee | ||
Comment 5•11 years ago
|
||
Pushed with MOZ_SUITE stuff addressed:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f2f60a737f5b
https://hg.mozilla.org/integration/mozilla-inbound/rev/7f083546aa95
Comment 6•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f2f60a737f5b
https://hg.mozilla.org/mozilla-central/rev/7f083546aa95
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Updated•11 years ago
|
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•