Skip to content

Fix Lua Detection on ci_new#3531

Merged
airween merged 4 commits intoowasp-modsecurity:v3/masterfrom
Easton97-Jens:v3/master_update_workflows
Apr 1, 2026
Merged

Fix Lua Detection on ci_new#3531
airween merged 4 commits intoowasp-modsecurity:v3/masterfrom
Easton97-Jens:v3/master_update_workflows

Conversation

@Easton97-Jens
Copy link
Copy Markdown
Contributor

what

  • Fixes Lua detection in ci_new.
  • Ensures Lua is correctly detected and handled within the CI pipeline.
  • Resolves issues caused by incorrect or missing Lua detection.

why

  • The previous Lua detection in ci_new was faulty or incomplete.
  • As a result, CI jobs involving Lua could be misexecuted or skipped.
  • This fix improves the stability and reliability of the CI processes.

references

EN:

  • Fix for Lua detection in ci_new

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the ci_new GitHub Actions workflow to more reliably detect and install the latest available libluaX.Y-dev package on Ubuntu runners.

Changes:

  • Switch Lua dev package discovery to apt-cache pkgnames | grep -E ... and select the newest version via sort -V.
  • Add a guard to fail early if a “best” Lua package cannot be determined.
  • Add a diagnostic step to print Lua-related installation details in CI logs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 1, 2026

@airween
Copy link
Copy Markdown
Member

airween commented Apr 1, 2026

Thanks, @Easton97-Jens!

@airween airween merged commit e5d00df into owasp-modsecurity:v3/master Apr 1, 2026
98 checks passed
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

Successfully merging this pull request may close these issues.

3 participants