Skip to content

fix: yajl detection for source installations (#3457) - #3458

Merged
airween merged 1 commit into
owasp-modsecurity:v2/masterfrom
weida:fix-build-find-yajl
Nov 2, 2025
Merged

fix: yajl detection for source installations (#3457)#3458
airween merged 1 commit into
owasp-modsecurity:v2/masterfrom
weida:fix-build-find-yajl

Conversation

@weida

@weida weida commented Nov 1, 2025

Copy link
Copy Markdown

what

  • Modified find_yajl.m4 to correctly detect yajl when installed from source.
  • Added a check for the lib subdirectory in addition to the top-level path.
  • Ensures that ModSecurity compilation succeeds when --with-yajl is specified.

why

  • The build failed to detect yajl 2.1.0 compiled from source, even when the correct path was provided.

references

@sonarqubecloud

sonarqubecloud Bot commented Nov 1, 2025

Copy link
Copy Markdown

@airween

airween commented Nov 2, 2025

Copy link
Copy Markdown
Member

Thanks for your patch, @weida!

@airween
airween merged commit 6c78df4 into owasp-modsecurity:v2/master Nov 2, 2025
101 of 114 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.

2 participants