8000 compat: Forward top-level builtin options by mrmacete · Pull Request #1295 · frida/frida-core · GitHub
[go: up one dir, main page]

Skip to content

compat: Forward top-level builtin options - #1295

Merged
oleavr merged 1 commit into
mainfrom
compat-forward-builtin-options
Nov 12, 2026
Merged

oleavr merged 1 commit into
mainfrom
compat-forward-builtin-options

Conversation

@mrmacete
Copy link
Copy Markdown
Member

Meson keys top-level builtin options with subproject None, while top-level project options use "". We only matched the latter, so builtins such as strip never reached compat builds, which resulted in e.g. a stripped arm64 gadget slice despite --enable-symbols.

@mrmacete
mrmacete requested a review from oleavr November 12, 2026 12:13
@oleavr oleavr left a comment
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! 🙌

Comment thread compat/build.py Resolved
Meson keys top-level builtin options with subproject None, while
top-level project options use "". We only matched the latter, so
builtins such as strip never reached compat builds, which resulted in
e.g. a stripped arm64 gadget slice despite --enable-symbols.
@oleavr
oleavr force-pushed the compat-forward-builtin-options branch from 83f6ed7 to d62dce1 Compare November 12, 2026 13:42
@oleavr
oleavr merged commit d62dce1 into main Nov 12, 2026
30 of 36 checks passed
@oleavr
oleavr deleted the compat-forward-builtin-options branch November 12, 2026 13:45
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

0