-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Update desugar_jdk_libs #11821
Update desugar_jdk_libs #11821
Conversation
eacf242
to
cb4c8bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most likely tools/android/desugar.sh
needs to be updated to account for changes/additions. Please have @donaldchai review this change as well.
This is new to me, but based on a964125 it seems like you need to add any new classes to |
oh.. this is a hard one to complete. I took a look at what Android Studio source code is specifying but it seems pretty much what you have right now here at HEAD. |
taken latest from https://github.com/google/desugar_jdk_libs
ping. |
@aiuto I don't mind waiting, but since this is a pretty small PR anyway I think it's okay as is. Your call. |
I'm on triage duty this week, so I'll merge it. But first the tests have to pass. |
See #12811. That does this in the new style. The tests should work now too, since I updated the mirror. |
Sorry this took so long. |
taken latest from https://github.com/google/desugar_jdk_libs