CICD: Improve Location of Scripts for the Android Build Environment.

Move around the scripts required for the setup of the build environment
for android to satisfy docker's requirement of locality.
This allows the removal of an extra copy step, and avoids the creation
of extra artefacts, while still providing the same functionality.

Signed-off-by: Michael Keller <github@ike.ch>
This commit is contained in:
Michael Keller 2024-01-08 07:51:43 +13:00
parent ec83e157e9
commit b5efaf661c
8 changed files with 4 additions and 21 deletions

View file

@ -5,8 +5,6 @@ on:
paths: paths:
- scripts/docker/android-build-container - scripts/docker/android-build-container
- .github/workflows/android-docker* - .github/workflows/android-docker*
- packaging/android/android-build-setup.sh
- packaging/android/variables.sh
jobs: jobs:
android-build-container: android-build-container:
@ -17,11 +15,6 @@ jobs:
steps: steps:
- uses: actions/checkout@v1 - uses: actions/checkout@v1
- name: Get our pre-reqs
run: |
cd scripts/docker/android-build-container
bash setup-docker.sh -no-docker-build
- name: Build the name for the docker image - name: Build the name for the docker image
id: build_name id: build_name
run: | run: |

2
.gitignore vendored
View file

@ -46,6 +46,4 @@ packaging/ios/Info.plist
packaging/ios/Qt packaging/ios/Qt
packaging/ios/asset_catalog_compiler.Info.plist packaging/ios/asset_catalog_compiler.Info.plist
appdata/subsurface.appdata.xml appdata/subsurface.appdata.xml
scripts/docker/android-build-container/android-build-setup.sh
scripts/docker/android-build-container/variables.sh
android-mobile/Roboto-Regular.ttf android-mobile/Roboto-Regular.ttf

View file

@ -85,7 +85,7 @@ unzip commandlinetools-linux-*.zip
git clone https://github.com/subsurface/subsurface git clone https://github.com/subsurface/subsurface
# now get the SDK, NDK, Qt, everything that's needed # now get the SDK, NDK, Qt, everything that's needed
bash /android/subsurface/packaging/android/android-build-setup.sh bash /android/subsurface/scripts/docker/android-build-container/android-build-setup.sh
``` ```
Once this has completed, you should have a working build environment. Once this has completed, you should have a working build environment.

View file

@ -4,7 +4,7 @@
# #
# this requires Qt5.14 or newer with matching NDK # this requires Qt5.14 or newer with matching NDK
# #
# the packaging/android/android-build-setup.sh sets up an environment that works for this # the scripts/docker/android-build-container/android-build-setup.sh sets up an environment that works for this
set -eu set -eu
@ -22,7 +22,7 @@ popd
BUILD_TYPE=Debug BUILD_TYPE=Debug
# Read build variables # Read build variables
source $SUBSURFACE_SOURCE/packaging/android/variables.sh source $SUBSURFACE_SOURCE/scripts/docker/android-build-container/variables.sh
# this assumes that the Subsurface source directory is in the same # this assumes that the Subsurface source directory is in the same
# directory hierarchy as the SDK and NDK # directory hierarchy as the SDK and NDK

View file

@ -12,13 +12,5 @@
# If this fails, go to https://developer.android.com/studio#cmdline-tools and # If this fails, go to https://developer.android.com/studio#cmdline-tools and
# click through for yourself, and then update the URL in the Dockerfile # click through for yourself, and then update the URL in the Dockerfile
# copy the dependency script into this folder
cp ../../../packaging/android/android-build-setup.sh .
cp ../../../packaging/android/variables.sh .
if [ "$1X" == "-no-docker-buildX" ]; then
exit 0
fi
# create the container (this takes a while) # create the container (this takes a while)
docker build -t android-build . docker build -t android-build .

View file

@ -104,7 +104,7 @@ fi
# FIX FOR ANDROID, # FIX FOR ANDROID,
if [ "$PLATFORM" == "singleAndroid" ] ; then if [ "$PLATFORM" == "singleAndroid" ] ; then
CURRENT_OPENSSL="OpenSSL_1_1_1m" CURRENT_OPENSSL="OpenSSL_1_1_1m"
# If changing the openSSL version here, make sure to change it in packaging/android/variables.sh also. # If changing the openSSL version here, make sure to change it in scripts/docker/android-build-container/variables.sh also.
fi fi
# no curl and old libs (never version breaks) # no curl and old libs (never version breaks)
# check whether to use curl or wget # check whether to use curl or wget