Skip to content
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

bug(YouTube - Shorts components): Patch failing during building with certain patch selection #2581

Closed
8 tasks done
ILoveOpenSourceApplications opened this issue Dec 21, 2024 · 4 comments
Labels
Bug report Something isn't working

Comments

@ILoveOpenSourceApplications

Type

Error while patching

Tools used

RVX Manager

Application

YouTube v19.44.39

Bug description

During patching the process fails at Shorts components. The following is the logs from the manager.

- Device Info
RVX Manager: 1.23.5
Model: SM-S911B
Android version: 14
Supported architectures: arm64-v8a, armeabi-v7a, armeabi
Root permissions: No

- Patch Info
App: com.google.android.youtube v19.44.39 (Suggested: 19.44.39)
Patches version: v5.1.1
Patches added: Default
Patches removed: Alternative thumbnails, Bypass image region restrictions, Hide feed flyout menu, Disable auto audio tracks, Remove viewer discretion dialog, Hook download actions, Layout switch, Enable gradient loading screen, Miniplayer, Hook YouTube Music actions, Navigation bar components, Disable splash animation, Spoof app version, Change start page, Toolbar components, Custom Shorts action buttons, Custom branding icon for YouTube, Custom branding name for YouTube, Custom double tap length, Translations for YouTube, Visual preferences icons for YouTube, Enable OPUS codec, Enable debug logging, Disable QUIC protocol, Change share sheet, Watch history, Hide action buttons, Hide player buttons, Change player flyout menu toggles, Disable haptic feedback, Overlay buttons, Swipe controls, Spoof streaming data, Return YouTube Dislike, Settings for YouTube, SponsorBlock
Default patch options changed: None

- Settings
Allow changing patch selection: true
Version compatibility check: false
Show universal patches: false
Remove unused library: true
Use pre-release: false
Patches source: inotia00/revanced-patches

- Logs
Reading APK
Decoding app manifest
Loading patches
Deleting existing temporary files directory
Decoding resources
Initializing lookup maps
Executing patches
Applied 19 patches
Shorts components failed: app.revanced.patcher.patch.PatchException: The patch "Shorts components" depends on "BytecodePatch (shortsCustomActionsPatch)", which raised an exception:
app.revanced.patcher.patch.PatchException: com.android.tools.smali.dexlib2.builder.instruction.BuilderInstruction11n cannot be cast to com.android.tools.smali.dexlib2.iface.instruction.ReferenceInstruction
	at app.revanced.patcher.Patcher$invoke$1.invokeSuspend$execute(SourceFile:90)
	at app.revanced.patcher.Patcher$invoke$1.invokeSuspend$execute(SourceFile:72)
	at app.revanced.patcher.Patcher$invoke$1.invokeSuspend(SourceFile:109)
	at app.revanced.patcher.Patcher$invoke$1.invoke(SourceFile:0)
	at app.revanced.patcher.Patcher$invoke$1.invoke(SourceFile:0)
	at kotlinx.coroutines.flow.SafeFlow.collectSafely(SourceFile:57)
	at kotlinx.coroutines.flow.AbstractFlow.collect(SourceFile:226)
	at app.revanced.manager.flutter.MainActivity$runPatcher$1$patcherResult$1$1.invokeSuspend(SourceFile:323)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(SourceFile:33)
	at kotlinx.coroutines.DispatchedTask.run(SourceFile:101)
	at kotlinx.coroutines.EventLoopImplBase.processNextEvent(SourceFile:263)
	at kotlinx.coroutines.BlockingCoroutine.joinBlocking(SourceFile:95)
	at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking(SourceFile:69)
	at kotlinx.coroutines.BuildersKt.runBlocking(SourceFile:1)
	at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking$default(SourceFile:47)
	at kotlinx.coroutines.BuildersKt.runBlocking$default(SourceFile:1)
	at app.revanced.manager.flutter.MainActivity.runPatcher$lambda$34(SourceFile:317)
	at app.revanced.manager.flutter.MainActivity.$r8$lambda$Oku38Rh6VOg9CiulxEElyw3q4GI(SourceFile:0)
	at app.revanced.manager.flutter.MainActivity$$ExternalSyntheticLambda0.run(SourceFile:0)
	at java.lang.Thread.run(Thread.java:1012)
Caused by: java.lang.ClassCastException: com.android.tools.smali.dexlib2.builder.instruction.BuilderInstruction11n cannot be cast to com.android.tools.smali.dexlib2.iface.instruction.ReferenceInstruction
	at app.revanced.patches.youtube.shorts.components.ShortsComponentPatchKt.shortsCustomActionsPatch$lambda$16$lambda$15(ShortsComponentPatch.kt:318)
	at app.revanced.patches.youtube.shorts.components.ShortsComponentPatchKt.$r8$lambda$_1kFXTZAyHEM0_da04n367_vgpo(Unknown Source:0)
	at app.revanced.patches.youtube.shorts.components.ShortsComponentPatchKt$$ExternalSyntheticLambda2.invoke(Unknown Source:2)
	at app.revanced.patcher.patch.Patch.execute(SourceFile:71)
	at app.revanced.patcher.patch.BytecodePatch.execute$revanced_patcher(SourceFile:166)
	at app.revanced.patcher.Patcher$invoke$1.invokeSuspend$execute(SourceFile:84)
	... 19 more

	at app.revanced.patcher.Patcher$invoke$1.invokeSuspend$execute(SourceFile:75)
	at app.revanced.patcher.Patcher$invoke$1.invokeSuspend(SourceFile:109)
	at app.revanced.patcher.Patcher$invoke$1.invoke(SourceFile:0)
	at app.revanced.patcher.Patcher$invoke$1.invoke(SourceFile:0)
	at kotlinx.coroutines.flow.SafeFlow.collectSafely(SourceFile:57)
	at kotlinx.coroutines.flow.AbstractFlow.collect(SourceFile:226)
	at app.revanced.manager.flutter.MainActivity$runPatcher$1$patcherResult$1$1.invokeSuspend(SourceFile:323)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(SourceFile:33)
	at kotlinx.coroutines.DispatchedTask.run(SourceFile:101)
	at kotlinx.coroutines.EventLoopImplBase.processNextEvent(SourceFile:263)
	at kotlinx.coroutines.BlockingCoroutine.joinBlocking(SourceFile:95)
	at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking(SourceFile:69)
	at kotlinx.coroutines.BuildersKt.runBlocking(SourceFile:1)
	at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking$default(SourceFile:47)
	at kotlinx.coroutines.BuildersKt.runBlocking$default(SourceFile:1)
	at app.revanced.manager.flutter.MainActivity.runPatcher$lambda$34(SourceFile:317)
	at app.revanced.manager.flutter.MainActivity.$r8$lambda$Oku38Rh6VOg9CiulxEElyw3q4GI(SourceFile:0)
	at app.revanced.manager.flutter.MainActivity$$ExternalSyntheticLambda0.run(SourceFile:0)
	at java.lang.Thread.run(Thread.java:1012)
Compiling modified resources
Ripping selected native libs
Aligning APK
Signing APK
Patched APK

Error logs

No response

Solution

No response

Additional context

No response

Device Environment

No response

Acknowledgements

  • This issue does not reproduce on unpatched YouTube or YT Music.
  • This issue is not a duplicate of an existing bug report.
  • I did not use any settings marked as Experimental Flags.
  • I did not use pre-release (dev branch).
  • I have patched the APK according to the documentation.
  • I have chosen an appropriate title.
  • All requested information has been provided properly.
  • I have written the title and contents in English.
@ILoveOpenSourceApplications ILoveOpenSourceApplications added the Bug report Something isn't working label Dec 21, 2024
@ghost
Copy link

ghost commented Dec 21, 2024

Clear rvx manager cache and data, should fix this.

This is caused by a change in the options.json file.

@ILoveOpenSourceApplications
Copy link
Author

Clear rvx manager cache and data, should fix this.

This is caused by a change in the options.json file.

Nope. There's a patch being relied on by Shorts components for some reason which I'm not selecting and hence it's failing. Not the fault of Manager. You can try with my set of patches if you like.

@ILoveOpenSourceApplications
Copy link
Author

I know this because with default selection, there's no failing.

Screenshot_20241221_175715_RVX Manager

@ILoveOpenSourceApplications ILoveOpenSourceApplications changed the title bug(YouTube - Hide Shorts components): Patch failing during building process bug(YouTube - Hide Shorts components): Patch failing during building with certain patch selection process Dec 21, 2024
@ILoveOpenSourceApplications ILoveOpenSourceApplications changed the title bug(YouTube - Hide Shorts components): Patch failing during building with certain patch selection process bug(YouTube - Hide Shorts components): Patch failing during building with certain patch selection Dec 21, 2024
@inotia00 inotia00 changed the title bug(YouTube - Hide Shorts components): Patch failing during building with certain patch selection bug(YouTube - Shorts components): Patch failing during building with certain patch selection Dec 21, 2024
inotia00 added a commit to inotia00/revanced-patches that referenced this issue Dec 22, 2024
@inotia00
Copy link
Owner

reflected in revanced-patches-5.1.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug report Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants