On Thu, May 01, 2014 at 09:17:25PM +0400, Sergey Bronnikov wrote:
> On 18:26 Thu 01 May , Antoine Jacoutot wrote:
> > On Thu, May 01, 2014 at 08:13:11PM +0400, Sergey Bronnikov wrote:
> > > $ cat misc/py-asciinema/pkg/DESCR
> > > asciinema is a free and open source solution for recording the terminal
> > > sessions and sharing them on the web.
> > > 
> > > Tested on -current, amd64
> > > 
> > > P.S. actually it was sent to ports@ on 2014-01-24, now updated to latest 
> > > version.
> > 
> > You don't want to use FULLPKGNAME in the PLIST.
> > WRKSRC is the default, you don't need that.
> > Use MASTER_SITE_PYPI instead of github.
> > py-* is usually for python modules, since this is a standalone software, no 
> > need to prefix it.
> 
> Thank you for review.
> 
> In my case:
> $ make show=WRKSRC
> /usr/ports/pobj/asciinema-0.9.8/v0.9.8
> and without setting of WRKSRC packaging failed.

It is _not_ needed if you change the MASTER_SITES like I wrote in my previous 
mail.

> - MASTER_SITES pointed to MASTER_SITE_PYPI
> - keep FULLPKGNAME in PLIST
> - renamed py-asciinema to asciinema
> 
> 
> > -- 
> > Antoine



-- 
Antoine

Reply via email to