Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Error wont go away (lazy.nvim luarocks) #191

Closed
d7omdev opened this issue Jul 3, 2024 · 24 comments · Fixed by sankantsu/image.nvim#1 or #236 · May be fixed by #172, b0o/image.nvim#1 or hareki/image.nvim#4
Closed

Error wont go away (lazy.nvim luarocks) #191

d7omdev opened this issue Jul 3, 2024 · 24 comments · Fixed by sankantsu/image.nvim#1 or #236 · May be fixed by #172, b0o/image.nvim#1 or hareki/image.nvim#4

Comments

@d7omdev
Copy link

d7omdev commented Jul 3, 2024

I keep getting this error whenever I open neovim although the plugin is working

image

@onurcanbektas
Copy link

Same here

@3rd
Copy link
Owner

3rd commented Jul 7, 2024

hey @kevinm6 do you know how we could avoid this? i think we don't have to force users to have a global lua 5.1 install (that fixes it) since it will be loaded by luajit anyway right?

@3rd 3rd changed the title Error wont go away Error wont go away (lazy.nvim luarocks) Jul 7, 2024
@3rd 3rd pinned this issue Jul 7, 2024
@3rd 3rd closed this as completed in 61c7651 Jul 7, 2024
@3rd
Copy link
Owner

3rd commented Jul 7, 2024

Pinned this and disabled rockspec until we find a workaround @abdulrahmanDev1 @kevinm6
There is a discussion about this on Reddit, I'm not that familiar with rocks, hopefully we can find the right way to do things soon: https://www.reddit.com/r/neovim/comments/1dng1d6/comment/lc2t5uh/

@3rd 3rd reopened this Jul 7, 2024
@kevinm6
Copy link
Contributor

kevinm6 commented Jul 7, 2024

Hi @3rd .. unfortunately for now it's required from Lazy the [email protected] !
A workaround is the one reported in the Lazy UI warnings (or checkhealth), in Lazy config set this

rocks = {
  hererocks = true
}

Another pointer where it was discussed

@3rd
Copy link
Owner

3rd commented Jul 7, 2024

What I think I'm missing is why is having a specific Lua version installed externally required when the end goal is pulling luarocks that will be executed with the LuaJIT bundled by Neovim anyway - I don't think I understand how it works. Is it for a build step on some of the rocks? Or is the external Lua installation used at runtime?

@kevinm6
Copy link
Contributor

kevinm6 commented Jul 7, 2024

I think is due to some internals of Lazy that are really integrated with Neovim, since that's the one that needs [email protected] and the way it loads the modules at runtime.

I don't think depends on something relative to LuaJIT or the rock build itself, since I'm capable of build it without Lua5.1 and in multiple ways outside NeoVim

@sebalfaro
Copy link

Hey guys @kevinm6 i'm stucked in the same point, so should i install globally lua 5.1 and also the the hererocks key on true, like this?

return {
  "3rd/image.nvim",
  dependencies = {
    "leafo/magick",
    "nvim-treesitter/nvim-treesitter",
    {
      "vhyrro/luarocks.nvim",
      opts = {
        rocks = {
          hererocks = true,
        },
      },
    },
  },

@3rd
Copy link
Owner

3rd commented Jul 8, 2024

I think if you install 5.1 globally it should just work, luarocks.nvim says it needs a global LuaJIT/Lua 5.1 installation.

@sebalfaro
Copy link

Thank you @3rd

@kapral18
Copy link

kapral18 commented Jul 8, 2024

@3rd luajit is installed with neovim when using brew because it's a core dependency. I have it now and I see the error about missing magick rock.

@kevinm6
Copy link
Contributor

kevinm6 commented Jul 8, 2024

Hey guys @kevinm6 i'm stucked in the same point, so should i install globally lua 5.1 and also the the hererocks key on true, like this?

  rocks = {
    hererocks = true,
  },

Sorry, maybe I wasn't clear.. the option rocks.hererocks = true is Lazy's one (it installs another plugin that is a python script and manage the rocks and lua versions installs).
If you enable it, it works without having to install manually Lua5.1 and make it available on $PATH.

There are other ways you can fix it, as reported from @3rd, like just install and make lua5.1 available on $PATH or if you stick with luarocks.nvim, just do it as before.

@kevinm6
Copy link
Contributor

kevinm6 commented Jul 8, 2024

@3rd luajit is installed with neovim when using brew because it's a core dependency. I have it now and I see the error about missing magick rock.

@kapral18
If you update image.nvim to the last commit, you should follow the Installation steps and requirements as before

@sebalfaro
Copy link

sebalfaro commented Jul 8, 2024

@kevinm6 @3rd working on my end without erros, i've added the hererocks line that @kevinm6 has suggested. Thank you both guys

  rocks = {
    hererocks = true,
  },

@d7omdev
Copy link
Author

d7omdev commented Jul 8, 2024

Thank you guys it's fixed for me too after I added the hererocks line I tried to do it before but it didn't work out for me,
Thank you all.

@kevinm6
Copy link
Contributor

kevinm6 commented Jul 8, 2024

@kevinm6 @3rd working on my end without erros, i've added the hererocks line that @kevinm6 has suggested. Thank you both guys

@sebalfaro @abdulrahmanDev1
Glad it helped. The amazing job is @3rd 's one, it seems now, my little contribution created more issues than benefits!

@3rd
Copy link
Owner

3rd commented Jul 8, 2024

@kevinm6 no no, your contribution is great, so Lazy requires Lua 5.1 installed correct?
Then we can re-enable the rockspec and document that requirement as well, would that be the best?

@sebalfaro
Copy link

sebalfaro commented Jul 8, 2024

@3rd at least in my case it wasn't necessary to install neither Lua 5.1 nor magick locally, just adding the hererocks line in the lazy.lua file

@kevinm6
Copy link
Contributor

kevinm6 commented Jul 8, 2024

@kevinm6 no no, your contribution is great, so Lazy requires Lua 5.1 installed correct?

Yes, Lazy requires lua5.1 for the luarocks, due to its integration.

Then we can re-enable the rockspec and document that requirement as well, would that be the best?

I think should be fine if it's okay for you!
We could try to make it really clear in the README and probably is the better choice to use so far, since is reported from Lazy's developer

@joncrangle
Copy link

Thanks for everyone's help in this thread. Since I didn't have any other rocks installed in my config, I found that adding the hererocks rock to my lazy config wasn't enough. I needed to specify the magick dependency to initiate hererocks getting added as a plugin and the magick plugin bindings getting built. Sharing my setup (with a utility function I use to disable on Windows) in case anyone else runs into this and might find it helpful:

return {
  {
    '3rd/image.nvim',
    -- Utility function to disable on Windows system
    enabled = function()
      if vim.fn.has 'win32' == 1 then
        return false
      else
        return true
      end
    end,
    dependencies = {
      'leafo/magick',
    },
    opts = {
      -- ... image.nvim config
    },
  },
}

@kevinm6
Copy link
Contributor

kevinm6 commented Jul 11, 2024

@joncrangle I think you can improve it and use Lazy cond to do not load a plugin if the condition isn't satisfied.

return {
{
   '3rd/image.nvim',
    -- Disable on Windows system
    cond = function()
       return vim.fn.has 'win32' ~= 1 
    end,
    dependencies = {
       'leafo/magick',
    },
    opts = {
       -- image.nvim config
    }
}
}

@NeelMishra
Copy link

Any updates on this issue?

@3rd
Copy link
Owner

3rd commented Aug 28, 2024

Any updates on this issue?

What issue are you encountering?
You can opt out of lazy + rocks and install the magick rock manually.

@sankantsu
Copy link
Contributor

sankantsu commented Nov 1, 2024

Thank you for great plugin! I recently developed plugin utilizing image.nvim.
image.nvim saved a lot of work for correctly handling scrolling/redrawing tasks!

I did some research about luarocks and lazy.nvim, and considering if we can re-enable rockspec support for lazy.nvim (which was introduced in #185).
I tested locally and the rockspec seems working without error when I set opts.rocks.hererocks (as already mentioned in #191 (comment)).
(NOTE: Even if opts.rocks.hererocks is not set and global Lua 5.4 (not Lua 5.1), image.nvim and magick build seems working, but :checkhealth of lazy.nvim complains. So I think it is not recommended configuration).

I wrote PR in my fork which should resolve this issue.
sankantsu#1

If you would like, I will happily send PR to your repo.
Thank you!

@3rd
Copy link
Owner

3rd commented Nov 1, 2024

Hey, thanks awesome! Please open the PR <3

@3rd 3rd closed this as completed in #236 Nov 2, 2024
Repository owner locked and limited conversation to collaborators Nov 2, 2024
@3rd 3rd converted this issue into discussion #239 Nov 2, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →