Closed Bug 882848 Opened 11 years ago Closed 11 years ago

Use MOCHITEST_BROWSER_FILES more

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla24

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Attached patch Patch v1Splinter Review
      No description provided.
Attachment #762199 - Flags: review?(gps)
Comment on attachment 762199 [details] [diff] [review]
Patch v1

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

THANK YOU. THANK YOU. THANK YOU.

These rules have pained me for so long. Good riddance.

::: browser/devtools/responsivedesign/test/Makefile.in
@@ +1,3 @@
> +# This Source Code Form is subject to the terms of the Mozilla Public
> +# License, v. 2.0. If a copy of the MPL was not distributed with this
> +# file, You can obtain one at http://mozilla.org/MPL/2.0/.

Cargo cult!

::: toolkit/mozapps/plugins/tests/Makefile.in
@@ -34,5 @@
>  $(PROGRAMS): $(TESTROOT)/% : %$(BIN_SUFFIX)
>  	cp $< $@
>  
> -libs::
> -	$(INSTALL) $(foreach f,$(_BROWSER_FILES),"$(srcdir)/$f") $(TESTROOT)

Wow.
Attachment #762199 - Flags: review?(gps) → review+
What's holding up checkin?
Flags: needinfo?(Ms2ger)
I hit a failure on try that I blamed on this patch at first, but turned out to be something else. I expect to land it tomorrow morning.
Flags: needinfo?(Ms2ger)
https://hg.mozilla.org/mozilla-central/rev/4c4f75c20e9b
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
Blocks: 773349
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: