search

softdevteam / ykrustc

Yorick Meta-tracer
Other
6 stars 4 forks source link

Temporary fix for CI #163

Closed vext01 closed 3 years ago

vext01 commented 3 years ago

x.py install seems to be broken at the moment and this in turn broke our CI. This branch proposes a workaround.

The first commit is the best "easy" fix I can come up with. I'd like this to be a temporary workaround, as it's not optimal: copying the stage 2 dir copies a lot more stuff than you'd find in a typical sysroot. We could try and remove the bits we don't need, but it's difficult to know what, and we'd have to keep it in sync with upstream. I'm hoping upstream will fix x.py install soon...

Tested on https://github.com/softdevteam/yk/pull/206. And here is the forced build log.

The second commit allows us to run .buildbot.sh in scenarios where there isn't necessarily a bors merge commit on the top of the branch (i.e. during a manual or forced build). I needed this to test my changes and it seems generally useful.

ltratt commented 3 years ago

Please squash.

vext01 commented 3 years ago

Splat.

vext01 commented 3 years ago

Hold on a minute. I think there's a bug in the second commit.

vext01 commented 3 years ago

Sorry, that's rubbish, please carry on :)

ltratt commented 3 years ago

bors r+

bors[bot] commented 3 years ago

Build succeeded: