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: Excluding 'custom-package-name' patch overrides default package name #1388

Closed
5 tasks done
hlytus opened this issue Sep 20, 2023 · 6 comments
Closed
5 tasks done
Labels
Bug report Something isn't working

Comments

@hlytus
Copy link
Collaborator

hlytus commented Sep 20, 2023

Type

Error while patching

Tools used

Revancify

Application

Youtube - 18.31.40, 18.35.35
YouTube music

Bug description

Even if the custom-package-name patch is excluded it still patches with the package name that is entered in the options file.

In the past, if the patch was excluded it used app.rvx.android.youtube as default, ignoring what was present in the options file.

Steps to reproduce

  1. Change package name in options file
  2. Exclude custom-package-name and patch youtube.

Output app will have the package name that was in the options file.

Relevant log output

_

Screenshots or videos

No response

Solution

No response

Additional context

No response

Device Environment

No response

Acknowledgements

  • I have searched the existing issues and this is a new and no duplicate or related to another open issue.
  • I have written a short but informative title.
  • I filled out all of the requested information in this issue properly.
  • I have written the title and contents in English.
  • This issue does not reproduce on unpatched YouTube or YT Music.
@hlytus hlytus added the Bug report Something isn't working label Sep 20, 2023
@inotia00
Copy link
Owner

Even if you exclude the custom-package-name patch, the custom-package-name patch is included if you include the microg-support patch.

From the beginning, the custom-package-name patch cannot be forcibly excluded or included.

Due to structural limitations of ReVanced-Patcher, OptionsContainer cannot be added to the microg-support patch.

So I separated it into microg-support patch and custom-package-name patch.
(In other words, you can just think of the custom-package-name patch as the same as the microg-support patch)

  1. If you do not want to use the custom-package-name patch, you must also exclude the microg-support patch.
  2. Even if you include the custom-package-name patch, the package name will not change unless you exclude the microg-support patch.

@KobeW50
Copy link
Contributor

KobeW50 commented Sep 20, 2023

  1. Even if you include the custom-package-name patch, the package name will not change unless you exclude the microg-support patch.

I'm having trouble understanding this, as I always include the microg-support patch (along with the custom-package-name patch) and the custom package name (specified in the patch options) is applied

Do you mean that the custom package name won't be applied unless you (also) include the microg-support patch?

@hlytus
Copy link
Collaborator Author

hlytus commented Sep 20, 2023

If microg-support is included, custom-package-name is also included with it even if you exclude it, so it will patch with whatever is entered in the options file.

But if you don't include microg-support patch then custom package name will not be applied and it will use the default YouTube package name i.e. com.google.android.yoututbe

In my understanding, custom-package-name is useless.

@inotia00
Copy link
Owner

If microg-support is included, custom-package-name is also included with it even if you exclude it, so it will patch with whatever is entered in the options file.

But if you don't include microg-support patch then custom package name will not be applied and it will use the default YouTube package name i.e. com.google.android.yoututbe

In my understanding, custom-package-name is useless.

You understood correctly.. just think of custom-package-name as effectively a dummy patch.

Nevertheless, the reason it is shown in the patch list is because if I make it hidden in the patch list, the custom-package-name option is not added to options.json.

This is a structural limitation of the current ReVanced-Patcher.

@hlytus
Copy link
Collaborator Author

hlytus commented Sep 20, 2023

Nevertheless, the reason it is shown in the patch list is because if I make it hidden in the patch list, the custom-package-name option is not added to options.json.

ohk. makes sense.

@inotia00
Copy link
Owner

While looking at the latest ReVanced Patches, I found out that this can be implemented in a fairly complicated way.

This will be reflected in the next release.

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

3 participants