From owner-svn-ports-all@FreeBSD.ORG  Tue Oct 22 15:55:20 2013
Return-Path: <owner-svn-ports-all@FreeBSD.ORG>
Delivered-To: svn-ports-all@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 1033)
 id 6799554B; Tue, 22 Oct 2013 15:55:20 +0000 (UTC)
Date: Tue, 22 Oct 2013 15:55:20 +0000
From: Alexey Dokuchaev <danfe@FreeBSD.org>
To: Bryan Drewery <bdrewery@FreeBSD.org>
Subject: Re: svn commit: r331243 - head/sysutils/tmux
Message-ID: <20131022155520.GA79375@FreeBSD.org>
References: <201310220906.r9M96xBM055204@svn.freebsd.org>
 <526653CB.3080106@FreeBSD.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <526653CB.3080106@FreeBSD.org>
User-Agent: Mutt/1.5.21 (2010-09-15)
Cc: Mathieu Arnold <mat@FreeBSD.org>, svn-ports-head@freebsd.org,
 svn-ports-all@freebsd.org, ports-committers@freebsd.org
X-BeenThere: svn-ports-all@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
List-Id: SVN commit messages for the ports tree <svn-ports-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/svn-ports-all>,
 <mailto:svn-ports-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-ports-all>
List-Post: <mailto:svn-ports-all@freebsd.org>
List-Help: <mailto:svn-ports-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-ports-all>,
 <mailto:svn-ports-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 22 Oct 2013 15:55:20 -0000

On Tue, Oct 22, 2013 at 05:30:35AM -0500, Bryan Drewery wrote:
> On 10/22/2013 4:06 AM, Mathieu Arnold wrote:
> >  .if ${PORT_OPTIONS:MEXAMPLES}
> > -	@${MKDIR} ${EXAMPLESDIR}
> > -	@${INSTALL_DATA} ${WRKSRC}/examples/* ${EXAMPLESDIR}
> > +	@${MKDIR} ${STAGEDIR}${EXAMPLESDIR}
> > +	@${INSTALL_DATA} ${WRKSRC}/examples/* ${STAGEDIR}${EXAMPLESDIR}
> >  .endif
> 
> I think the general trend is to not hide any install commands since it
> is touching the user's system, and is good for debugging build logs.

(Not just to Mathieu, writing this given another occasion.)

Yes, please, make installation commands vocal (muting mkdir's is optional,
some people like it, it's okay).

I would also like to mention that, unless DOCS/EXAMPLES require extra
dependencies (e.g. Doxygen/DocBook) or just huge enough to create insane
I/O load, for staged ports, you can remove those .if ${PORT_OPTIONS...}
checks.  In many cases, it would collaterally allow to remove .include
<bsd.port.options.mk> (when there are no other PORT_OPTIONS checks).

./danfe