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

bug: New lazy loading broken for vim-ledger #1084

Closed
3 tasks done
gregorias opened this issue Oct 7, 2023 · 2 comments · Fixed by #1086
Closed
3 tasks done

bug: New lazy loading broken for vim-ledger #1084

gregorias opened this issue Oct 7, 2023 · 2 comments · Fixed by #1086
Labels
bug Something isn't working

Comments

@gregorias
Copy link

Did you check docs and existing issues?

  • I have read all the lazy.nvim docs
  • I have searched the existing issues of lazy.nvim
  • I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)

v0.9.2

Operating system/version

macOS 14.0

Describe the bug

vim-ledger started throwing startup errors due to undefined variables upon opening a ledger file.

The error is:

Error detected while processing BufReadPost Autocommands for "*.ledger"..FileType Autocommands for "*"..Syntax Autocommands
 for "*"..function <SNR>24_SynSet[25]..script /root/.repro/plugins/vim-ledger/syntax/ledger.vim:
line   15:
E121: Undefined variable: g:ledger_is_hledger
line   22:
E121: Undefined variable: b:is_hledger
line   70:
E121: Undefined variable: s:line_comment_chars
line   74:
E121: Undefined variable: b:is_hledger
line   88:
E121: Undefined variable: b:is_hledger
line  147:
E121: Undefined variable: b:is_hledger

The error has started appearing since a recent commit, ef2a5d0. @JulesNP also reports there an initialization error, which suggest that something happened to lazy-loading logic.

Steps To Reproduce

  1. touch ledger.ledger
  2. nvim ledger.ledger

Expected Behavior

No error is thrown at start up.

Repro

-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  { "ledger/vim-ledger", ft = "ledger" },
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
@gregorias gregorias added the bug Something isn't working label Oct 7, 2023
@gregorias gregorias changed the title bug: bug: New lazy loading broken for vim-ledger Oct 7, 2023
@folke folke closed this as completed in e4ea874 Oct 7, 2023
@folke
Copy link
Owner

folke commented Oct 7, 2023

Should be fixed now!

@gregorias
Copy link
Author

Looks good. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants