Skip to content

Conversation

@keith
Copy link
Member

@keith keith commented Jul 11, 2023

@keith keith marked this pull request as draft July 11, 2023 19:08
@keith keith force-pushed the ks/add-initial-visionos-support branch from fa7fb2a to 3c70f29 Compare July 11, 2023 19:14
Base automatically changed from ks/add-stripped-down-apple_verification_test to master July 11, 2023 21:46
@keith keith force-pushed the ks/add-initial-visionos-support branch from d894fc5 to a56e41f Compare July 14, 2023 00:30
@keith keith force-pushed the ks/add-initial-visionos-support branch from a56e41f to e922bd5 Compare August 10, 2023 17:52
@brentleyjones
Copy link
Collaborator

brentleyjones commented Aug 10, 2023

Depends on bazelbuild/bazel#18905

Does that mean when we merge this that we drop support for anything before Bazel 7?

(Though I think we can conditionalize our code to work around that.)

@keith
Copy link
Member Author

keith commented Aug 10, 2023

Well if you backport that to 6.x maybe we'd only have to worry about 5.x (assuming it's reasonable to only support the newest 6.x). otherwise I do think we could conditionalize the setup that uses the platforms somehow which i think would be all we'd have to do

@keith keith force-pushed the ks/add-initial-visionos-support branch from e47c706 to e32820e Compare August 10, 2023 18:41
@keith keith force-pushed the ks/add-initial-visionos-support branch from e32820e to bbf416f Compare August 12, 2023 00:58
@keith keith marked this pull request as ready for review August 12, 2023 01:32
@keith keith requested a review from brentleyjones August 12, 2023 01:32
@keith keith merged commit 4a86f2f into master Aug 12, 2023
@keith keith deleted the ks/add-initial-visionos-support branch August 12, 2023 01:44
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.

4 participants