Android CI fix: Prevent future android SDK releases from messing up CI

[gillamkid]

Reason Fix is Needed

"android-35-ext15" was just added and it broke Android CI. Instead of adding a new uninstall command for "android-35-ext15" I made a command which uninstalls anything following the pattern android-* (except for android-33)

Sponsor

This contribution was sponsored by Firestorm

[DonLakeFlyer]

Doesn't this need to be in master as well?

[gillamkid]

Doesn't this need to be in master as well?

I don't think so. On master the CI .yml files for Android have changed quite a bit and master does not appear to run uninstall to remove unused Android SDKs. Here is the equivalent section of code on mater

qgroundcontrol/.github/actions/qt-android/action.yml

Lines 41 to 57 in bafa732

	- name: Install Android NDK
	uses: nttld/setup-ndk@v1
	id: setup-ndk
	with:
	ndk-version: r26d
	add-to-path: false
	- run: |
	echo "ANDROID_NDK_ROOT=${{ steps.setup-ndk.outputs.ndk-path }}" >> "$GITHUB_ENV"
	echo "ANDROID_NDK_HOME=${{ steps.setup-ndk.outputs.ndk-path }}" >> "$GITHUB_ENV"
	echo "ANDROID_NDK=${{ steps.setup-ndk.outputs.ndk-path }}" >> "$GITHUB_ENV"
	shell: bash
	- name: Update Android SDK / NDK / Tools
	if: ${{ runner.os != 'Windows' }}
	run: sdkmanager --update
	shell: bash

I'm not sure what masteris doing to avoid the need to run the uninstall commands (maybe it's setting the right set of environment variables??) but I would say the change I've suggested in this PR is prolly the smallest/easiest/least-intrusive change that will keep the Android CI build of Stable_V4.4 working (whereas now it repeatedly breaks as Android releases new SDKs)

[HTRamsey]

Doesn't this need to be in master as well?

No, I've got it set up so this no longer matters

[julianoes]

Thanks, that makes a lot more sense!