Skip to content

lqc failed to run without installed ffi module #28

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

Open
ligurio opened this issue Apr 8, 2021 · 4 comments
Open

lqc failed to run without installed ffi module #28

ligurio opened this issue Apr 8, 2021 · 4 comments

Comments

@ligurio
Copy link
Contributor

ligurio commented Apr 8, 2021

README says that FFI module is optional, unfortunately lqc failed to run without it:

sergeyb@pony:~/sources/MRG/topology$ lqc 
Random seed = 1617869614
/usr/bin/lua5.1: ./.rocks/share/tarantool/checks.lua:193: module 'ffi' not found:No LuaRocks module found for ffi
        no field package.preload['ffi']
        no file '/root/.luarocks/share/lua/5.1/ffi.lua'
        no file '/root/.luarocks/share/lua/5.1/ffi/init.lua'
        no file '/usr/local/share/lua/5.1/ffi.lua'
        no file '/usr/local/share/lua/5.1/ffi/init.lua'
        no file './ffi.lua'
        no file '/usr/local/lib/lua/5.1/ffi.lua'
        no file '/usr/local/lib/lua/5.1/ffi/init.lua'
        no file '/usr/share/lua/5.1/ffi.lua'
        no file '/usr/share/lua/5.1/ffi/init.lua'
        no file '/home/sergeyb/.luarocks/share/lua/5.1/ffi.lua'
        no file '/home/sergeyb/.luarocks/share/lua/5.1/ffi/init.lua'
        no file '/root/.luarocks/lib/lua/5.1/ffi.so'
        no file '/usr/local/lib/lua/5.1/ffi.so'
        no file './ffi.so'
        no file '/usr/lib/x86_64-linux-gnu/lua/5.1/ffi.so'
        no file '/usr/lib/lua/5.1/ffi.so'
        no file '/usr/local/lib/lua/5.1/loadall.so'
        no file '/home/sergeyb/.luarocks/lib/lua/5.1/ffi.so'

lqc was successfully installed using LuaRocks: luarocks --local install lua-quickcheck

Version: https://luarocks.org/lua-quickcheck-0.2-4.src.rock

@luc-tielen
Copy link
Owner

I think it used to be optional, but README is out of date?
I guess it's fine to add it to the set of dependencies, quite a lot of other libs probably use ffi anyway.

@ligurio
Copy link
Contributor Author

ligurio commented Apr 8, 2021

I think it would be fine.
Right now rockspec missed this dependence.

@ligurio
Copy link
Contributor Author

ligurio commented Apr 8, 2021

BTW ffi depency is required only in case of using PUC Rio Lua, because LuaJIT has builtin ffi module [1].

  1. https://luajit.org/ext_ffi.html

@luc-tielen
Copy link
Owner

Yes, that's what is configured in CI also. It only installs for non-JIT Luas.

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

No branches or pull requests

2 participants