Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 23 Jan 2015 13:28:52 -0700
From:      Ian Lepore <ian@freebsd.org>
To:        Colin Percival <cperciva@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r277606 - head/release
Message-ID:  <1422044932.1038.3.camel@freebsd.org>
In-Reply-To: <201501232024.t0NKODZq079972@svn.freebsd.org>
References:  <201501232024.t0NKODZq079972@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 2015-01-23 at 20:24 +0000, Colin Percival wrote:
> Author: cperciva
> Date: Fri Jan 23 20:24:13 2015
> New Revision: 277606
> URL: https://svnweb.freebsd.org/changeset/base/277606
> 
> Log:
>   Provide a recipe of "true" for building the "release" target once its
>   prerequisites are satisfied, in order to avoid having an implicit rule
>   triggered by the presence of release.sh.
>   
>   Approved by:	gjb
> 
> Modified:
>   head/release/Makefile
> 
> Modified: head/release/Makefile
> ==============================================================================
> --- head/release/Makefile	Fri Jan 23 19:43:57 2015	(r277605)
> +++ head/release/Makefile	Fri Jan 23 20:24:13 2015	(r277606)
> @@ -279,6 +279,7 @@ ftp: packagesystem
>  	cp *.txz MANIFEST ftp
>  
>  release:	real-release vm-release cloudware-release
> +	@true
>  
>  real-release:
>  	${MAKE} -C ${.CURDIR} ${.MAKEFLAGS} obj
> 

Wouldn't '.PHONY: release' be a better/more-correct fix?

-- Ian





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1422044932.1038.3.camel>