Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Enable support for AEGs#1585

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
brentleyjones wants to merge1 commit intomaster
base:master
Choose a base branch
Loading
frombj/enable-aegs
Draft

Conversation

@brentleyjones
Copy link
Collaborator

(cherry picked from commit8a4310e)

brentleyjones referenced this pull requestOct 7, 2025
PiperOrigin-RevId: 750219409
brentleyjones referenced this pull requestOct 7, 2025
PiperOrigin-RevId: 750306755
brentleyjones referenced this pull requestOct 7, 2025
PiperOrigin-RevId: 750985667
(cherry picked from commit8a4310e)Signed-off-by: Brentley Jones <github@brentleyjones.com>
@brentleyjonesbrentleyjones changed the titleEnable AEGsEnable support for AEGsOct 7, 2025
@brentleyjones
Copy link
CollaboratorAuthor

brentleyjones commentedOct 7, 2025
edited
Loading

We will need to applybazelbuild/rules_apple@3da7b31 inrules_apple before landing this. as well as some other toolchains/AEG stuff in general.

@brentleyjones
Copy link
CollaboratorAuthor

Current failures are unrelated to PR, but I don't feel safe merging this while they are there.

@brentleyjones
Copy link
CollaboratorAuthor

Also looks like we need to set toolchains for the proto stuff. Upstream definesswift_proto_toolchain, and I think we have to do something similar. cc:@AttilaTheFun.

@AttilaTheFun
Copy link
Contributor

@brentleyjones I'm not familiar with Automatic Extension Groups but you're welcome to modify the swift_proto_library rule to support them.

However, longer term I was thinking it would be more maintainable to refactor them into a macro with a source-generating rule which passes the generated sources into a regular swift_library.

This is the approach I took at Snap for our new Swift dependency injection system and it has been working well for us so far.

There are a couple of tradeoffs to this approach, though, and it would be a source-breaking change so we would have to retain the old behavior with some kind of legacy_mode flag.

@brentleyjones
Copy link
CollaboratorAuthor

Ignore the AEGs portion, I was more calling out the toolchain-ification of the proto rules inupstream and how that probably applies to us as well (and in general using toolchains is required for AEGs to work).

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@mattrobmattrobmattrobmattrobmattrobmattrob approved these changes

@aaronskyaaronskyAwaiting requested review from aaronskyaaronsky will be requested when the pull request is marked ready for reviewaaronsky is a code owner

@adincebicadincebicAwaiting requested review from adincebicadincebic will be requested when the pull request is marked ready for reviewadincebic is a code owner

@keithkeithAwaiting requested review from keithkeith will be requested when the pull request is marked ready for reviewkeith is a code owner

@luispadronluispadronAwaiting requested review from luispadronluispadron will be requested when the pull request is marked ready for reviewluispadron is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@brentleyjones@AttilaTheFun@mattrobmattrob@allevato

[8]ページ先頭

©2009-2025 Movatter.jp