client.mk: consolidate $(wildcard calls) to reduce overhead

RESOLVED FIXED in mozilla14

Status

defect
RESOLVED FIXED
7 years ago
Last year

People

(Reporter: joey, Assigned: joey)

Tracking

unspecified
mozilla14
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

No description provided.
Assignee: nobody → joey
Replaced several wildcard calls with a single call.  Cmd line length will not be an issue with only a handful of files being collected.
Attachment #611848 - Flags: review?(ted.mielczarek)
ping on the patch review
Comment on attachment 611848 [details] [diff] [review]
replaced [n] wildcard calls with a single call to gather files

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

::: client.mk
@@ +293,5 @@
>  
>  CONFIG_STATUS_DEPS := \
> +	$(wildcard \
> +	    $(CONFIGURES) \
> +		$(TOPSRCDIR)/allmakefiles.sh \

This indentation is a little funky. Can you just make everything inside the $(wildcard) line up?
Attachment #611848 - Flags: review?(ted.mielczarek) → review+
Same patch logic as last time with [tab] replaced by [space] within the indentation for $(wildcard ) arguments.  r=ted
Attachment #611848 - Attachment is obsolete: true
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/b23771ca17fd
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla14
https://hg.mozilla.org/mozilla-central/rev/b23771ca17fd
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Blocks: 1195264
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.