On 2016/11/09 16:12, Antoine Jacoutot wrote:
> On Tue, Nov 08, 2016 at 06:56:45PM -0500, Jiri B wrote:
> > Hi,
> > 
> > an update of unpaper, it needs now libav which is part of ffmpeg.
> > 
> > I tooks diffs from pkgsrc who use ffmpeg and not libav directly.
> > I can't comment those diff but it works for me on amd64.
> 
> I don't like adding random patches without an explanation about what they do.

Agreed. And I'm not sure the unpaper.c patch is needed. There's another
version of the ffmpeg patch at https://github.com/Flameeyes/unpaper/pull/42
but nothing committed upstream.

> > I removed README as it was useless anyway and new doc files are
> > markdown-based so I did not include them.
> > 
> > I'v tried my best ;)
> 
> You are probably missing a bdep on textproc/libxslt for xsltproc.
> HOMEPAGE should be set to https://www.flameeyes.eu/projects/unpaper
> NO_TEST sould be removed and oh by the way, all tests fail so you need to 
> have a look at that.
> 
> It's a good start though ;-)

This fixes a bunch of them:

TEST_DEPENDS=   sysutils/coreutils

pre-test:
        ln -fs ${LOCALBASE}/bin/gmd5sum ${WRKDIR}/bin/md5sum

Reply via email to