Skip to content
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

Disable preserving mtimes on archives #7465

Merged
merged 1 commit into from
Oct 2, 2019

Conversation

alexcrichton
Copy link
Member

These are just wasted syscalls for our purposes, no need to issue
updates to the modification/creation/access times of files we unpack!

These are just wasted syscalls for our purposes, no need to issue
updates to the modification/creation/access times of files we unpack!
@rust-highfive
Copy link

r? @nrc

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 1, 2019
@Eh2406
Copy link
Contributor

Eh2406 commented Oct 2, 2019

Good catch.

@bors r+

@bors
Copy link
Collaborator

bors commented Oct 2, 2019

📌 Commit 8c92e88 has been approved by Eh2406

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 2, 2019
@bors
Copy link
Collaborator

bors commented Oct 2, 2019

⌛ Testing commit 8c92e88 with merge 053b767...

bors added a commit that referenced this pull request Oct 2, 2019
Disable preserving mtimes on archives

These are just wasted syscalls for our purposes, no need to issue
updates to the modification/creation/access times of files we unpack!
@bors
Copy link
Collaborator

bors commented Oct 2, 2019

☀️ Test successful - checks-azure
Approved by: Eh2406
Pushing 053b767 to master...

@bors bors merged commit 8c92e88 into rust-lang:master Oct 2, 2019
@bors bors deleted the no-preserve branch October 2, 2019 15:55
bors added a commit to rust-lang/rust that referenced this pull request Oct 8, 2019
Update Cargo

To pull rust-lang/cargo#7482

List of merged PRs:
- Fix wrong directories in PATH on Windows (rust-lang/cargo#7482)
- Update SPDX list to 3.6 (rust-lang/cargo#7481)
- Mark Emscripten's .wasm files auxiliary (rust-lang/cargo#7476)
- Update `curl-sys` dependency requirement (rust-lang/cargo#7464)
- add dependencies for `pkg-config` (rust-lang/cargo#7443)
- Removing hash from output files when using MSVC (rust-lang/cargo#7400)
- Disable preserving mtimes on archives (rust-lang/cargo#7465)
- Removed redundant borrow (rust-lang/cargo#7462)
- Public dependency refactor and re-allow backjumping (rust-lang/cargo#7361)
- unify the quote in Cargo.toml (rust-lang/cargo#7461)
bors-servo pushed a commit to servo/libfontconfig that referenced this pull request Dec 5, 2019
Fix intermittent build error

This is a symptom of a change in Cargo (rust-lang/cargo#7465) that no longer preserves the mtime of files extracted during the package download process. This means that generated files included in this repository can end up being regenerated unnecessarily unless we explicitly force them to be treated as more recent than their source files.
@ivanhernandez
Copy link

That happened to be the solution for #6238 issue, so no Rust compiling from FAT devices again.

It broke my Rust from USB environment :)

@ehuss ehuss added this to the 1.40.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants