Skip to content

[android] Update docker image for SDK and Qt support#44548

Closed
dg0yt wants to merge 6 commits intomicrosoft:masterfrom
dg0yt:android-docker
Closed

[android] Update docker image for SDK and Qt support#44548
dg0yt wants to merge 6 commits intomicrosoft:masterfrom
dg0yt:android-docker

Conversation

@dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Mar 22, 2025

Complements #42080.
CC @BillyONeal

## PowerShell
dpkg -i packages-microsoft-prod.deb
rm -f packages-microsoft-prod.deb
add-apt-repository universe
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Discovered by set -e.


ADD https://packages.microsoft.com/config/ubuntu/24.04/packages-microsoft-prod.deb /packages-microsoft-prod.deb
ADD https://builds.openlogic.com/downloadJDK/openlogic-openjdk/11.0.25+9/openlogic-openjdk-11.0.25+9-linux-x64.tar.gz /openlogic-openjdk-11.0.25+9-linux-x64.tar.gz
ADD https://github.com/adoptium/temurin17-binaries/releases/download/jdk-17.0.14+7/OpenJDK17U-jdk_x64_linux_hotspot_17.0.14_7.tar.gz /OpenJDK17U-jdk_x64_linux_hotspot_17.0.14_7.tar.gz
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment on lines +85 to +83
find android-sdk/platforms -type d -exec chmod o+rx '{}' ';'
find android-sdk/platforms -type f -exec chmod o+r '{}' ';'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

unzip creating rwxrwx--- 🤦

@dg0yt dg0yt mentioned this pull request Mar 23, 2025
3 tasks
@jimwang118 jimwang118 added category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. labels Mar 24, 2025
@jimwang118
Copy link
Contributor

jimwang118 commented Mar 24, 2025

@Victor-Bergman Add reviewed tag for further review by the vcpkg team.

@BillyONeal BillyONeal added depends:vm-update PR contains changes to the VM provisioning scripts and removed info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. labels Mar 27, 2025
@PhoebeHui PhoebeHui assigned Cheney-W and unassigned jimwang118 Apr 2, 2025
@dg0yt
Copy link
Contributor Author

dg0yt commented Apr 8, 2025

@BillyONeal I hope there is a slot for this.

BillyONeal added a commit to BillyONeal/vcpkg that referenced this pull request Apr 8, 2025
@BillyONeal
Copy link
Member

The contents of this PR were incorporated into #44922 ; thanks!

@BillyONeal BillyONeal closed this Apr 10, 2025
@dg0yt dg0yt deleted the android-docker branch April 10, 2025 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly depends:vm-update PR contains changes to the VM provisioning scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants