Skip to content

Conversation

@h6ah4i
Copy link
Contributor

@h6ah4i h6ah4i commented Nov 8, 2025

This change fixes a compilation error that occurs when android.nonTransitiveRClass=true is enabled in gradle.properties.

@icbaker icbaker self-assigned this Nov 10, 2025
@icbaker icbaker self-requested a review November 10, 2025 10:01
@icbaker
Copy link
Collaborator

icbaker commented Nov 10, 2025

Can I just check that this only causes problems if you are building the library locally?

I tried to reproduce the problem with a project depending on implementation("androidx.media3:media3-session:1.9.0-alpha01") and didn't see an issue.

@icbaker
Copy link
Collaborator

icbaker commented Nov 10, 2025

Actually after looking more closely at this code, these are private resource IDs we shouldn't be referencing - I removed the reference in 420caf5.

So I'll close this PR as obsolete.

@icbaker icbaker closed this Nov 10, 2025
copybara-service bot pushed a commit that referenced this pull request Nov 14, 2025
And fix the violations of `nonTransitiveRClass`

Tested with `./gradlew clean lint testReleaseUnitTest`

Follow-up to Issue: #2883

PiperOrigin-RevId: 832250353
@icbaker
Copy link
Collaborator

icbaker commented Nov 14, 2025

I have also removed this setting from our gradle.properties file so we shouldn't introduce any similar problems in future: b770edf

marcbaechinger pushed a commit that referenced this pull request Nov 24, 2025
And fix the violations of `nonTransitiveRClass`

Tested with `./gradlew clean lint testReleaseUnitTest`

Follow-up to Issue: #2883

PiperOrigin-RevId: 832250353
(cherry picked from commit b770edf)
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