Skip to content
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

Fix app intents with Xcode 15.3+ #2418

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

Fix app intents with Xcode 15.3+ #2418

wants to merge 6 commits into from

Conversation

keith
Copy link
Member

@keith keith commented Feb 29, 2024

keith referenced this pull request Feb 29, 2024
keith referenced this pull request Feb 29, 2024
…pects all values to be statically allocated and resolved at compile time.

This pulls all of the module names found from SwiftInfo when gathering the swiftconstfiles. This appears to work in the test targets set up, which do lean on the multiple module strategy.

PiperOrigin-RevId: 589938532
keith referenced this pull request Feb 29, 2024
…e's app intents metadata processor tool allows for.

PiperOrigin-RevId: 590322476
keith referenced this pull request Feb 29, 2024
… Xcode 15.3 builds, based on the version string reported from the XcodeVersionConfig provider.

PiperOrigin-RevId: 601786735
BalestraPatrick added a commit to bazelbuild/rules_swift that referenced this pull request May 17, 2024
…e extraction. (#1170)

This cherry-picks support for producing const values produced via the
compiler flags `-emit-const-values-path` and
`-const-gather-protocols-file`. rules_apple will start using this
feature in bazelbuild/rules_apple#2418.

---------

Co-authored-by: Tony Allevato <[email protected]>
Co-authored-by: Keith Smiley <[email protected]>
@pennig
Copy link

pennig commented May 20, 2024

Has the time come at last?

@BalestraPatrick
Copy link
Member

Yes, we just need a new rules_swift release I believe depending on when we're happy with the status of the cherry-picks there

@BalestraPatrick BalestraPatrick force-pushed the ks/app-intents-fixes branch 2 times, most recently from 878e81b to 03a242c Compare May 28, 2024 17:24
BalestraPatrick added a commit that referenced this pull request Jun 3, 2024
…2465)

This was recently enabled in rules_swift (with
bazelbuild/rules_swift#1191) and it seems to be
required in the next `rules_swift` bump. I saw the failure initially
when testing a rules_swift bump in #2418 which was
[failing](https://buildkite.com/bazel/rules-apple-darwin/builds/9191#018fdd69-abfb-451a-a454-4b0771e26b77)
with the following error only in the Bazel 6.x configuration as expected
(since it's enabled by default on Bazel 7):
```
Error: No attribute 'aspect_hints' in attr. Make sure you declared a rule attribute with this name.
```
@BalestraPatrick BalestraPatrick force-pushed the ks/app-intents-fixes branch 2 times, most recently from 03a242c to ff75463 Compare June 3, 2024 14:37
nglevin and others added 6 commits June 11, 2024 16:10
…r logic.

PiperOrigin-RevId: 589151949
(cherry picked from commit 41d5bb7)
…pects all values to be statically allocated and resolved at compile time.

This pulls all of the module names found from SwiftInfo when gathering the swiftconstfiles. This appears to work in the test targets set up, which do lean on the multiple module strategy.

PiperOrigin-RevId: 589938532
(cherry picked from commit 5668eb4)
…e's app intents metadata processor tool allows for.

PiperOrigin-RevId: 590322476
(cherry picked from commit 21b435b)
… Xcode 15.3 builds, based on the version string reported from the XcodeVersionConfig provider.

PiperOrigin-RevId: 601786735
(cherry picked from commit ea240b8)
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.

None yet

4 participants