Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 28 Mar 2016 14:08:36 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Ryan Steinmetz <zi@FreeBSD.org>
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r411744 - head/sysutils/file
Message-ID:  <20160328140836.GB27920@FreeBSD.org>
In-Reply-To: <201603232323.u2NNNX7R045554@repo.freebsd.org>
References:  <201603232323.u2NNNX7R045554@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Mar 23, 2016 at 11:23:33PM +0000, Ryan Steinmetz wrote:
> New Revision: 411744
> URL: https://svnweb.freebsd.org/changeset/ports/411744
> 
> Log:
>   - Update to 5.25
>   
> [...]
> @@ -16,7 +16,7 @@ USES=		libtool
>  USE_LDCONFIG=	yes
>  
>  GNU_CONFIGURE=	yes
> -CONFIGURE_ARGS+=	--enable-fsect-man5
> +CONFIGURE_ARGS+=--enable-fsect-man5

This change is wrong for two reasons: first, it is gratuitous, and second,
even if you were distressed by "broken" indentation, you should've s/+=/=/
instead (as amending CONFIGURE_ARGS here is bogus).

./danfe



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