search

cgrindel / rules_swift_package_manager

Collection of utilities and Bazel rules to aid in the development and maintenance of Swift repositories using Bazel.
Apache License 2.0
68 stars 22 forks source link

fix: Use `swift_interop_hint` instead of `swift_module` tag #1107

Open brentleyjones opened 2 weeks ago

brentleyjones commented 2 weeks ago

Needed for rules_swift 2.0. Only works with rules_swift 2.0+.

brentleyjones commented 2 weeks ago

I can't think of a good way to determine if we are depending on rules_swift 2.0 or not, so right now this is unconditional. Ideally we would only make this change if transitively depending on rules_swift 2.0.

edit: Looks like with bzlmod we would be able to determine the version of rules_swift with https://bazel.build/rules/lib/builtins/bazel_module.html#version. Not much we can do in WORKSPACE, afaik.

brentleyjones commented 2 weeks ago

Once rules_swift 2.0 is released I'll change the DNM commit to a proper bump, then we can merge this.