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