Skip to content

Update and improve android_gradle#965

Merged
paulthomson merged 1 commit intogoogle:mainfrom
paulthomson:update_android
Sep 28, 2021
Merged

Update and improve android_gradle#965
paulthomson merged 1 commit intogoogle:mainfrom
paulthomson:update_android

Conversation

@paulthomson
Copy link
Collaborator

  • Update CMakeLists.txt files so we can build libamber_ndk.so for
    Android via CMake.

  • android_gradle/:

    • Update Gradle version, settings, and build files to match the latest
      Android sample. Importantly, this removes the use of jcenter and
      bintray repositories, which were shut down.

    • Specify NDK version so that Gradle can install required native build
      tools automatically.

    • Define externalNativeBuild so Gradle can perform the Android CMake
      build automatically.

  • Replace main with android_main when building the Amber tool as a
    library for Android to avoid compiler warnings, and rename other
    Android-related symbols to avoid confusion.

Fixes #964

* Update CMakeLists.txt files so we can build libamber_ndk.so for
Android via CMake.

* `android_gradle/`:

  * Update Gradle version, settings, and build files to match the latest
  Android sample. Importantly, this removes the use of jcenter and
  bintray repositories, which were shut down.

  * Specify NDK version so that Gradle can install required native build
  tools automatically.

  * Define externalNativeBuild so Gradle can perform the Android CMake
  build automatically.

* Replace `main` with `android_main` when building the Amber tool as a
library for Android to avoid compiler warnings, and rename other
Android-related symbols to avoid confusion.

Fixes google#964
Copy link
Contributor

@asuonpaa asuonpaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@paulthomson paulthomson merged commit 93e89ee into google:main Sep 28, 2021
@paulthomson paulthomson deleted the update_android branch September 28, 2021 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

android_gradle no longer builds

3 participants