On Sat, Feb 11, 2017 at 05:41:21PM +0000, Edd Barrett wrote:
> Hi,
>
> Here's neovim.
>
> Notes:
>
> * We can't run the full test suite until "busted" is ported.
>
> * The other tests have issues, but only when run under the ports
> tree. I *think* this is because the ports tree is running the tests
> under some terminal configuration that the suite doesn't expect. When
> run under ports, the terminal is confined to what looks like 80x25,
> yet if I run the tests outside of ports, the output uses the whole
> terminal. Weird. Any ideas?
>
> * No attempt to use LuaJIT for now, just plain Lua.
>
> * I've not ported any language plugin modules. You can get the Python
> helpers from pip for now. Lua is built in.
>
> * Needs uncommitted ports: lubuv, libvterm, lua-bitop. All submitted.
>
> If you have never used neovim, the config dir is ~/.config/nvim, and the
> config file is ~/.config/nvim/init.vim. You can symlink your existing
> vim config and directory, and this will work. Once in nvim, run
> ':CheckHealth' to see what else you might need to do.
>
> Please also read the README if you want to use this in tmux.
>
> I've been using manual source builds of nvim for a few months now, and
> I've had very few problems. However, there is a bug where nvim crashes
> when the window size changes under xfce terminal
> (https://github.com/neovim/neovim/issues/5692). (I'll add that the the
> README on the next iteration actually).
>
> I'll test this for the duration of next week to iron out problems.
>
> P.S. Whilst debugging the tests, I was searching for TERM in
> bsd.port.mk. Around line 1956, we have:
>
> ${SUDO} ${SETENV} ${_TERM_ ENV} ...
>
> Which I suspect should be:
>
> ${SUDO} ${SETENV} ${_TERM_ENV} ...
>
> Right?
>
As I mentioned in private, devel/gperf is a missing BUILD_DEPENDS.
Moreover, now that libuv no longer needs kvm, it seems that neovim
shouldn't need to link against it either, at least that's the impression
I got when grepping the sources for kvm. I used the patch below. It does
the trick, but probably there are better ways of doing it.
I don't do anything fancy with vim except using it a lot: no plugins
and 30 lines of .vimrc. For me, the port seems to work just fine.
diff -pu -rN neovim/Makefile /usr/ports/editors/neovim/Makefile
--- neovim/Makefile Sat Feb 11 18:21:09 2017
+++ /usr/ports/editors/neovim/Makefile Sun Feb 12 03:29:33 2017
@@ -18,13 +18,14 @@ MAINTAINER = Edd Barrett <e...@openbsd.org>
# Apache 2.0 + Vim License
PERMIT_PACKAGE_CDROM = Yes
-WANTLIB += c kvm m msgpackc pthread termkey unibilium util uv vterm
+WANTLIB += c m msgpackc pthread termkey unibilium util uv vterm
MODULES = devel/cmake \
devel/gettext \
lang/lua \
textproc/intltool
BUILD_DEPENDS = ${RUN_DEPENDS} \
+ devel/gperf \
devel/lpeg \
devel/lua-bitop
diff -pu -rN neovim/patches/patch-cmake_FindLibUV_cmake
/usr/ports/editors/neovim/patches/patch-cmake_FindLibUV_cmake
--- neovim/patches/patch-cmake_FindLibUV_cmake Thu Jan 1 01:00:00 1970
+++ /usr/ports/editors/neovim/patches/patch-cmake_FindLibUV_cmake Sun Feb
12 03:10:43 2017
@@ -0,0 +1,12 @@
+$OpenBSD$
+--- cmake/FindLibUV.cmake.orig Sun Feb 12 03:10:31 2017
++++ cmake/FindLibUV.cmake Sun Feb 12 03:10:33 2017
+@@ -66,7 +66,7 @@ endif()
+
+ check_library_exists(kvm kvm_open "kvm.h" HAVE_LIBKVM)
+ if(HAVE_LIBKVM)
+- list(APPEND LIBUV_LIBRARIES kvm)
++ list(APPEND LIBUV_LIBRARIES)
+ endif()
+
+ check_library_exists(nsl gethostbyname "nsl.h" HAVE_LIBNSL)
