Issue #54 resolved

headers of lua/luajit not included if first library (without 5.1) matches

Andrzej Giniewicz avatarAndrzej Giniewicz created an issue

When I was testing out LuaJIT support from rev 369 aka 370, I found out, that /usr/include/luajit-2.0 isn't included. After short looking, I found out, that on my system $ac_cv_search_lua_pcall is found inside luajit not luajit5.1, thus

if test "$ac_cv_search_lua_pcall" = "-lluajit5.1"; then

does not catch and -I/usr/include/luajit-2.0 isn't added, even that all is OK and configure passes.

Anyway, the test should be removed or changed to something like

if test ... -lluajit5.1; then ... else if test ... -lluajit; then ... fi fi

or something nicer probably. I think in all cases the headers are in same place, the /usr/include/luajit-2.0, not like in case of Lua inside /usr/include or /usr/include/lua5.1?

Comments (6)

  1. Andrzej Giniewicz

    yup, works with rev371. I switched the love-hg package I maintain in Arch Linux user repository to use LuaJIT 2.0 beta 5, so there will some more people to test it out. As far as I played with it (on x86_64 architecture) it works great.

    Anyway I think you can close that ticket :)

  2. Log in to comment
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.