Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[new release] ocaml-lsp-server, lsp and jsonrpc (1.10.0) #20718

Merged
merged 1 commit into from
Feb 11, 2022

Conversation

rgrinberg
Copy link
Member

LSP Server for OCaml

CHANGES:

Features

  • Add better support for code folding: more folds and more precise folds

Fixes

CHANGES:

## Features

- Add better support for code folding: more folds and more precise folds

## Fixes

- Fix infer interface code action crash when implementation source does not
  exist (ocaml/ocaml-lsp#597)

- Improve error message when the reason plugin for merlin is absent (ocaml/ocaml-lsp#608)

- Fix `chdir` races when running ppx (ocaml/ocaml-lsp#550)

- More accurate completion kinds.
  New completion kinds for variants and fields. Removed inaccurate completion
  kinds for constructors and types. (ocaml/ocaml-lsp#510)

- Fix handling request cancellation (ocaml/ocaml-lsp#616)
@mseri
Copy link
Member

mseri commented Feb 11, 2022

Was it always failing on centos7?

#=== ERROR while compiling ocaml-lsp-server.1.10.0 ============================#
# context              2.0.10 | linux/x86_64 | ocaml-base-compiler.4.13.1 | pinned(https://github.com/ocaml/ocaml-lsp/releases/download/1.10.0/jsonrpc-1.10.0.tbz)
# path                 ~/.opam/4.13/.opam-switch/build/ocaml-lsp-server.1.10.0
# command              ~/.opam/opam-init/hooks/sandbox.sh build dune build -j 71 ocaml-lsp-server.install --release
# exit-code            1
# env-file             ~/.opam/log/ocaml-lsp-server-3024-60459a.env
# output-file          ~/.opam/log/ocaml-lsp-server-3024-60459a.out
### output ###
#          gcc vendor/lev/ev.o (exit 1)
# (cd _build/default/vendor/lev && /usr/bin/gcc -std=gnu99 -O2 -fno-strict-aliasing -fwrapv -fno-builtin-memcmp -pthread -fPIC -D_FILE_OFFSET_BITS=64 -std=gnu99 -O2 -fno-strict-aliasing -fwrapv -fno-builtin-memcmp -pthread -fPIC -libm -g -I /home/opam/.opam/4.13/lib/ocaml -o ev.o -c ev.c)
# In file included from submodules/lev/lev/vendor/ev.c:45:0:
# submodules/lev/lev/src/config.h:5:18: error: missing binary operator before token "("
# submodules/lev/lev/src/config.h:14:18: error: missing binary operator before token "("
# submodules/lev/lev/src/config.h:19:18: error: missing binary operator before token "("
# submodules/lev/lev/src/config.h:24:18: error: missing binary operator before token "("
# submodules/lev/lev/src/config.h:30:18: error: missing binary operator before token "("
# submodules/lev/lev/src/config.h:34:18: error: missing binary operator before token "("
# submodules/lev/lev/src/config.h:39:18: error: missing binary operator before token "("
# submodules/lev/lev/src/config.h:44:18: error: missing binary operator before token "("
# submodules/lev/lev/src/config.h:48:18: error: missing binary operator before token "("
# submodules/lev/lev/src/config.h:52:18: error: missing binary operator before token "("
# submodules/lev/lev/src/config.h:56:18: error: missing binary operator before token "("
# submodules/lev/lev/vendor/ev.c:2143:31: warning: 'ev_default_loop_ptr' initialized and declared 'extern' [enabled by default]

@rgrinberg
Copy link
Member Author

I don't know but centos 7 is so old that I don't have much desire to investigate. Let's just mark it unavailable there.

@mseri
Copy link
Member

mseri commented Feb 11, 2022

I don’t think we can make such a fine availability check, let’s merge as is. I think we can figure it out if somebody has issues with it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants