Skip to content

lua: support luzer-based testing#14610

Open
ligurio wants to merge 1 commit intogoogle:masterfrom
ligurio:ligurio/test-lua-with-luzer
Open

lua: support luzer-based testing#14610
ligurio wants to merge 1 commit intogoogle:masterfrom
ligurio:ligurio/test-lua-with-luzer

Conversation

@ligurio
Copy link
Copy Markdown
Contributor

@ligurio ligurio commented Dec 19, 2025 •
edited
Loading

Depends on #13929
Depends on ligurio/luzer#74
Depends on ligurio/lunapark#179
Depends on ligurio/luzer#82
Related to ligurio/lunapark#180
Related to ligurio/luzer#83

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 19, 2025

ligurio is either the primary contact or is in the CCs list of projects/lua.
ligurio has previously contributed to projects/lua. The previous PR was #12685
ligurio is integrating a new project:
- Main repo: https://github.com/ligurio/luzer
- Criticality score: 0.23736

@ligurio ligurio mentioned this pull request Dec 24, 2025
Open
26 tasks
@ligurio ligurio mentioned this pull request Jan 7, 2026
Merged
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from 6bca742 to 648bcfa Compare January 17, 2026 15:47
@ligurio ligurio marked this pull request as draft January 30, 2026 07:34
ligurio added a commit to ligurio/luzer that referenced this pull request Mar 27, 2026
@ligurio
rockspec: define directories with headers and lib
09a22e4 
Sometimes one need to compile a luzer module with a non-system Lua
library and headers, for which the LUA_LIBRARIES and
LUA_INCLUDE_DIR CMake options are used. The patch adds variables
for the rockspec to set these options via environment variables:
LUA_LIBRARIES and LUA_INCLUDE_DIR.

Needed for google/oss-fuzz#14610
@ligurio ligurio mentioned this pull request Mar 27, 2026
Merged
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from 648bcfa to b14c354 Compare March 27, 2026 12:26
ligurio added a commit to ligurio/lunapark that referenced this pull request Mar 27, 2026
@ligurio
cmake: introduce option ENABLE_LIBFUZZER_STATIC_LINKAGE
10ec8ad 
The patch itnroduces a CMake option ENABLE_LIBFUZZER_STATIC_LINKAGE
that enables static linkage wituh libFuzzer. By default linkage
is disabled.

Needed for google/oss-fuzz#14610
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch 3 times, most recently from 5b5ddaa to ff72ffc Compare March 27, 2026 14:35
@ligurio ligurio marked this pull request as ready for review March 27, 2026 14:40
ligurio added a commit to ligurio/luzer that referenced this pull request Mar 27, 2026
@ligurio
rockspec: define directories with headers and lib
6c4c5ed 
Sometimes one need to compile a luzer module with a non-system Lua
library and headers, for which the LUA_LIBRARIES and
LUA_INCLUDE_DIR CMake options are used. The patch adds variables
for the rockspec to set these options via environment variables:
LUA_LIBRARIES and LUA_INCLUDE_DIR.

Needed for google/oss-fuzz#14610
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch 4 times, most recently from 48eff90 to 5aca8b2 Compare March 27, 2026 15:42
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 6, 2026
@ligurio
rockspec: propagate CMake options
a752226 
Sometimes one need to build a luzer module using rockspec with
a non-system Lua library and headers, for which the CMake options
LUA_LIBRARIES and LUA_INCLUDE_DIR are used. The patch adds
variables for the rockspec to set these options via environment
variables: CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR,
ENABLE_LUAJIT and LUAJIT_FRIENDLY_MODE.

luarocks install --lua-version 5.1 --verbose --tree=lua_modules \
  luzer-scm-1.rockspec ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 6, 2026
@ligurio
rockspec: propagate CMake options
9d68fcd 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers, for which the CMake options
LUA_LIBRARIES and LUA_INCLUDE_DIR are used. The patch adds
variables for the rockspec to set these options via environment
variables: CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR,
ENABLE_LUAJIT and LUAJIT_FRIENDLY_MODE.

luarocks install --lua-version 5.1 --verbose --tree=lua_modules \
  luzer-scm-1.rockspec ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 6, 2026
@ligurio
rockspec: propagate CMake options
f0be0ac 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers, for which the CMake options
LUA_LIBRARIES and LUA_INCLUDE_DIR are used. The patch adds
variables for the rockspec to set these options via environment
variables: CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR,
ENABLE_LUAJIT and LUAJIT_FRIENDLY_MODE.

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
008c2d7 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR, ENABLE_LUAJIT and
LUAJIT_FRIENDLY_MODE:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames a CMake variables used for setting custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
96c0f92 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR, ENABLE_LUAJIT and
LUAJIT_FRIENDLY_MODE:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames a CMake variables used for setting custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
34d1cee 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR, ENABLE_LUAJIT,
LUAJIT_FRIENDLY_MODE and OSS_FUZZ:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames a CMake variables used for setting custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
8111c89 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
ENABLE_LUAJIT, LUAJIT_FRIENDLY_MODE and OSS_FUZZ:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames CMake variables used for setting a custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

After renaming these options can be specified for luarocks.

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
7c4e79f 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
ENABLE_LUAJIT, LUAJIT_FRIENDLY_MODE and OSS_FUZZ:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames CMake variables used for setting a custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

After renaming these options can be specified for luarocks.

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
6f9c893 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
ENABLE_LUAJIT, LUAJIT_FRIENDLY_MODE and OSS_FUZZ:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames CMake variables used for setting a custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

After renaming these options can be specified for luarocks.
The patch also sorts variables in alphabetical order.

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
2a58b8a 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
ENABLE_LUAJIT, LUAJIT_FRIENDLY_MODE and OSS_FUZZ:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames CMake variables used for setting a custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

After renaming these options can be specified for luarocks.
The patch also sorts variables in alphabetical order.

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/lunapark that referenced this pull request Apr 7, 2026
@ligurio
cmake: introduce option ENABLE_LIBFUZZER_STATIC_LINKAGE
018efe6 
The patch itnroduces a CMake option ENABLE_LIBFUZZER_STATIC_LINKAGE
that enables static linkage wituh libFuzzer. By default linkage
is disabled.

Needed for google/oss-fuzz#14610
@ligurio ligurio mentioned this pull request Apr 7, 2026
Merged
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: fix propagating variables
927b671 
The previous patch is not set LUA_INCLUDE_DIR when custom Lua
include directory is used. The patch fixes that. Also, rockspec
has missed variables CMAKE_LUA_LIBRARIES and CMAKE_LUA_INCLUDE_DIR.
The patch adds these variables.

Follows up the commit 2a58b8a
("rockspec: propagate CMake options").

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/lunapark that referenced this pull request Apr 7, 2026
@ligurio
cmake: introduce DISABLE_LIBFUZZER_STATIC_LINKAGE
18ac78c 
The patch itnroduces a CMake option DISABLE_LIBFUZZER_STATIC_LINKAGE
that disables static linkage wituh libFuzzer. By default linkage
is enabled.

Needed for google/oss-fuzz#14610
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from 5aca8b2 to 73f555c Compare April 7, 2026 13:40
@ligurio ligurio mentioned this pull request Apr 7, 2026
Merged
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from 73f555c to 0e03ce8 Compare April 7, 2026 14:03
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: fix propagating variables
1d91c24 
The previous patch is not set LUA_INCLUDE_DIR when custom Lua
include directory is used. The patch fixes that. Also, rockspec
has missed variables CMAKE_LUA_LIBRARIES and CMAKE_LUA_INCLUDE_DIR.
The patch adds these variables.

Follows up the commit 2a58b8a
("rockspec: propagate CMake options").

Needed for google/oss-fuzz#14610
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from d24af8f to c169cce Compare April 7, 2026 14:22
ligurio added a commit to ligurio/lunapark that referenced this pull request Apr 7, 2026
@ligurio
cmake: introduce DISABLE_LIBFUZZER_STATIC_LINKAGE
944f891 
The patch itnroduces a CMake option DISABLE_LIBFUZZER_STATIC_LINKAGE
that disables static linkage wituh libFuzzer. By default linkage
is enabled.

Needed for google/oss-fuzz#14610
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch 2 times, most recently from ad3c460 to e4accfb Compare April 7, 2026 14:51
@ligurio
lua: support luzer-based testing
323e75d 
sudo python infra/helper.py build_fuzzers lua
sudo python infra/helper.py check_build lua utf8_char_test

Related to ligurio/lunapark#180
Related to ligurio/luzer#83
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from e4accfb to 323e75d Compare April 7, 2026 15:25
@ligurio
Copy link
Copy Markdown
Contributor Author

ligurio commented Apr 7, 2026

@DavidKorczynski ok to merge?

@ligurio
Copy link
Copy Markdown
Contributor Author

ligurio commented Apr 9, 2026

@jonathanmetzman ok to merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ligurio