Closed Bug 774054 Opened 10 years ago Closed 10 years ago

Switch mochitest file $(INSTALL) to install_cmd

Categories

(Firefox Build System :: General, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla17

People

(Reporter: rain1, Assigned: rain1)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
Easy win, since with pymake install_cmd is a native command.
Attachment #642356 - Flags: review?(mh+mozilla)
Comment on attachment 642356 [details] [diff] [review]
patch

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

::: config/makefiles/mochitest.mk
@@ +14,5 @@
>  endif
>  
>  define mochitest-libs-rule-template
>  libs:: $$($(1))
> +	$(call install_cmd,$$(foreach f,$$^,"$$(f)") $$(call mochitestdir,$(2)))

r=me with $$(call instead of $(call
Attachment #642356 - Flags: review?(mh+mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/e1b7eef10f3e
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.