On Sun, 14 Jun 2020 03:51:02 +0200
Klemens Nanni <k...@openbsd.org>:

> On Sat, Jun 13, 2020 at 10:15:01PM +0200, Frédéric GALUSIK wrote:
>  [...]  
> Thanks to cmake, this is relatively straight forward;  only a few
> things I could spot for now:
> 
> PKGNAME defaults to DISTNAME, so the former is redundant.
> 
> WANTLIB is incomplete, you'll want to see "make
> port-lib-depends-check".
> 
> COMPILER is set correctly, however a comment is appropiate to tell
> *why* such a selection is needed;  here it's probably the use of
> -std=c++17, so you could just add "# -std=c++17" above the COMPILER
> line, that's it.
> 
> For readability, "=" is usually followed by tabs in ports Makefiles to
> align the second column.
> 
> I've done all of this in the attached tarball, thanks a lot for
> working on kakoune :-)

looks good, I'm ok to import it with a minor change.

Comparing to Makefile.template, the variables MASTER_SITES and
EXTRACT_SUFX should be moved below WANTLIB.

Reply via email to