Skip to content

Modernize Gradle Configuration, Dependencies, and Upgrade Gradle to 8…#503

Open
jawad111 wants to merge 1 commit into
tensorflow:masterfrom
jawad111:master
Open

Modernize Gradle Configuration, Dependencies, and Upgrade Gradle to 8…#503
jawad111 wants to merge 1 commit into
tensorflow:masterfrom
jawad111:master

Conversation

@jawad111

@jawad111 jawad111 commented Sep 30, 2024

Copy link
Copy Markdown

Modernize Gradle Configuration, Dependencies, and Upgrade to Gradle 8.1.4

This pull request resolves build issues and modernizes the project's build system by:

  • Upgrading Gradle to 8.1.4 and using the Kotlin DSL.
  • Updating compile and target SDK versions to 34.
  • Refreshing key dependencies, including AppCompat and Guava.
  • Improving JNI libraries configuration and packaging options.
  • Ensuring Kotlin Stdlib version consistency.

These changes enhance compatibility with the latest Android platform, improve build efficiency, and reduce the final APK size. They also leverage modern Gradle features for a more maintainable and optimized build process.

Considerations:

  • Verify compatibility with the updated dependencies.
  • Thoroughly test functionality across different devices.

Changes Made:

To be more specific, here's a summary of the changes:

  • Gradle: Updated to 8.1.4, enabling new features and Kotlin DSL usage.
  • SDK Versions: compileSdk and targetSdk are now 34 for latest Android support.
  • Dependencies: AppCompat, Guava, and support design library updated for better compatibility.
  • JNI Libraries: Improved configuration for proper packaging within the APK.
  • Kotlin: Resolution strategy ensures consistent Kotlin Stdlib versions.
  • Packaging: Optimized to reduce APK size and improve debug build efficiency.

….1.4 in /lite/examples/super_resolution/android
@zoldaten

Copy link
Copy Markdown

could it be implemented to all code examples or only to superresolution ?

@jawad111

Copy link
Copy Markdown
Author

could it be implemented to all code examples or only to superresolution ?

Perhaps. I have not tried it on other examples yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review size:XL CL Change Size: Extra Large

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants