ci: make musl.sh look for patches next to the script - #163120
lambdageek wants to merge 1 commit into
Conversation
Instead of hardcoding the container layout assuming that the local patches are in /build, look for them next to the script. This helps downstream distros to run this script if their container layout is different and they don't put the musl working dir in /build. This might be an unexpected change for some dowstream distros that were using this script from another directory,but arranged for the patches to still be in `/build`
|
rustbot has assigned @Mark-Simulacrum. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
Hm, can you say a bit more about what your(?) distro is doing here? I would have somewhat expected that most / all distros want to use their own musl rather than whatever Rust happened to package (we're not necessarily amazing at staying up to date...). If I'm reading the code right it seems pretty harmless to land this though. |
Context: we distribute in-house builds of the Rust toolchain for teams inside Microsoft (see blog - although that post is mostly talking about the special things we're doing on Windows, but we do also build Linux host tools and target several platforms using the normal cg_llvm backend) For some targets, we build musl. When we build musl, we're building it in a step in an Azure Pipelines job, not in a Docker container like the ones upstream has defined, so we don't have an analog of these rust/src/ci/docker/host-x86_64/dist-various-1/Dockerfile Lines 67 to 69 in 88638df We don't have a
The paved path for our internal Linux customers steers them to |
Instead of hardcoding the container layout assuming that the local patches are in
/build, look for them next to the script. This helps downstream distros to run this script if their container layout is different and they don't put the musl working dir in/build.This might be an unexpected change for some downstream distros that were using this script from another directory,but arranged for the patches to still be in
/build