Closed Bug 935343 Opened 11 years ago Closed 11 years ago

Convert content/media/tests to manifest format

Categories

(Core :: Audio/Video, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla28

People

(Reporter: kinetik, Assigned: kinetik)

References

Details

(Whiteboard: [qa-])

Attachments

(2 files)

No description provided.
Attached patch bug935343_mozinfo_v0.patch (deleted) — Splinter Review
Assignee: nobody → kinetik
Status: NEW → ASSIGNED
Attachment #827776 - Flags: review?(ted)
Attached patch bug935343_tests_v0.patch (deleted) — Splinter Review
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)
Filed bug 935349 for removing DASH.
Depends on: 935349
Attachment #827778 - Flags: review?(chris.double) → review+
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+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Whiteboard: [qa-]
Depends on: 975640
Depends on: 1496377
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: