-
Notifications
You must be signed in to change notification settings - Fork 40
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 asm to version 9.2 #141
Comments
Looks like we're on 9.3 internally - shall we jump straight there? |
yes - that would be great - want this downstream in google allocation instrumenter - see google/allocation-instrumenter#43 |
Hey, @ronshapiro. I'm on issue triage this week, and I'm a little confused by your remark about 9.3. AFAICT there's no such thing as 9.3 in maven central (https://mvnrepository.com/artifact/org.ow2.asm/asm), so I'm not sure how we could update to it. Could you explain this further for someone new to updating bazel-common dependencies? |
I'm not sure where I saw that to be honest. 9.2 is fine if thats the latest
…On Fri, Oct 29, 2021 at 10:58 PM Alan Malloy ***@***.***> wrote:
Hey, @ronshapiro <https://github.com/ronshapiro>. I'm on issue triage
this week, and I'm a little confused by your remark about 9.3. AFAICT
there's no such thing as 9.3 in maven central (
https://mvnrepository.com/artifact/org.ow2.asm/asm), so I'm not sure how
we could update to it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#141 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGBRXP6272ML6HU4XYR7SLUJL4E3ANCNFSM5GNWFR4Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
@amalloy any chance this update could be done? I see there is a new update script to help |
@ronshapiro @amalloy any chance you could help to progress this? thanks! |
Sorry I kept losing track of this. Just opened a commit internally |
Need to support java 17
The text was updated successfully, but these errors were encountered: