608036 - Cleanup some of client.mk

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Justin Wood (:Callek)]

Attachment: clean it up

There are a few parts of client.mk that can use a bit of cleaning...

Most of this was found while I was working on Bug 607775. Explanations to follow

Comment 1

[Justin Wood (:Callek)]

Comment on attachment 486661 [details] [diff] [review]
clean it up

>-#
>-CVS = cvs
>+

Not used here, and sync's to what c-c has.

>-RUN_AUTOCONF_LOCALLY = 1

When no longer have an in-tree version of the configures, so must run them locally anyway.

>-	$(wildcard $(TOPSRCDIR)/config/chrome-versions.sh) \
>-  $(wildcard $(addsuffix confvars.sh,$(wildcard $(TOPSRCDIR)/*/))) \
>+	$(wildcard $(addsuffix confvars.sh,$(wildcard $(TOPSRCDIR)/*/))) \

tab-vs-space, and there is no longer a chrome-versions.sh

>-ifdef MOZ_TOOLS
>-  CONFIGURE = $(TOPSRCDIR)/configure
>-endif
>-

Not needed since we have that ifeq for TOPSRCDIR and OBJDIR above. This is always set right, independant of MOZ_TOOLS

>-.PHONY: checkout real_checkout depend build profiledbuild maybe_clobber_profiledbuild export libs alldep install clean realclean distclean cleansrcdir pull_all build_all clobber clobber_all pull_and_build_all everything configure preflight_all preflight postflight postflight_all
>+.PHONY: checkout real_checkout depend build profiledbuild maybe_clobber_profiledbuild export libs alldep install clean realclean distclean cleansrcdir pull_all build_all clobber clobber_all pull_and_build_all everything configure preflight_all preflight postflight postflight_all upload sdk

adding the new |upload| and |sdk| targets to .PHONY for correctness.

Comment 2

[Justin Wood (:Callek)]

http://hg.mozilla.org/mozilla-central/rev/63c2aed72f69