-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
1.compat android 13 14 #1191
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
base: vxp
Are you sure you want to change the base?
1.compat android 13 14 #1191
Conversation
can make it compatible for android 15 also? |
It runs on the Android Studio emulator(Android 13), but I haven't tested the actual functionality yet. EDIT EDIT2 Also, please refer to the Android emulator information below. |
I've found that the DataUtil.safeToInt() method should be applied more broadly. For example, a error occurs on the following line: Line 981 in 4d844b5
To fix this and other similar assignment casts throughout the project, we can use a simple regex find-and-replace. Find
Replace
|
I would like to evaluate the 'me.weishu:epic:0.11.1' being used here. Please refer to the following link: The same package can be found on the following GitHub repository, which has advanced to version 0.11.2: It's not yet clear whether this code causes issues, and we should potentially consider the Android support versions of the dependencies. |
Issue (Solved)This error occurs simultaneously on both real Android 13 and x86_64 emulator, so it should be possible to debug through the emulator. The class This could be an issue that occurs during the reflection process.
SolutionI apologize for the confusion. The taobao issue was related to the dependency version.
I replaced the above code with the below code to work around compilation issues:
Although the issue has been resolved in the latest code from https://github.com/android-hacker/exposed, version 0.3.6 is very old and was the latest tagged release available. The root cause was that my existing environment had errors with jcenter, so I used I have now found a way to still use jcenter and resolved the issue. I will continue to report on problems with using |
Here, This is the support issue I mentioned earlier regarding To summarize:
Since we likely cannot wait for tiann to accept PRs, it appears we would need to work through a GitHub fork, then deploy our own repository via jitpack.io to proceed with the work. To actually achieve Android 13 and 14 support, the above work is necessary, and there may be additional errors even after resolving this issue. |
To resolve the issue, I will try applying Looking at the build error, the issue is a duplicate class conflict:
This suggests that when you replaced Regarding the duplicate class issue: The problem likely occurs because:
Potential solutions:
The issue suggests that the transition from the official Maven repository version to the JitPack GitHub version introduces dependency conflicts that need to be resolved through proper exclusions or version alignment.
|
You can remove this dependency. Just fix the compilation errors within it. This is not necessary. |
Sorry, I couldn't understand your intention. I analyzed and resolved it as follows: The original working configuration was: dependencies {
implementation fileTree(include: ['*.jar'], dir: 'libs')
api("me.weishu.exposed:exposed-core:0.8.1") {
exclude group: 'me.weishu', module: 'free_reflection'
exclude group: 'me.weishu', module: 'epic'
exclude group: 'me.weishu.exposed', module: 'exposed-xposedapi'
}
implementation "me.weishu:free_reflection:3.0.1"
implementation 'me.weishu:epic:0.11.1'
implementation "me.weishu.exposed:exposed-xposedapi:0.4.6"
} When I changed to use // implementation 'me.weishu:epic:0.11.1'
implementation 'com.github.tiann:epic:0.11.2' // This caused BuildConfig conflict I discovered that When I removed all FreeReflection dependencies: // implementation "me.weishu:free_reflection:3.0.1"
// implementation 'com.github.tiann:FreeReflection:3.2.2'
implementation 'com.github.tiann:epic:0.11.2' The compilation succeeded, but Finally, I used the compatible combination: implementation 'com.github.tiann:FreeReflection:3.2.2'
implementation 'com.github.tiann:epic:0.11.2' This resolved both the compilation errors and ensured that |
Remove all the dependencies? |
Let me clarify what I meant: By "When I removed all FreeReflection dependencies," I meant I only removed the active So the configuration was: // implementation "me.weishu:free_reflection:3.0.1" // <- REMOVED this line
// implementation 'com.github.tiann:FreeReflection:3.2.2' // <- This was already commented
implementation 'com.github.tiann:epic:0.11.2' I didn't remove ALL dependencies - that would obviously cause compilation errors. Are you suggesting |
Description
What does this PR do?
Type of change
Checklist