Skip to content

Automated build for Android APK#878

Closed
nefarius2001 wants to merge 22 commits into
jamulussoftware:masterfrom
nefarius2001:master
Closed

Automated build for Android APK#878
nefarius2001 wants to merge 22 commits into
jamulussoftware:masterfrom
nefarius2001:master

Conversation

@nefarius2001

Copy link
Copy Markdown
Contributor

Credit and thanks to @ann0see for contributing & support

Resolves #835 Automated build for Android APK

open/related issues, that I see as upcomming improvements, but not part of this PR/issue:
#856 harmonized folders
#857 multiple APK-platforms

this PR creates automatically:

  • a prerelease for all tags that start with "r" and contain "beta"
  • a release for all tags that start with "r"
    I ask you to accept the PR with this minimal behaviour and lets continue the discussion about behaviour in Automated github builds #854

The build process for android can be improved still, but I think it is already very good to have a working state as basis.

@corrados

Copy link
Copy Markdown
Contributor

Wow, I am very impressed! I saw that you also did some changes in the Windows installer script (deploy_windows.ps1). @ann0see, have you already reviewed that part?

@corrados

Copy link
Copy Markdown
Contributor

@ann0see BTW, what is the best way to write you a PM?

@ann0see

ann0see commented Jan 23, 2021

Copy link
Copy Markdown
Member

I've rewritten the windows part.

I will have a look at these changes soon.

@ann0see

ann0see commented Jan 23, 2021

Copy link
Copy Markdown
Member

@corrados you can have a look at the billing E-Mail address in the jamulussoftware Organisation and I can afterwards send you my private E-Mail Adress, Threema ID/Signal number,...

@pljones

pljones commented Jan 23, 2021

Copy link
Copy Markdown
Collaborator

I would still like, if possible, the CodeQL checks run against this -- if CodeQL doesn't support scanning the Android code, fair enough, though.

@corrados

Copy link
Copy Markdown
Contributor

I would still like, if possible, the CodeQL checks run against this

I agree, CodeQL check makes sense. But we should get rid of the Travis builds, don't we?

@ann0see ann0see left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A few comments.
Same problem as you had with my PR: commit history ;-)

Comment thread .github/actions_scripts/analyse_git_reference.py
Comment thread .github/actions_scripts/analyse_git_reference.py
Comment thread .github/workflows/autobuild.yml
Comment thread .github/workflows/autobuild.yml
Comment thread android/automated_build/action.yml
Comment thread android/automated_build/install-qt.sh
Comment thread android/automated_build/install-qt.sh
# Install Qt
pip install aqtinstall
echo "Get Qt 64 bit..."
# intermediate solution if the main server is down: append e.g. " -b https://mirrors.ocf.berkeley.edu/qt/" to the "aqt"-line below

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A fallback argument has been added to aqt recently. Maybe we could use this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

good idea. I hop I will not forget this.
Likely I will make the qt-install in Android build aqt based, too

@nefarius2001

Copy link
Copy Markdown
Contributor Author

opened new PR with nicer git history (opened PR from master and not branch by mistake)

please continue discussion there:
#880

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.

Automated build for Android APK

4 participants