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.