From owner-svn-ports-head@FreeBSD.ORG  Mon Jan 27 14:20:52 2014
Return-Path: <owner-svn-ports-head@FreeBSD.ORG>
Delivered-To: svn-ports-head@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 1033)
 id 4C4BB2BC; Mon, 27 Jan 2014 14:20:52 +0000 (UTC)
Date: Mon, 27 Jan 2014 14:20:52 +0000
From: Alexey Dokuchaev <danfe@FreeBSD.org>
To: Baptiste Daroussin <bapt@FreeBSD.org>
Subject: Re: svn commit: r341406 - head/devel/bncsutil-ghost++
Message-ID: <20140127142052.GA71123@FreeBSD.org>
References: <201401271337.s0RDbvuO022647@svn.freebsd.org>
 <20140127140836.GB68244@FreeBSD.org>
 <20140127141225.GA33006@ithaqua.etoilebsd.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20140127141225.GA33006@ithaqua.etoilebsd.net>
User-Agent: Mutt/1.5.22 (2013-10-16)
Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org,
 ports-committers@freebsd.org
X-BeenThere: svn-ports-head@freebsd.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: SVN commit messages for the ports tree for head
 <svn-ports-head.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/svn-ports-head>,
 <mailto:svn-ports-head-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-ports-head/>
List-Post: <mailto:svn-ports-head@freebsd.org>
List-Help: <mailto:svn-ports-head-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-ports-head>,
 <mailto:svn-ports-head-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 27 Jan 2014 14:20:52 -0000

On Mon, Jan 27, 2014 at 03:12:26PM +0100, Baptiste Daroussin wrote:
> On Mon, Jan 27, 2014 at 02:08:36PM +0000, Alexey Dokuchaev wrote:
> > On Mon, Jan 27, 2014 at 01:37:57PM +0000, Baptiste Daroussin wrote:
> > > New Revision: 341406
> > > URL: http://svnweb.freebsd.org/changeset/ports/341406
> > > QAT: https://qat.redports.org/buildarchive/r341406/
> > > 
> > > +MAKE_ARGS=	PREFIX=${STAGEDIR}${PREFIX}
> > 
> > It would be nice (per our previous discussion) to briefly summarize in
> > the commit log why, in the particular case, such construct is safe, as
> > generally is is not.
> 
> Right; in that case I check that PREFIX is only used during the install
> phase (in the upstream build process) I also checked that the install
> process wasn't going to hardcode PREFIX somewhere. [...]

Baptiste, I know that you, as overall accurate committer, had done all
the checking required to ensure that this construct, in this particular
case is OK.  My point is that it better be briefly reflected in the log,
as other, less careful or experienced folks might see it as "Oh, OK, I
can simply set prefix to ${STAGEDIR}/${PREFIX} and be cone with it."

./danfe