Skip to content

feat: support zig_library as cc_* dependency#445

Merged
aherrmann merged 10 commits intomainfrom
cc-info-lib
Mar 18, 2025
Merged

feat: support zig_library as cc_* dependency#445
aherrmann merged 10 commits intomainfrom
cc-info-lib

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