Disable the requirement for the fish-lsp to forward logging to a file.
Currently defaults to ~/path/to/fish-lsp/logs.txt
Future will default to an empty string, and accessing the logs from something other than the server/client connection will be disabled.
Once complete, the server will handle logging through the connection.console when a $fish_lsp_logsfile is set or unset
This will remove edge cases from distribution packages like NixOS, where the installation can't create a file with write permissions, or has different directory structure than our default repo expects
Initial Discussion Below:
I'm going to try to make the default logging behavior for the server opt-in by defining the variable fish_lsp_logfile and removing the fish-lsp start --enable/--disable logging key. This would avoid the opt-out workarounds the project currently enforces. When logging is not enabled to a file, it will still be available from most language clients because of connection.console.
Disable the requirement for the fish-lsp to forward logging to a file.
~/path/to/fish-lsp/logs.txtconnection.consolewhen a$fish_lsp_logsfileis set or unsetThis will remove edge cases from distribution packages like NixOS, where the installation can't create a file with write permissions, or has different directory structure than our default repo expects
Initial Discussion Below:
Originally posted by @ndonfris in https://github.com/ndonfris/fish-lsp/issues/46#issuecomment-2251101186