Skip to content

Commit

Permalink
Fix computation of LUAJIT_INCLUDE_DIR
Browse files Browse the repository at this point in the history
I don't think the current way cmake uses to locate the correct lua.h
works.  Entries like "include/foo.h" cause cmake to look for a file
like "/usr/include/include/foo.h".  I.e., the include/ prefix for
all the entries is wrong as is adding "include" at the end.

This patch removes the prefix.  With it bcc finally compiles on
Fedora.

If this change causes a problem there is something seriously wrong
with cmake.
  • Loading branch information
drepper committed May 4, 2017
1 parent ce0981b commit 1af7a1c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmake/FindLuaJIT.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
FIND_PATH(LUAJIT_INCLUDE_DIR lua.h
HINTS
$ENV{LUAJIT_DIR}
PATH_SUFFIXES include/luajit-2.0 include/luajit2.0 include/luajit include/luajit-2.1 include
PATH_SUFFIXES luajit-2.0 luajit2.0 luajit luajit-2.1
PATHS
~/Library/Frameworks
/Library/Frameworks
Expand Down

0 comments on commit 1af7a1c

Please sign in to comment.