Skip to content

feat: support header generation (-femit-h) for Zig <=0.13.0 #447

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 26 commits into from
Mar 24, 2025
Merged

feat: support header generation (-femit-h) for Zig <=0.13.0 #447

merged 26 commits into from
Mar 24, 2025

Conversation

aherrmann
Copy link
Owner

Adds a generate_header attribute to the library rules. When enabled these rules will instruct Zig to emit a header and expose it in the "header" output group and the CcInfo provider.

Note, this feature was disabled as of Zig 0.14.0, see ziglang/zig#9698.

  • Start e2e test for header generation
  • Test generate_header attribute
  • Add generate_header attribute
  • Refactor output groups
  • e2e test for header output group
  • Generate header and expose in output group
  • header generation requires -lc
  • fix assert_outputs test
  • C generated header include test
  • Expose generated header in CcInfo
  • Forward zig.h together with the generated header
  • move zig.h into transitive CcInfos
  • zig_configure: forward output groups
  • Configure ReleaseSmall for header generation tests
  • mark tests as small
  • Add simple header generation test targets
  • analysis test that no header is generated by default
  • analysis test for header generation
  • fix shared library analysis test
  • Skip header generation with Zig 0.14.0
  • document the header output group and CcInfo
  • Document zig_header target
  • update generated files

@aherrmann
Copy link
Owner Author

Closes #428 and #14.

@aherrmann aherrmann enabled auto-merge March 23, 2025 16:41
@aherrmann aherrmann disabled auto-merge March 23, 2025 16:46
@aherrmann aherrmann enabled auto-merge March 23, 2025 16:52
@aherrmann aherrmann force-pushed the emit-h branch 2 times, most recently from 974291a to a14f163 Compare March 23, 2025 18:55
aherrmann added 24 commits March 24, 2025 08:45
@aherrmann
Start e2e test for header generation
61d496d
@aherrmann
Test generate_header attribute
0247c2c
@aherrmann
Add generate_header attribute
02d2a78
@aherrmann
Refactor output groups
8635571 
Enable future output groups from the build stage.
@aherrmann
e2e test for header output group
46e4e86
@aherrmann
Generate header and expose in output group
4b77dcf
@aherrmann
header generation requires -lc
6c68772
@aherrmann
fix assert_outputs test
420fae5
@aherrmann
C generated header include test
8f4930d
@aherrmann
Expose generated header in CcInfo
5112214
@aherrmann
Forward zig.h together with the generated header
37a30bd
@aherrmann
move zig.h into transitive CcInfos
4b01a37
@aherrmann
zig_configure: forward output groups
8c1b361
@aherrmann
Configure ReleaseSmall for header generation tests
74f5bd5
@aherrmann
mark tests as small
b70c259
@aherrmann
Add simple header generation test targets
952ab56
@aherrmann
analysis test that no header is generated by default
eb25e03
@aherrmann
analysis test for header generation
ada3566
@aherrmann
fix shared library analysis test
50865fc
@aherrmann
Skip header generation with Zig 0.14.0
54d94e4
@aherrmann
document the header output group and CcInfo
d8104d5
@aherrmann
Document zig_header target
4402cfb
@aherrmann
update generated files
0c9ac88
@aherrmann
Disable header generation for Zig 0.12.0
7fdfd70 
The Zig compiler segfaults.
@aherrmann
Disable header generation on MacOS
fb2806a
@aherrmann
tag header generation segfaults as manual
4179d69 
Header generation fails on debug builds. Tag the unconfigured targets as
manual to avoid these failures. Targets configured to a ReleaseSmall
configuration succeed.
@aherrmann aherrmann merged commit 11442ac into main Mar 24, 2025
55 checks passed
@aherrmann aherrmann deleted the emit-h branch March 24, 2025 07:58
@aherrmann aherrmann mentioned this pull request Mar 24, 2025
Closed
@aherrmann aherrmann mentioned this pull request Aug 13, 2025
Open
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
@aherrmann