-
Notifications
You must be signed in to change notification settings - Fork 535
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 #3855: Removed Pinview thirdparty library #4041
Fix #3855: Removed Pinview thirdparty library #4041
Conversation
@BenHenning i have removed library from build.bazel and gradle and also from versions.bzl after that i removed dependencies from maven_install.json when i tried to repin dependencies its showing error running one into another(kind of loop) |
@deepanshu731 I suggest reverting the changes to the maven_install.json file, then re-running the pin command to see if it regenerates correctly. If that doesn't work, you'll need to follow the step-by-step instructions that the error outputted in your screenshot. |
…731/oppia-android into remove-pinview-library d
Hi. As of today, some main reviewers have taken time off for the next few weeks, so it may take a little while before we can look at this PR. We appreciate your patience while some of our team members recharge. We'll be fully returning on 4 January 2021. |
Hi @deepanshu731, I'm going to mark this PR as stale because it hasn't had any updates for 7 days. If no further activity occurs within 7 days, it will be automatically closed so that others can take up the issue. |
thanks @BenHenning maven dependencies got repinned but some CL checks are failing. can you help me out with that? |
@deepanshu731 apologies for the late reply. The Bazel failure is due to an incompatibility between the new dependency graph & the Proguard configuration. Here are the specific errors:
This is probably happening because removing the pinview library inadvertently downgraded appcompat (see https://github.com/oppia/oppia-android/pull/4041/files#diff-c6821e7e22305da601268ca77e90fa7b1d33be10fc8ea6746832d2c3a794d2feR14). I suggest updating appcompat in versions.bzl to 1.2.0 so that it doesn't get downgraded & see if it builds. To test locally, run:
(It'll probably take 10-15 minutes to build since it runs 3 Proguard optimization passes). The Gradle failure seems like a flake, so after you fix the dependencies & restart CI I'd expect it to then pass. |
thanks @BenHenning PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This LGTM @deepanshu731. Thanks again!
Fixes #3855 removed lib from build bazel,gradle and versions.bzl