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

refactor: use new location for log files #116

Merged
merged 1 commit into from
Jun 9, 2022
Merged

refactor: use new location for log files #116

merged 1 commit into from
Jun 9, 2022

Conversation

ojroques
Copy link
Contributor

@ojroques ojroques commented Jun 9, 2022

Hello,

Neovim 0.8 has changed the default location of log files to ~/.local/state/nvim: see neovim/neovim#14090 (comment). The issue is that the old log folder ~/.cache/nvim is not created upon installation of Neovim 0.8 anymore so Paq would crash trying to write into it.

This PR updates the log file location accordingly while keeping backward compatibility.

(I also changed the case of logfile for consistency, I can revert this change if needed).

@savq
Copy link
Owner

savq commented Jun 9, 2022

Neovim 0.8 has changed the default location of log files

Good to know. Thanks!

@savq savq merged commit b3c7d04 into savq:master Jun 9, 2022
@sethidden
Copy link

sethidden commented Jun 21, 2022

If you're here because you're:

  1. getting an error about something with "log" and "stdpath" realted to this line: https://github.com/savq/paq-nvim/blob/master/lua/paq.lua#L6
  2. and you have nvim 0.8 installed

it's possible your version - despite being 0.8 - is not new enough. Make sure you have the latest nightly

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.

3 participants