From owner-svn-ports-head@FreeBSD.ORG Sat Aug 25 18:18:57 2012 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 1033) id 19C241065670; Sat, 25 Aug 2012 18:18:57 +0000 (UTC) Date: Sat, 25 Aug 2012 18:18:57 +0000 From: Alexey Dokuchaev To: svn-ports-all@freebsd.org Message-ID: <20120825181857.GA47730@FreeBSD.org> References: <201208251648.q7PGmtS6066622@svn.freebsd.org> <20120825170211.GA34647@FreeBSD.org> <50390BDE.20606@FreeBSD.org> Mime-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline In-Reply-To: <50390BDE.20606@FreeBSD.org> User-Agent: Mutt/1.4.2.1i Cc: svn-ports-head@freebsd.org, ports-committers@freebsd.org Subject: Re: svn commit: r303150 - head/www/mod_auth_cas X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 25 Aug 2012 18:18:57 -0000 On Sat, Aug 25, 2012 at 07:31:10PM +0200, Olli Hauer wrote: > On 2012-08-25 19:02, Alexey Dokuchaev wrote: > > On Sat, Aug 25, 2012 at 04:48:55PM +0000, Olli Hauer wrote: > >> New Revision: 303150 > >> URL: http://svn.freebsd.org/changeset/ports/303150 > >> > >> +post-install: > >> + @${CAT} ${PKGMESSAGE} > >> + > > > > cat(1)ing ${PKGMESSAGE} in post-install could benefit from being > > surrounded by ${ECHO_MSG}'s on both sides. This makes installation > > from port and package behave similarly. > > I agree, thats why I always add a line with ========... at the top and at > the bottom and have at last one whitespace before the text. > Instead two times ${ECHO_MSG} I would prefer an additional empty line at > the top/bottom in pkg-message. Hmm, last time I checked (couple of years ago), padding pkg-message[.in] file itself would result in \n emitted twice upon pkg_add(1). That said, the only way is to use ${ECHO_MSG} before and after. Yes, it is ugly, and ultimately should be fixed (if not already). ./danfe