Skip to content

Conversation

@am11
Copy link
Member

@am11 am11 commented Feb 18, 2023

This is to complement @ayakael's toolchain changes #11608.

  • Set minimum versions to 3.15 for s390x and ppc64le architectures which provides all packages we need.
  • riscv64 doesn't provide compiler-rt-static and lldb-dev; skip those two packages.
  • riscv64 requires (the bleeding) edge version atm. The bleeding edge version can have any latest version of llvmXY-libs package, which is unfortunately version specific. To accommodate that, add a lookup step.
  • Restructure case-esac flow for alpine to support future versions and edge.
  • Update apk-tool for riscv64. The new versions are provided from gitlab as apk.static.

cc @janvorli

@am11 am11 force-pushed the feature/rootfs/alpine branch from bc68534 to c365ba9 Compare February 18, 2023 15:13
@am11
Copy link
Member Author

am11 commented Feb 18, 2023

Since the script arguments are position-agnostic, testing requires both direction:

$ eng/common/cross/build-rootfs.sh alpine s390x --rootfsdir /tmp/foo1
$ eng/common/cross/build-rootfs.sh s390x alpine --rootfsdir /tmp/foo2

$ eng/common/cross/build-rootfs.sh alpine3.15 s390x --rootfsdir /tmp/foo3
$ eng/common/cross/build-rootfs.sh s390x alpine3.15 --rootfsdir /tmp/foo4

$ eng/common/cross/build-rootfs.sh alpine3.16 ppc64le --rootfsdir /tmp/foo5
$ eng/common/cross/build-rootfs.sh ppc64le alpine3.16 --rootfsdir /tmp/foo6

$ eng/common/cross/build-rootfs.sh alpineedge riscv64 --rootfsdir /tmp/foo7
$ eng/common/cross/build-rootfs.sh riscv64 alpineedge --rootfsdir /tmp/foo8

# etc.

@am11 am11 force-pushed the feature/rootfs/alpine branch 2 times, most recently from ca3c18b to 83cae8d Compare February 19, 2023 11:49
@am11 am11 force-pushed the feature/rootfs/alpine branch from 83cae8d to f8d0381 Compare February 19, 2023 11:50
@am11 am11 closed this Feb 19, 2023
@am11 am11 reopened this Feb 19, 2023
Copy link
Member

@janvorli janvorli left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@janvorli janvorli merged commit c4a85ad into dotnet:main Feb 20, 2023
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.

3 participants