Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 29 Jan 2016 04:03:19 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Adam Weinberger <adamw@adamw.org>
Cc:        Adam Weinberger <adamw@FreeBSD.org>, ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r407331 - head/net/geoipupdate
Message-ID:  <20160129040319.GA58352@FreeBSD.org>
In-Reply-To: <17E923F7-F432-46B5-BF9F-C77D3BB9B0BE@adamw.org>
References:  <201601270116.u0R1G1Vl041516@repo.freebsd.org> <20160129015402.GA96343@FreeBSD.org> <17E923F7-F432-46B5-BF9F-C77D3BB9B0BE@adamw.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Jan 28, 2016 at 08:51:55PM -0700, Adam Weinberger wrote:
> > On 28 Jan, 2016, at 18:54, Alexey Dokuchaev <danfe@FreeBSD.org> wrote:
> > On Wed, Jan 27, 2016 at 01:16:01AM +0000, Adam Weinberger wrote:
> >> New Revision: 407331
> >> URL: https://svnweb.freebsd.org/changeset/ports/407331
> >> 
> >> [...]
> >> @@ -1,4 +1,4 @@
> >> bin/geoipupdate
> >> -@sample etc/GeoIP.conf.sample
> >> +@sample etc/GeoIP.conf etc/GeoIP.conf.default
> > 
> > I think you've got it backwards, per what I read in CHANGE@20150926 and
> > running post-install() from +MANIFEST with commands prepended with echo:
> > [...]
> 
> Oh! You're totally right. I got confused by the wording from the
> porters-handbook.
> 
> "@sample file [file]" doesn't do much to resolve ambiguity :-P

Yes, I agree that "@sample file [file]" is not the best wording.  But in
that section 7.3, there's a Note with correct and more clear example.

> I'd suggest adding, after the first paragraph of 7.6.9:
> 
> 	So,
> 		@sample etc/myapp.conf.sample
> 	and
> 		@sample etc/myapp.conf.sample etc/myapp.conf
> 	are equivalent.

Yeah, perhaps some clarification of this sort could be helpful.

./danfe



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