-
Notifications
You must be signed in to change notification settings - Fork 85
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
AndroidX migration to solve problems in demo app. #131
base: master
Are you sure you want to change the base?
Conversation
Hello @emreakcan! For us it's the opposite: it works fine right now but compilation breaks in your branch. We've checked on two separate instances, same result. Would you like to investigate? |
I will investigate with clean workspace and let you know, thank you. |
@zsoltk For fresh start, I cloned the project again. With 2 different computers. One worked okay, one gave following error, I'm comparing now to find the issue, it seems like a problem from my side. Build result
Result of
|
@emreakcan Did you find the issue? |
@zsoltk not yet, but since it's from my side I closed the PR. I will give an update when I find it. I'm suspected about Android Studio version, gradle version, jdk version, sdk version etc. |
No ideas unfortunately. Let me know if you figure it out. |
using android x for lifecycle-common.
@zsoltk hello, I did further investigation but still a mystery, also asked here https://stackoverflow.com/questions/63811131/android-arch-lifecyclecommon-java8-is-not-recognised Are you thinking to migrate this project to andriodX ? if so, I can continue to work on it, I didn't migrate the support libraries yet. And can you please check my new changes if it compiles for you? (added 2 commits) |
Discussion apparently continued at https://www.reddit.com/r/androiddev/comments/iqa3dm/why_androidarch_doesnt_work_anymore/ . EDIT: But I tried and it works on my machine. You might just need to invalidate caches, restart, and clean rebuild. |
@Zhuinden thanks !, that reddit post is mine also. |
@Zhuinden @zsoltk Works for me. But still doesn't explain why 3.4.2 doesn't work. @zsoltk, Like I said if migrating to androidX is okay, I can continue, if it's not desired, I will stop working on it. |
I couldn't run the demo app straight away, as I see some library upgrades are needed so I resolved the problem by small migration, now I can run the app.
Please warn me if it is not a necessary change.