-
Notifications
You must be signed in to change notification settings - Fork 5.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ci: run Linux release builds in sysroot with Ubuntu 18.04 and LLVM #10920
Conversation
03bbad7
to
70ce55d
Compare
1e6caa0
to
b8817a7
Compare
git clone --depth 1 --no-checkout \ | ||
https://github.com/rust-lang/crates.io-index \ | ||
~/.cargo/registry/index/github.com-1ecc6299db9ec823 | ||
fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand this. Is this related to the sysroot?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Separated out the commit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this do tho? What is "1ecc6299db9ec823" ? Will it ever need to be updated? I think this could have a comment as to the purpose
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this do tho?
As reflected in the name, it makes a shallow clone of the crates.io index.
What is "1ecc6299db9ec823"
I don't know.
Will it ever need to be updated?
I don't know. But in my Rust-using career it has never changed.
I think this could have a comment as to the purpose
The purpose is to make CI faster. Want me add # This makes CI faster
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Manually clone the cargo crate index shallowly so that cargo doesn't
# clone it deeply. This saves about 1 minute of CI time. We don't believe the
# identifier '1ecc6299db9ec823' will ever change, but if it does then this command
# must be updated.
...by avoiding a full clone of crates.io index history.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
...by avoiding a full clone of crates.io index history.
...by avoiding a full clone of crates.io index history.
...by avoiding a full clone of crates.io index history.
Depends on: denoland/rusty_v8#708