Reduce moz.build/Makefile.in files in /browser/metro tests

RESOLVED FIXED in Firefox 27

Status

--
minor
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: mbrubeck, Assigned: mbrubeck)

Tracking

Trunk
Firefox 27
All
Windows 8.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Created attachment 814261 [details] [diff] [review]
patch

In response to a plea by gps on dev-platform [1], this patch reduces the number of build steps and build files in our test directory, to help builds run faster.

The patch depends on bug 923273.

[1] https://groups.google.com/d/topic/mozilla.dev.platform/F-uESi8ByPE/discussion
Attachment #814261 - Flags: review?(rsilveira)
(Assignee)

Comment 1

5 years ago
Comment on attachment 814261 [details] [diff] [review]
patch

Can we also remove TEST_DIRS from /browser/metro/moz.build?

I wonder if we can also get rid of /browser/metro/base/Makefile.in by moving its content up a level...
Attachment #814261 - Flags: review?(Ms2ger)
(Assignee)

Comment 2

5 years ago
Created attachment 814264 [details] [diff] [review]
patch v2

The answer is yes, we do need to remove TEST_DIRS.
Attachment #814261 - Attachment is obsolete: true
Attachment #814261 - Flags: review?(rsilveira)
Attachment #814261 - Flags: review?(Ms2ger)
Attachment #814264 - Flags: review?(rsilveira)
Attachment #814264 - Flags: review?(Ms2ger)
Comment on attachment 814264 [details] [diff] [review]
patch v2

Review of attachment 814264 [details] [diff] [review]:
-----------------------------------------------------------------

r=me on the moz.build changes.
Attachment #814264 - Flags: review?(Ms2ger) → review+
Attachment #814264 - Flags: review?(rsilveira) → review+
https://hg.mozilla.org/mozilla-central/rev/d7e9a858c4b6
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 27
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.