On Sun, May 03, 2015 at 03:35:15PM +0200, albat0...@wanadoo.fr wrote:
> Hi,
> 
> Here is a sum up of the diff:
> 
> - Source 2.1.3 instead of 2.0.4 (a jump of 19 months).
> 
> - Inclusion of libmagic (thanks Josh Grosse for pointed that
> out). The port can build without but port-lib-depends-check
> stumbles on it and the RELEASE-NOTES mentions it improves the file
> format detection.
> 
> - Mirror list against those on the official site.
> 
> - Removal of unknown configure options: without-aiksaurus and
> enable-threads=posix (replaced by with-packaging=posix).
> 
> - Usage of QT4DIR to find moc and iuc binaries.
> 
> This was tested on amd64 and i386. Is there anyone to have a go on
> powerpc and sparc64 (better backup ~/.lyx) and/or commit it?
> # pkg_info lyx|grep Maintainer
> Maintainer: The OpenBSD ports mailing-list <ports@openbsd.org>

Minor nits: EXTRACT_SUFX defaults to .tar.gz so you shouldnt need it,
and empty REVISION isnt needed either.
You shouldnt remove @pkgpath annotations on the top, nor the @exec lines
at the bottom. Since you add icons in /hicolor/ a RUN_DEPENDS on
desktop-file-utils should be added, and the corresponding @exec
annotations after the update-desktop-database ones.

Other than that it generally looks good.

Landry

Reply via email to