DONOTMERGE: CI: remove cached Qt when building Android APK #21537

pull icota wants to merge 1 commits into bitcoin:master from icota:fix-apk-ci changing 1 files +3 −0

icota commented at 8:38 AM on March 28, 2021: contributor

No description provided.
CI: remove cached Qt if building Android APK ec925f65db
fanquake added the label Tests on Mar 28, 2021
in ci/test/05_before_script.sh:36 in ec925f65db
```
  31 | @@ -32,6 +32,9 @@ if [ -n "$ANDROID_TOOLS_URL" ]; then
  32 |    DOCKER_EXEC unzip -o "$ANDROID_TOOLS_PATH" -d "${ANDROID_HOME}/cmdline-tools"
  33 |    DOCKER_EXEC "yes | ${ANDROID_HOME}/cmdline-tools/tools/bin/sdkmanager --install \"build-tools;${ANDROID_BUILD_TOOLS_VERSION}\" \"platform-tools\" \"platforms;android-${ANDROID_API_LEVEL}\" \"ndk;${ANDROID_NDK_VERSION}\""
  34 |  
  35 | +  # Remove cached Qt
  36 | +  DOCKER_EXEC rm -rf "${DEPENDS_DIR}"/built/aarch64-linux-android/qt
```
MarcoFalke commented at 1:00 PM on March 28, 2021:

Is there any insight available as to why this fixes the ci?

icota commented at 4:21 PM on March 28, 2021:

There's partial insight.

APK CI build fails when there is no qtbase*.tar.xz archive in depends/sources to extract "Qt for Android" java files from. Line above forces a redownload of Qt sources and fixes that particular issue.

I could produce a proper fix if I knew why or when the sources are removed in between CI runs. I see a clean-all target that wipes $(SOURCES_PATH) clean but surely that's not it? Since that one would delete build as well.

MarcoFalke commented at 4:32 PM on March 28, 2021:

depends/sources isn't cached and never has been

See #21541 for a fix

icota commented at 4:39 PM on March 28, 2021:

That was extermely fast. Thank you!

Closing this.
icota closed this on Mar 28, 2021
DrahtBot locked this on Aug 16, 2022

Contributors

icota

MarcoFalke

Labels

Tests