Skip to content

feat: support zig_library as cc_* dependency #445

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.

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

Merged
merged 10 commits into from
Mar 18, 2025
Merged

feat: support zig_library as cc_* dependency #445

merged 10 commits into from
Mar 18, 2025

Conversation

aherrmann
Copy link
Owner

Similar to #443.

Note, Zig does not include compiler runtime symbols in static archives by default.
Users may need to set compiler_runtime = "include" on zig_library targets that they wish to use as inputs to CC targets.

  • e2e test for zig_library as a cc_ dependency*
  • feat: emit CcInfo for zig_library to provide static library
  • e2e test for transitive zig_library cc dependency
  • Analysis test for zig_library
  • analysis test for transitive zig_library
  • load cc_library
  • Update generated files
  • Update Zig library documentation

@aherrmann
Copy link
Owner Author

Required for #14 and #428.

It was only needed for the debug symbols.
@aherrmann aherrmann enabled auto-merge March 17, 2025 05:32
Fails with
```
ld: archive member 'bazel-out/darwin_arm64-fastbuild-ST-8e42560474ff/bin/cc-dependencies/static-library/libadd.a.o' not a mach-o file in
'/private/var/tmp/_bazel_runner/5ca2a14b857dddeda41464da6fe32452/execroot/_main/bazel-out/darwin_arm64-fastbuild-ST-8e42560474ff/bin/cc-dependencies/static-library/libmul.a'
```
@aherrmann aherrmann merged commit dc8e0d5 into main Mar 18, 2025
63 checks passed
@aherrmann aherrmann deleted the cc-info-lib branch March 18, 2025 07:47
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.

1 participant