Jonathan Schear

Results 25 comments of Jonathan Schear

Amazing! I saw Keith's post on [rule extensions](https://www.smileykeith.com/2025/10/31/bazel-rule-extensions/) and was thinking that might be applicable here, neat. Definitely nice to not pollute the build graph with unnecessary targets. I was...

I guess [rules_swift_package_manager also reimplements module map generation,](https://github.com/cgrindel/rules_swift_package_manager/blob/main/swiftpkg/internal/module_maps.bzl) it isn't the end of the world to duplicate it again 😄

I'd like to contribute here. I believe all we need to do in the BUILD file generation side is add `swift.suppress_warnings` to `swift_library`'s `features` attribute. The more involved part is...

Great, I'll get to work on adding `swift_features` to `from_package` (for global configuration) and `configure_package` (for per-package configuration). I also just realized https://github.com/bazelbuild/rules_swift/blob/master/doc/rules.md#swift_package_configuration exists, does anyone know if that can...

The version of `rules_ios` the example uses isn't compatible with more recent releases of `rules_swift` & `rules_apple`. I'll get that building again, sorry for the trouble.

Never got around to this, and now the `rules_ios` example has been removed. Please see https://github.com/bazel-ios/rules_ios/discussions/952 for notes on migrating away from `rules_ios`.

Oops, missed some tests, will take a look.

I can repro the same issue when a `rules_ios` `apple_framework` depends directly on a `swift_library`, without involving `rules_swift_package_manager` at all: https://github.com/MobileNativeFoundation/rules_xcodeproj/compare/main...jschear:rules_xcodeproj:js/rules_ios_swift_library_merging. I can see that there are three `mergeable_infos` for...

Thanks for taking a look at this. > This seems like a rules_ios bug then? The new swift_library should be a dep of MixedAnswer_swift and/or MixedAnswer_objc, but not the packaging...

Ah, rules_xcodeproj does actually consider `transitive_deps` of `apple_framework_packaging` when merging targets: https://github.com/MobileNativeFoundation/rules_xcodeproj/pull/2750 I don't have full context on why `transitive_deps` exists in the first place, but I'm going to check...