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

fix(YouTube - Spoof client): Clarify that only enter/exit fullscreen gesture does not work with Android VR spoof #3243

Merged
merged 2 commits into from
May 30, 2024

Conversation

SodaWithoutSparkles
Copy link
Contributor

@SodaWithoutSparkles SodaWithoutSparkles commented May 29, 2024

Using a more conservative description of "may not work". From a sample size of 2, all other gestures did work.

Clarify that ReVanced gestures (brightness/volume) works and enter/exit mini-player works. Just that the enter/exit fullscreen by swiping the middle of the screen does not work.

Re: #3208 , ReVanced/revanced-integrations#644

Using a more conservative description of "may not work". From a sample size of 2, it did work. This could be removed after further testing
@SodaWithoutSparkles SodaWithoutSparkles changed the title fix(YouTube): Outdated side effects description for Android VR spoof fix(YouTube): Clarify that only enter/exit fullscreen gesture doesn't work on Android VR spoof May 29, 2024
@LisoUseInAIKyrios LisoUseInAIKyrios changed the title fix(YouTube): Clarify that only enter/exit fullscreen gesture doesn't work on Android VR spoof fix(YouTube - Spoof client): Clarify that only enter/exit fullscreen gesture does not work with Android VR spoof May 29, 2024
@LisoUseInAIKyrios LisoUseInAIKyrios merged commit 06d8f55 into ReVanced:dev May 30, 2024
2 checks passed
Copy link

welcome bot commented May 30, 2024

Thank you for contributing to ReVanced. Join us on Discord to receive a role for your contribution.

revanced-bot pushed a commit that referenced this pull request May 30, 2024
## [4.8.3-dev.2](v4.8.3-dev.1...v4.8.3-dev.2) (2024-05-30)

### Bug Fixes

* **YouTube - Spoof client:** Clarify that only enter/exit fullscreen gesture does not work with Android VR spoof ([#3243](#3243)) ([06d8f55](06d8f55))
@cyberboh
Copy link

I get this error while patching with 4.8.3-dev.2

Exception in thread "main" java.lang.NoSuchMethodError: 'void com.android.tools.smali.dexlib2.immutable.reference.ImmutableMethodReference.<init>(java.lang.String, java.lang.String, java.util.List, java.lang.String)'
        at app.revanced.patches.songpal.badge.fingerprints.ShowNotificationFingerprint.<clinit>(ShowNotificationFingerprint.kt:31)
        at app.revanced.patches.songpal.badge.RemoveNotificationBadgePatch.<init>(RemoveNotificationBadgePatch.kt:17)
        at app.revanced.patches.songpal.badge.RemoveNotificationBadgePatch.<clinit>(RemoveNotificationBadgePatch.kt)
        at java.base/jdk.internal.misc.Unsafe.ensureClassInitialized0(Native Method)
        at java.base/jdk.internal.misc.Unsafe.ensureClassInitialized(Unsafe.java:1160)
        at java.base/jdk.internal.reflect.MethodHandleAccessorFactory.ensureClassInitialized(MethodHandleAccessorFactory.java:300)
        at java.base/jdk.internal.reflect.MethodHandleAccessorFactory.newFieldAccessor(MethodHandleAccessorFactory.java:140)
        at java.base/jdk.internal.reflect.ReflectionFactory.newFieldAccessor(ReflectionFactory.java:145)
        at java.base/java.lang.reflect.Field.acquireFieldAccessor(Field.java:1185)
        at java.base/java.lang.reflect.Field.getFieldAccessor(Field.java:1164)
        at java.base/java.lang.reflect.Field.get(Field.java:442)
        at app.revanced.patcher.PatchBundleLoader$Utils.getInstance$revanced_patcher(PatchBundleLoader.kt:72)
        at app.revanced.patcher.PatchBundleLoader$3.invoke(PatchBundleLoader.kt:51)
        at app.revanced.patcher.PatchBundleLoader$3.invoke(PatchBundleLoader.kt:50)
        at kotlin.sequences.TransformingSequence$iterator$1.next(Sequences.kt:210)
        at kotlin.sequences.FilteringSequence$iterator$1.calcNext(Sequences.kt:170)
        at kotlin.sequences.FilteringSequence$iterator$1.hasNext(Sequences.kt:194)
        at kotlin.sequences.FilteringSequence$iterator$1.calcNext(Sequences.kt:169)
        at kotlin.sequences.FilteringSequence$iterator$1.hasNext(Sequences.kt:194)
        at kotlin.collections.CollectionsKt__MutableCollectionsKt.addAll(MutableCollections.kt:129)
        at app.revanced.patcher.PatchBundleLoader.<init>(PatchBundleLoader.kt:55)
        at app.revanced.patcher.PatchBundleLoader.<init>(PatchBundleLoader.kt)
        at app.revanced.patcher.PatchBundleLoader.<init>(PatchBundleLoader.kt:34)
        at app.revanced.patcher.PatchBundleLoader$Jar.<init>(PatchBundleLoader.kt:102)
        at app.revanced.cli.command.PatchCommand.run(PatchCommand.kt:259)
        at picocli.CommandLine.executeUserObject(CommandLine.java:2026)
        at picocli.CommandLine.access$1500(CommandLine.java:148)
        at picocli.CommandLine$RunLast.executeUserObjectOfLastSubcommandWithSameParent(CommandLine.java:2461)
        at picocli.CommandLine$RunLast.handle(CommandLine.java:2453)
        at picocli.CommandLine$RunLast.handle(CommandLine.java:2415)
        at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:2273)
        at picocli.CommandLine$RunLast.execute(CommandLine.java:2417)
        at picocli.CommandLine.execute(CommandLine.java:2170)
        at app.revanced.cli.command.MainCommandKt.main(MainCommand.kt:12)

Seems error in xml or something :/

@LisoUseInAIKyrios
Copy link
Contributor

It's something with the smali library bump from 3.0.5 to 3.0.7

dd1a087#r142432572

While Patcher is still at 3.0.5
https://github.com/ReVanced/revanced-patcher/blob/7be131d348ba70689369e4ecf82478c89d03e2a3/gradle/libs.versions.toml#L7

I think bumping the patcher dependency to 3.0.7 will fix this.

@oSumAtrIX
Copy link
Member

That indicates a breaking change on smali even though patch was bumped. This should probably be reported.

@LisoUseInAIKyrios
Copy link
Contributor

LisoUseInAIKyrios commented May 30, 2024

Should Patcher be bumped to 3.0.7, or should Patches revert back to 3.0.5?

Bumping patcher means Manager would need an update, so it's probably easier to revert patches to 3.0.5

@oSumAtrIX
Copy link
Member

We can revert but eventually we need to update

@oSumAtrIX
Copy link
Member

oSumAtrIX commented May 30, 2024

google/smali@bad9474#diff-1b64ab2237880ceb837437aa949dca5004f305bd0aac42fe8f6a67b3d6e971d6R60 here is the breaking change.

Tracking: google/smali#58

revanced-bot pushed a commit that referenced this pull request May 31, 2024
## [4.8.3](v4.8.2...v4.8.3) (2024-05-31)

### Bug Fixes

* **3rd-party Reddit apps:** Spoof user agent to work around Reddit API issues ([#3253](#3253)) ([495e6d6](495e6d6))
* **Reddit - Hide ads:** Constrain to last working version 2024.17.0 ([#3192](#3192)) ([4fb3456](4fb3456))
* **YouTube - Spoof client:** Clarify that only enter/exit fullscreen gesture does not work with Android VR spoof ([#3243](#3243)) ([06d8f55](06d8f55))
* **YouTube - Spoof client:** Improve Android spoofing ([#3230](#3230)) ([b688923](b688923))
E85Addict pushed a commit to E85Addict/revanced-patches that referenced this pull request May 31, 2024
## [4.8.3](v4.8.2...v4.8.3) (2024-05-31)

### Bug Fixes

* **3rd-party Reddit apps:** Spoof user agent to work around Reddit API issues ([ReVanced#3253](https://github.com/E85Addict/revanced-patches/issues/3253)) ([495e6d6](495e6d6))
* **Reddit - Hide ads:** Constrain to last working version 2024.17.0 ([ReVanced#3192](https://github.com/E85Addict/revanced-patches/issues/3192)) ([4fb3456](4fb3456))
* **YouTube - Spoof client:** Clarify that only enter/exit fullscreen gesture does not work with Android VR spoof ([ReVanced#3243](https://github.com/E85Addict/revanced-patches/issues/3243)) ([06d8f55](06d8f55))
* **YouTube - Spoof client:** Improve Android spoofing ([ReVanced#3230](https://github.com/E85Addict/revanced-patches/issues/3230)) ([b688923](b688923))

### Performance Improvements

* Personal Logo && Add upstream sync ([929bf2b](929bf2b))
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.

4 participants