Landry Breuil, 19 Nov 2014 21:30:
> On Wed, Nov 19, 2014 at 09:18:54PM +0100, frantisek holop wrote:
> > Henrik Friedrichsen, 19 Nov 2014 19:55:
> >
> > >
> > > On Thu, Nov 13, 2014 at 03:35:41AM +0100, Henrik Friedrichsen wrote:
> > > > Some notes:
> > > > - multimedia/libdvdnav is required
> > > > - You probably shouldn't hardcode /usr/local, but instead use
> > > > ${LOCALBASE}
> > > > - During the initial check it says: Bluray support: no
> > > > If BluRay does not work with the current libbluray version, you
> > > > should probably remove the dependency or update the lib
> > > > - DESCR has lines that are longer than 80 characters
> > > > - WANTLIB is missing some entries (check "make port-lib-depends-check")
> > >
> > > The attached version includes fixes for the notes above, as well as:
> > > - Use GitHub variables to set master site
> > > - Bootstrap and download the recommended waf with the script provided
> >
> > i think it is not a good idea to let a port
> > roam around and download stuff after the make fetch
> > phase. also, the way i did it, waf also had a nice
> > checksum included in the port.
>
> waf ? no fucking way. We got rid of that utter piece of crap for a
> reason....
and the reason was: it is not meant to be packaged on
system level. it comes embedded in the project using
it. unfortunately the mpv guys do not do that and
made a fancy download script instead.
(do it with SUPDISTFILES)
-f
--
call it fate, call it luck, call me beautiful.
