svn commit: r324179 - in head/devel: . acme

[Alexey Dokuchaev]

On Mon, Aug 05, 2013 at 11:02:38AM -0400, Adam Weinberger wrote:
> On 2013-08-04, at 12:35, Alexey Dokuchaev <danfe at FreeBSD.org> wrote:
> > On Sat, Aug 03, 2013 at 07:33:16AM +0000, Jason Helfman wrote:
> >> +PLIST_FILES=	bin/acme \
> >> +		%%DATADIR%%/6502 \
> >> +		%%DATADIR%%/6502/Help.txt \
> >> +		%%DATADIR%%/6502/std.a \
> >> +		%%DATADIR%%/65816 \
> >> +		%%DATADIR%%/65816/Help.txt \
> >> +		%%DATADIR%%/65816/std.a \
> >> +		%%DATADIR%%/Own \
> >> +		%%DATADIR%%/Own/Help.txt
> >> +PLIST_DIRSTRY=	%%DATADIR%%/6502 \
> >> +		%%DATADIR%%/65816 \
> >> +		%%DATADIR%%/Own \
> >> +		%%DATADIR%%
> > 
> > Typically using PLIST_* makes sense for short (like, five or so) entries;
> > in this case I would say that traditional pkg-plist should have been used.
> 
> I think the problem here is that he has directories listed in both
> PLIST_FILES and PLIST_DIRSTRY.

Indeed, slipped through my eyes.  Apparently, the fact of misused PLIST_*
knobs was good enough reason to skip deeper analysis. :-)

./danfe