Skip to content

feat: add strip_debug_symbols attribute#508

Merged
aherrmann merged 2 commits intoaherrmann:mainfrom
z8v:strip-symbols
Sep 8, 2025
Merged

feat: add strip_debug_symbols attribute#508
aherrmann merged 2 commits intoaherrmann:mainfrom
z8v:strip-symbols

Conversation

@z8v
Copy link
Contributor

@z8v z8v commented Sep 5, 2025

The attribute appends -fstrip to the zig compiler in order to remove debug symbols.

Removing debug symbols is a requirement in order to get reproducible builds on the release modes.

Copy link
Owner

@aherrmann aherrmann left a comment

Choose a reason for hiding this comment

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

Thank you, that looks great!

@aherrmann aherrmann changed the title feat: Add strip_debug_symbols attribute feat: add strip_debug_symbols attribute Sep 8, 2025
@aherrmann aherrmann enabled auto-merge September 8, 2025 07:09
@aherrmann aherrmann disabled auto-merge September 8, 2025 13:40
@aherrmann aherrmann enabled auto-merge September 8, 2025 13:41
@aherrmann aherrmann force-pushed the strip-symbols branch 2 times, most recently from 23190ad to 9118810 Compare September 8, 2025 15:38
aherrmann and others added 2 commits September 8, 2025 17:39
Secrets, such as the BuildBuddy API key, are not available on CI runs
on external PRs. Fall back to local execution in such cases.
The attribute appends `-fstrip` to the zig compiler in order to remove debug
symbols.

Removing debug symbols is a requirement in order to get reproducible
builds on the release modes.
@aherrmann aherrmann merged commit 0d14217 into aherrmann:main Sep 8, 2025
13 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