[PATCH 1 of 2 STABLE] make: introduce a target to clean everything but packages

Yuya Nishihara yuya at tcha.org
Fri Jul 29 10:31:37 EDT 2016


On Fri, 29 Jul 2016 00:52:48 +0200, Pierre-Yves David wrote:
> # HG changeset patch
> # User Pierre-Yves David <pierre-yves.david at ens-lyon.org>
> # Date 1469745599 -7200
> #      Fri Jul 29 00:39:59 2016 +0200
> # Branch stable
> # Node ID a52f8af0393b37ab6ddd8de4d91ee7d9cf2c36aa
> # Parent  387bdd53c77e9f61bd7b0d491717440f7f57563a
> make: introduce a target to clean everything but packages

Seems fine.

I'd rather add clean/distclean pair, but that might not be good for stable.

> diff -r 387bdd53c77e -r a52f8af0393b Makefile
> --- a/Makefile	Mon Jul 25 12:00:55 2016 -0700
> +++ b/Makefile	Fri Jul 29 00:39:59 2016 +0200
> @@ -60,7 +60,7 @@ wheel:
>  doc:
>  	$(MAKE) -C doc
>  
> -clean:
> +cleanbutpackages:
>  	-$(PYTHON) setup.py clean --all # ignore errors from this command
>  	find contrib doc hgext hgext3rd i18n mercurial tests \
>  		\( -name '*.py[cdo]' -o -name '*.so' \) -exec rm -f '{}' ';'
> @@ -68,10 +68,13 @@ clean:
>  	rm -f MANIFEST MANIFEST.in hgext/__index__.py tests/*.err
>  	rm -f mercurial/__modulepolicy__.py
>  	if test -d .hg; then rm -f mercurial/__version__.py; fi
> -	rm -rf build packages mercurial/locale
> +	rm -rf build mercurial/locale
>  	$(MAKE) -C doc clean
>  	$(MAKE) -C contrib/chg distclean

Nit: cleanbutpackages should be included in .PHONY list. Perhaps it can be
fixed in flight.


More information about the Mercurial-devel mailing list