svn commit: r314117 - head/games/kye

[Jason Helfman]

On Thu, Mar 14, 2013 at 1:17 AM, Alexey Dokuchaev <danfe at freebsd.org> wrote:

> On Wed, Mar 13, 2013 at 11:59:22PM -0800, Beech Rintoul wrote:
> > On Wednesday 13 March 2013 23:53:30 Alexey Dokuchaev wrote:
> > > I think that tindy will complain about nonexistent files upon package
> > > deletion.
> > >
> > > > as this is a PLIST_SUB, and it will translate to:
> > > >
> > > > share/doc/kye/NEWS
> > > > share/doc/kye/README
> > >
> > > Correct.  That means, it now duplicates PORTDOCS set in the Makefile.
> >
> > This did not fail either on my tindy or on redports
>
> Hmm, did you use -plistcheck switch when running ./tc?  'cause it should,
> per this quick test:
>
> $ make generate-plist add-plist-docs
> $ grep doc /tmp/usr/ports/games/kye/work/.PLIST.mktmp
> share/doc/kye/NEWS
> share/doc/kye/README
> @dirrm share/doc/kye
> share/doc/kye/NEWS
> share/doc/kye/README
> @dirrm share/doc/kye
>
> ./danfe
>
>
PORTDOCS make variable assignment is different then a PLIST_SUB. It also
depends how it is used in the Makefile. In this instance, they are both
used correctly.

-jgh