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
Android wrapper: Modernize build files #619
Conversation
Supersedes #555 |
a399655
to
9d65b12
Compare
The main road-block that prevented #555 from moving forward is still there: https://github.com/zxing-cpp/zxing-cpp/actions/runs/6262637658/job/17005466542?pr=619#step:3:34 |
9d65b12
to
51ecbf0
Compare
Updated the Java version used by the CI worker, should build now. |
Thanks, but apparently there is still something missing: https://github.com/zxing-cpp/zxing-cpp/actions/runs/6262900310/job/17006568650?pr=619 |
This commit updates the Android build files. No source files were changed. Notable changes: - Convert build scripts to Kotlin (build.gradle -> build.gradle.kts) I tried to introduce this change incrementally to keep the build file history tracked by Git, but was unable to do so. - Kotlin 1.9.10 - Gradle 8.3, including the Gradle wrapper - Use Gradle Version Catalog - Android Gradle Plugin 8.1.1
51ecbf0
to
4df2556
Compare
Right... Didn't know it was a required field. |
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.
Thanks for your contribution. Much appreciated.
Thanks for the contribution @okarmazin. I still want to overhaul the iOS wrapper someday very soon :) That's a nice push! |
This PR updates the Android build files and updates project dependencies. No source files were changed.
I tried to introduce this change by incremental atomic commits to keep the build file history tracked by Git, but was unable to do so. The amount of changes required for the Kotlin script conversion was too great.
Notable changes: