rust-marker / marker

An experimental linting interface for Rust. Let's make custom lints a reality
https://rust-marker.github.io/marker/
Other
140 stars 12 forks source link

Fix marker_rustc_driver build ARM just one more time #317

Closed Veetaha closed 9 months ago

Veetaha commented 9 months ago

When last time I fixed the build of marker_rustc_driver on ARM I tested my changes only locally in an isolated folder where I manually "unit-tested" the upload.sh script. Of course, when the build happens on CI the folders layout is entirely different. That's why the ARM packaging script broke but in other way.

On CI we mount the . fodler as /artifacts in quemu, but . folder includes the marker_rustc_driver crate source code folder. So when we do cp target/release/marker_rustc_driver /artifacts/marker_rustc_driver we copy the binary inside of that folder...

I fixed this by mounting ./artifacts which is guaranteed to be an empty folder.

Also manually fixed and reuploaded the artifacts in 0.4:

image

This time I tested it end-to-end in my fork repo

xFrednet commented 9 months ago

Looks good to me, thank you!