aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | Merge pull request #10293 from bfredl/hi_msgBjörn Linse2019-07-05
|\ \ | |/ |/| highlight: show "hi Group" message correctly when not using the screen
| * highlight: show "hi Group" message correctly when not using the screenBjörn Linse2019-07-05
|/ | | | | | | ext_message doesn't set msg_col. Add a space and let client deal with wrapping. When using silent redirect show the unwrapped message form. Removed check is already part of msg_advance()
* build: FindLibIntl: fix warning about CMP0075 (#10427)Daniel Hahler2019-07-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | * build: FindLibIntl: fix warning about CMP0075 The common pattern elsewhere to set this only during the check, and here it was not unset, resulting in a warning later (on Alpine 3.10): -- Found Iconv -- Looking for pthread.h CMake Warning (dev) at /usr/share/cmake/Modules/CheckIncludeFile.cmake:80 (message): Policy CMP0075 is not set: Include file check macros honor CMAKE_REQUIRED_LIBRARIES. Run "cmake --help-policy CMP0075" for policy details. Use the cmake_policy command to set the policy and suppress this warning. CMAKE_REQUIRED_LIBRARIES is set to: /usr/lib/libintl.so For compatibility with CMake 3.11 and below this check is ignoring it. Call Stack (most recent call first): /usr/share/cmake/Modules/FindThreads.cmake:105 (CHECK_INCLUDE_FILE) CMakeLists.txt:482 (find_package) This warning is for project developers. Use -Wno-dev to suppress it. -- Looking for pthread.h - found * build: remove lists / REMOVE_ITEM around check_c_source_compiles
* Merge pull request #10425 from blueyed/luarocks-buildDaniel Hahler2019-07-05
|\ | | | | BuildLuarocks.cmake: use ROCKS_DIR
| * BuildLuarocks: improve commentsDaniel Hahler2019-07-05
| | | | | | | | | | | | | | I found it often irritation to see the same comment about "DEPENDS" there all the time. This uses the LuaRocks rock name as a comment instead (which is hard to parse from the recipe), moves the note about DEPENDS to the top (extended), and fixes/aligns some whitespace.
| * BuildLuarocks.cmake: use ROCKS_DIRDaniel Hahler2019-07-05
|/ | | | | | | | With LuaRocks 3 it will include the version, and this will make the upgrade easier then. It also makes sense to factor this out in general for clarity already. Pulled out of https://github.com/neovim/neovim/pull/10292.
* .gitignoreJustin M. Keyes2019-07-04
|
* tests: loop_spec: retry (#10413)Daniel Hahler2019-07-04
| | | | | | | | | | | | | | Flaky test (osx): [ FAILED ] ...is/build/neovim/neovim/test/functional/lua/loop_spec.lua @ 23: vim.loop timer ...is/build/neovim/neovim/test/functional/lua/loop_spec.lua:56: Expected objects to be the same. Passed in: (number) 0 Expected: (number) 2 stack traceback: ...is/build/neovim/neovim/test/functional/lua/loop_spec.lua:56: in function <...is/build/neovim/neovim/test/functional/lua/loop_spec.lua:23> It was bumped from sleeping for 20ms to 50ms in d33aaa0f5f already.
* build: use -fdiagnostics-color=always with Ninja (#10419)Daniel Hahler2019-07-04
| | | | | | "auto" does not work, since Ninja does not appear as a TTY to gcc/clang, but handles stripping escape codes itself when output is not to a TTY. Ref: https://github.com/ninja-build/ninja/blob/ca041d88f4d6/src/build.cc#L158-L169
* tests: fix flaky ':digraphs displays digraphs' (#10406)Daniel Hahler2019-07-04
|
* Merge pull request #10414 from blueyed/luacheckDaniel Hahler2019-07-04
|\ | | | | Improve luacheck integration, fix more issues.
| * Improve luacheck setup [skip appveyor]Daniel Hahler2019-07-04
| | | | | | | | | | | | | | | | - Move .luacheckrc to root, add read_globals=vim - Simplify lualint target, run it on all lua files - Lint preload.lua, but ignore W211 - Remove testlint target, included in lualint (and lint) - Clean up .luacheckrc
| * Fix luacheck errors for all Lua source filesSameed Ali2019-07-04
| |
* | build: FindLuaJit: handle luajit-2.1 include path suffix (#10418)Daniel Hahler2019-07-04
| |
* | third-party: fix warning with (un)bundled libtermkey/unibilium (#10416)Daniel Hahler2019-07-04
| | | | | | | | | | | | Fixes warning with: > cmake -S third-party -B .deps -DUSE_BUNDLED=OFF -DUSE_BUNDLED_UNIBILIUM=0 -DUSE_BUNDLED_LIBTERMKEY=1 > The dependency target "unibilium" of target "libtermkey" does not exist.
* | win,fs.c: Fix is_executable_ext #10209erw72019-07-04
|/ | | | | | | | | | | Fix issue that increment expression is executable and pointer ext pointing out of the buffer, if the pointer ext points to the terminating NUL. * Change termination condition judgment to one place * Change first condition judgment Change to not evaluate *(ext -1) in the first condition judgment. * Change to use copy_option_part instead of STRLCPY
* build: bundle: clean binary dir with new downloads (#10411)Daniel Hahler2019-07-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is required to (re)build e.g. libluv when the version changes (which triggers a new download). With `make deps`, changing the `LUV_URL`/`LUV_SHA256`, and `make deps` again: Before: > Up-to-date: /home/daniel/Vcs/neovim/.deps/usr/lib/libluv.a After: > Installing: /home/daniel/Vcs/neovim/.deps/usr/lib/libluv.a See with https://github.com/neovim/neovim/pull/10358 - where .deps contained libluv 1.29, the merge updates it to 1.30, but then it failed to link because `libluv.a` is considered to be up-to-date (after downloading the new version). Note that header files get installed, since they have the original time stamp, but `libluv.a` is being generated (does not use the timestamp from the archive here, but needs to get rebuild). It could be argued that the build system of the included project should catch/handle this, but it seems to be good practice to clean the binary / build dir with a new download to start from scratch. Ref: https://gitlab.kitware.com/cmake/cmake/issues/19452 Also fixes cmake/BuildLuv / luv-static: use name with -DTARGET for download command, and pass (shared) `SRC_DIR` explicitly instead.
* tests: shell-test: add INTERACT mode (#10405)Daniel Hahler2019-07-03
| | | | Use it to improve ls_spec: it should not use the user's real shell for performance and other reasons.
* tests: executable_spec: keep assertion (#10408)Daniel Hahler2019-07-03
| | | | It was moved to become pending in 18127f64c, but the assertion should be kept.
* tests: fix/improve "TUI background color" tests (#10229)Daniel Hahler2019-07-03
| | | | - Ensure the opposite of the expected bg is active. - Improves performance: 1.2s instead of 4.5s.
* ci: Travis: do not close fold on failure [skip ci]Daniel Hahler2019-07-03
| | | | | This makes it easier to inspect the test failure, without having to open two folds (typically), and scrolling around.
* build: Fix rule of `build/.ran-third-party-cmake` #10402Eisuke Kawashima2019-07-03
| | | In order that `$(DEPS_BUILD_DIR)/Makefile` exists under parallel make
* tests: fix flaky "TermClose event triggers when fast-exiting terminal job ↵Daniel Hahler2019-07-02
| | | | | | | | | | | stops" (#10377) [ ERROR ]...neovim/neovim/test/functional/autocmd/termclose_spec.lua @ TermClose event triggers when fast-exiting terminal job stops test/functional/helpers.lua:96: Vim(call):E900: Invalid channel id stack traceback: test/functional/helpers.lua:96: in function 'request' test/functional/helpers.lua:254: in function 'command' ...neovim/neovim/test/functional/autocmd/termclose_spec.lua:23: in function <...neovim/neovim/test/functional/autocmd/termclose_spec.lua:20>
* tests: fix flaky "terminal (with fake shell) with not arguments …" (#10401)Daniel Hahler2019-07-02
| | | | | | `retry()` would only try it three times (waiting for 20ms in between), despite the large timeout. Fixes https://github.com/neovim/neovim/issues/10265.
* Merge pull request #10397 from bfredl/cmdcleanupBjörn Linse2019-07-02
|\ | | | | cmdline: remove local variables i and j from command_line_state
| * cmdline: remove local variables i and j from command_line_stateBjörn Linse2019-07-02
|/
* defaults: wildoptions=pum,tagfile #10384Justin M. Keyes2019-07-01
| | | ref #6289
* test/old: pass Test_recover_root_dir on Windows (#10207)Jan Edmund Lazo2019-07-01
| | | | | | | | | | | | | * Revert "oldtests: win: keep set dir=/" This reverts commit 714e0f8bf05cc6febcbcd8cbcbe5f119c37a14b0. * test/old: set swap directory to non-existent drive Appveyor uses Administrator user so it can write anywhere in C:\\. Neovim creates a directory for the swap file if the directory does not exist. D:\\ is the backup/recovery drive so use F:\\ invoke error E303.
* Merge pull request #10389 from bfredl/luafloatBjörn Linse2019-07-01
|\ | | | | api/lua: make nvim_execute_lua use native lua floats, not special tables
| * api/lua: make nvim_execute_lua use native lua floats, not special tablesBjörn Linse2019-07-01
| | | | | | | | | | | | | | | | | | | | Rationale: the purpose of nvim_execute_lua is to simply call lua code with lua values. If a lua function expects a floating point value, it should be enough to specify a float as argument to nvim_execute_lua. However, make sure to preserve the existing roundtripping behavior of API values when using `vim.api` functions. This is covered by existing lua/api_spec.lua tests.
* | vim-patch:8.1.0452: MS-Windows: not finding intl.dll #10388Jan Edmund Lazo2019-07-01
| | | | | | | | | | Problem: MS-Windows: not finding intl.dll. Solution: Also find intl.dll next to libintl.dll. (Ken Takata) https://github.com/vim/vim/commit/7554c548a493cba50b2d0ea3521cac14f28a1f07
* | Merge pull request #10387 from bfredl/wildposBjörn Linse2019-07-01
|\ \ | |/ |/| cmdline: correct the column position of wildoptions=pum popupmenu
| * cmdline: correct the column position of wildoptions=pum popupmenuBjörn Linse2019-07-01
|/ | | | | - position might get invalid with "longest" match - position might be wrong when completed pattern ends in "/"
* build: fix warning with passively available libintl (#10380)Daniel Hahler2019-07-01
| | | Since 0364f51 it would display a warning.
* build: LibLUV: update required version [ci skip] (#10381)Daniel Hahler2019-07-01
| | | | | | | Required after d33aaa0f5. Does not really make a difference, since the VERSION is not handled with our FindLibLUV (due to missing pkg-config information (https://github.com/luvit/luv/issues/354)).
* Merge pull request #10316 from bfredl/cb_safetyBjörn Linse2019-06-30
|\ | | | | luv callbacks: throw error on deferred methods instead of crashing
| * make vim.loop == require'luv'Björn Linse2019-06-30
| | | | | | | | | | This avoids initializing libluv a second time if a plugin invokes require'luv'. It is probably not an issue, but better to be safe.
| * libluv: use luv_set_callback to control callback executionBjörn Linse2019-06-30
| | | | | | | | | | Disable the use of deferred API functions in a fast lua callback Correctly display error messages from a fast lua callback
| * rename: FUNC_API_ASYNC => FUNC_API_FASTBjörn Linse2019-06-30
| |
* | fileio.c: eliminate set_file_time() #10357Justin M. Keyes2019-06-30
| | | | | | Introduce os_file_settime(), remove cruft.
* | man.vim: Handle ANSI escape sequences with ":" #10267Kovas Palunas2019-06-30
|/ | | | closes #10267
* :digraphs : check for CTRL-C less often #10376Justin M. Keyes2019-06-30
| | | No need to do this so frequently.
* Makefile: CMAKE_INSTALL_PREFIX: skip parsing CMAKE_EXTRA_FLAGS if set (#10374)Daniel Hahler2019-06-30
| | | | | Fixes: > Recursive variable 'CMAKE_EXTRA_FLAGS' references itself (eventually). Stop.
* build: update cmake/LibFindMacros.cmake (#10355)Daniel Hahler2019-06-30
| | | Source: https://github.com/Tronic/cmake-modules/blob/d6b5e94625d41469eaf1e2c484d7204cd263893d/LibFindMacros.cmake
* Makefile: move `all` target to the top [ci skip] #10375Daniel Hahler2019-06-30
| | | Fixes "make: Nothing to be done for 'checkprefix'." after 5031e3298.
* doc [ci skip] #10177Justin M. Keyes2019-06-30
| | | ref #10278 #10279 #10353
* build: update some test dependencies (#10339)Daniel Hahler2019-06-29
| | | | | | | | | | * build: update some test dependencies * luacheck ignores * BuildLua: add ${BUSTED} to depends for ${BUSTED_LUA} This is required to rebuild it when busted gets updated.
* build: FindLibLUV: use PkgConfig (#10359)Daniel Hahler2019-06-29
| | | | | | Apparently forgotten in c83926cd0. - Also LIMIT_SEARCH was never used. - Also add LIBLUV_DEFINITIONS, as used with other Find modules.
* Makefile: revisit/improve checkprefix handling (#10348)Daniel Hahler2019-06-29
| | | | | | | | | | | | | | Main improvement: do not error out, but re-run CMake in case CMAKE_INSTALL_PREFIX changed, and only check it for "install". - only look at CMAKE_EXTRA_FLAGS via shell if not empty - add CMAKE_INSTALL_PREFIX to CMAKE_EXTRA_FLAGS (not CMAKE_FLAGS), to override it being set in CMAKE_EXTRA_FLAGS from local.mk - use an empty "checkprefix" target if CMAKE_INSTALL_PREFIX is not provided - skip checking of cached value without build/.ran-cmake; it will be run then anyway - only use it with "install" target; it is only relevant there - do not error, but re-run CMake (by removing the stamp file)
* :ls : show "R", "F" for terminal-jobs #10370Justin M. Keyes2019-06-29
| | | | | | | | | | | This matches Vim behavior. From `:help :ls` : R a terminal buffer with a running job F a terminal buffer with a finished job ? a terminal buffer without a job: `:terminal NONE` TODO: implement `:terminal NONE`. ref #10349