Skip to content

/tmp/lsp.log doesn't work for a shared server #47

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.

Sign up for GitHub

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

Jump to bottom
Closed
hoichi opened this issue Dec 11, 2020 · 1 comment · Fixed by rescript-lang/rescript-editor-support#42
Closed

/tmp/lsp.log doesn't work for a shared server #47

hoichi opened this issue Dec 11, 2020 · 1 comment · Fixed by rescript-lang/rescript-editor-support#42

Comments

@hoichi
Copy link

hoichi commented Dec 11, 2020

Currently, the language server crashes for me when I work on a remote server (via a Remote - SSH plugin), with the following message:

Error: Command failed: /home/sergey/.vscode-server/extensions/chenglou92.rescript-vscode-1.0.2/server/linux/rescript-editor-support.exe dump /home/sergey/monorepo/frontend/packages/product-calendar/src/components/PcLink.res:19:14
Fatal error: exception Sys_error("/tmp/lsp.log: Permission denied")

And this is because /tmp/lsp.log is created and owned by my teammate, not by me. Deleting the file resolves the problem.

But even if we figure out the file permissions, won't it mean that only one person at a time is able to work at one particular machine? That would be unfortunate because the server is meant to be used for development by many people at once.
@hoichi
Copy link
Author

hoichi commented Dec 11, 2020 •
edited
Loading

As a workaround, setting a TMPDIR variable for every user should help. Unfortunately, I haven't figured out exactly how to set that variable so that the lang server can consume it. E.g., setting it in .profile on the remote machine doesn't seem to work.

cristianoc added a commit to rescript-lang/rescript-editor-support that referenced this issue Dec 14, 2020
@cristianoc
Don't create a log file by default.
bbab07a 
Fixes rescript-lang/rescript-vscode#47.
@cristianoc cristianoc mentioned this issue Dec 14, 2020
Merged
cristianoc added a commit to rescript-lang/rescript-editor-support that referenced this issue Dec 14, 2020
@cristianoc
Don't create a log file by default.
535d502 
Fixes rescript-lang/rescript-vscode#47.
cristianoc added a commit to rescript-lang/rescript-editor-support that referenced this issue Dec 14, 2020
@cristianoc
Don't create a log file by default.
e61bb6c 
Fixes rescript-lang/rescript-vscode#47.
cristianoc added a commit that referenced this issue Jan 6, 2021
@cristianoc
Sync rescript-editor support to commit 214d220.
d857b04 
- Fix type shown when hovering on record fields (see #52), and doc comments for records.
- Fix issue where type variables are printed with global renaming when hovering or autocompleting a module (see rescript-lang/rescript-editor-support#38).
- Fix issue where a log file was always created (see #47).
- Add support for hover on the id of toplevel module definitions (```module Id = ...```).
@cristianoc cristianoc mentioned this issue Jan 6, 2021
Merged
chenglou pushed a commit to chenglou/rescript-editor-support that referenced this issue Apr 24, 2021
@cristianoc @chenglou
Don't create a log file by default.
bca8e43 
Fixes rescript-lang/rescript-vscode#47.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't create a log file by default. rescript-lang/rescript-editor-support
1 participant
@hoichi