neo451 / feed.nvim

feature-rich neovim web feed reader, rss, atom and json, all in lua
https://luarocks.org/modules/neo451/feed.nvim
GNU General Public License v3.0
62 stars 1 forks source link

Had to manually create default dir #27

Closed BlakeLeBlancGuernsey closed 1 month ago

BlakeLeBlancGuernsey commented 1 month ago

On Windows, after installing the plugin, executing Feed returned the following error:

Error executing Lua callback: ...a/Local/nvim-data/lazy/plenary.nvim/lua/plenary/path.lua:511:
FileNotFoundError stack traceback:
[C]: in function 'error'
...a/Local/nvim-data/lazy/plenary.nvim/lua/plenary/path.lua:511: in function 'mkdir'
...c/AppData/Local/nvim-data/lazy/feed.nvim/lua/feed/db.lua:123: in function 'prepare_db'
...c/AppData/Local/nvim-data/lazy/feed.nvim/lua/feed/db.lua:144: in function
<...c/AppData/Local/nvim-data/lazy/feed.nvim/lua/feed/db.lua:141>
...pData/Local/nvim-data/lazy/feed.nvim/lua/feed/render.lua:15: in main chunk
[C]: in function 'require'
...lanc/AppData/Local/nvim-data/lazy/feed.nvim/lua/feed.lua:4: in function
<...lanc/AppData/Local/nvim-data/lazy/feed.nvim/lua/feed.lua:3>   

The fix was to manually create the default directory:

> mkdir ~/.local/share/nvim/feed/

Bad idea to add this step to the readme?

Thanks for a cool plugin! :)

neo451 commented 1 month ago

Thank you for the issue! This is ultimately because plenary.path has issues on windows. I will try to fix this further so that there is no need to manually create this directory. In the meantime I think people can just reference your solution.

neo451 commented 1 month ago

Upon further look, it was just a small bug in file path checking. No need to manually create now.

I also updated a the default path to the more idiomatic neovim's standard data path. You can get a update to see if it works, after update you can run Feed blowup to rm the old dir, and reenter neovim and start using the new dir. or just mv the directory.

BlakeLeBlancGuernsey commented 1 month ago

Cool! Nice one @neo451 !