Skip to content
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

Draft: Android CI production build and Play Store auto-submission #3390

Closed
wants to merge 2 commits into from

Conversation

pljones
Copy link
Collaborator

@pljones pljones commented Oct 2, 2024

Short description of changes

Revival of #2909

The primary goal of the patch was to enable Google Play Store automated uploads as part of the GitHub build process.

CHANGELOG: DO NOT MERGE

Context: Fixes an issue?

The patch not only adopts the new action (r0adkll/upload-google-play@v1) but changes dependencies:

Variable from to
COMMANDLINETOOLS_VERSION 6858069 7.0
ANDROID_NDK_VERSION r21d 25.1.8937393
ANDROID_PLATFORM android-30 android-33
ANDROID_BUILD_TOOLS 30.0.2 33.0.0
AQTINSTALL_VERSION 3.1.18 2.1.0
QT_VERSION 5.15.2 6.3.2

As a side effect, it enables the Jamulus.pro to build on Windows by removing the Linux dependency (or at least side-stepping it).

Code changes are a side effect of the move to Qt6 (but I think probably work in Qt5.15.2, too, so could be trialled).

Does this change need documentation? What needs to be documented and how?

We should add documentation about the build process somewhere but that's not part of this ticket. Unless it already exists and needs changing.

Status of this Pull Request

Proof of concept (not to be merged soon).

What is missing until this pull request can be merged?

  1. Decide if we do this in r4.0 rather than r3.12.0.
  2. Someone who knows the build chain needs to review it.
  3. We need to understand the proposed Github action and agree it's acceptable (and choose a version to pin).
  4. The updated dependencies need to be understood, including their effect on the supported Android versions and how that affects Jamulus documentation.

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I tested my code and it does what I want
  • My code follows the style guide
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I've filled all the content above

danryu and others added 2 commits October 2, 2024 11:11
* android build conf changes

* fix shellcheck warnings

android fix audio permission handling

android fix androidextras include

android remove redundant jdk install

add ANDROID_NDK_ROOT comment

fix for clang-format style check

android add all ABI support

android: remove redundant minSdkVersion

clang-format fixes

android: fix manifest xml, remove old boilerplate

add ANDROID_SDK_ROOT shellcheck fix

android fix assignments in pro file

fix typo
@pljones
Copy link
Collaborator Author

pljones commented Oct 2, 2024

The main issue is the move to Qt6. I think that should be addressed separately. If this change requires Qt6, then it gets blocked until we've made the decision to move Android to Qt6 and made the necessary code changes correctly.

@pljones pljones closed this Oct 2, 2024
@pljones pljones deleted the feature/koord/CIPlaystore branch October 2, 2024 11:03
@pljones pljones restored the feature/koord/CIPlaystore branch October 2, 2024 14:27
@pljones pljones reopened this Oct 2, 2024
@pljones pljones closed this Oct 2, 2024
@pljones pljones deleted the feature/koord/CIPlaystore branch October 2, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants