search

libvips / lua-vips

Lua binding for the libvips image processing library
MIT License
125 stars 10 forks source link

Update rockspec and README, remove Makefile #75

Closed rolandlo closed 5 months ago

rolandlo commented 5 months ago

Updated the rockspec (rockspec-format, labels, dependencies, test, copy-directories).

luaffi-tkl is now a dependency of lua-vips. LuaJIT users will have to exhibit this module as provided by the VM by editing the config-5.1.lua or by running luarocks config --lua-version=5.1 rocks_provided.luaffi-tkl 2.1-1. Ideally this would be listed in https://github.com/luarocks/luarocks/blob/c16bdbc4cbb98c560fb7e8d4895d4d84afca2eb9/src/luarocks/util.lua#L599-L604, so LuaJIT user's wouldn't have to do it manually, but that would only work for those with an up to date luarocks then.

When running luarocks test in the lua-vips directory the test-dependency busted is automatically installed, if it is not found.

Updated the README (where lua-vips works and where it is tested, how to install on Windows MinGW-w64).

Removed the Makefile, as it is not needed any longer. @RiskoZoSlovenska What do you think about it?

RiskoZoSlovenska commented 5 months ago

Looks good to me; nice work!

rolandlo commented 5 months ago

Thanks for the review @RiskoZoSlovenska