Skip to content

dep(proguard): Update proguard cache format #1734

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

Merged
merged 3 commits into from
Jul 17, 2025

Conversation

loewenheim
Copy link
Contributor

This updates the proguard dependency to 5.6.1 to get access to "synthesized" flags in the cache format
(see getsentry/rust-proguard#52).

Since the cache format has changed, this requires an incompatible bump of the proguard cache version.

It also adds a static version check for the cache version exposed by the proguard crate in analogy to symcaches and cficaches.

ref: #1720. ref: SYMBOLI-22.

@loewenheim loewenheim requested a review from a team as a code owner July 17, 2025 12:37
Copy link

linear bot commented Jul 17, 2025

@loewenheim loewenheim changed the title feat(proguard): Update proguard cache format dep(proguard): Update proguard cache format Jul 17, 2025
loewenheim added a commit that referenced this pull request Jul 17, 2025
Depends on #1734. This
adds a field `method_synthesized` to `JvmFrame` that denotes whether the
frame's method was synthesized by the compiler.
This updates the `proguard` dependency to 5.6.1 to get access to
"synthesized" flags in the cache format
(see getsentry/rust-proguard#52).

Since the cache format has changed, this requires an _incompatible_
bump of the proguard cache version.

It also adds a static version check for the cache version exposed by the
`proguard` crate in analogy to symcaches and cficaches.
@loewenheim loewenheim force-pushed the sebastian/proguard-update branch from 0f31478 to 29ee13d Compare July 17, 2025 12:48
loewenheim added a commit that referenced this pull request Jul 17, 2025
Depends on #1734. This
adds a field `method_synthesized` to `JvmFrame` that denotes whether the
frame's method was synthesized by the compiler.
filename: EditActivity
filename: R8$$SyntheticClass
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is an effect of the improved file name resolution from getsentry/rust-proguard#55. The previous filename was incorrect.

@loewenheim loewenheim merged commit 8971ce0 into master Jul 17, 2025
20 checks passed
@loewenheim loewenheim deleted the sebastian/proguard-update branch July 17, 2025 13:10
loewenheim added a commit that referenced this pull request Jul 17, 2025
Depends on #1734. This
adds a field `method_synthesized` to `JvmFrame` that denotes whether the
frame's method was synthesized by the compiler.
loewenheim added a commit that referenced this pull request Jul 17, 2025
Depends on #1734. This
adds a field `method_synthesized` to `JvmFrame` that denotes whether the
frame's method was synthesized by the compiler.

Closes #1720. Closes SYMBOLI-22.
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.

2 participants