Skip to content

Conversation

@xFrednet
Copy link
Contributor

This adds the discussed hack to touch clippy_lints/src/lib.rs to invalidate cargos cache and force metadata collection. I've decided to use the filetime crate instead of the touch command to make is cross-platform and just cleaner in general. Looking at the maintainers I would say that it's a save crate to use xD.


cc: #7172 I know this ID without looking it up now... This is not good

changelog: none

r? @flip1995

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label May 20, 2021
Copy link
Member

@flip1995 flip1995 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm good with adding filetime, since it's just another crate by Alex.

@flip1995
Copy link
Member

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented May 25, 2021

📌 Commit e3a1ae7 has been approved by flip1995

@bors
Copy link
Contributor

bors commented May 25, 2021

⌛ Testing commit e3a1ae7 with merge 2d4c2c3...

bors added a commit that referenced this pull request May 25, 2021
…on, r=flip1995

Adding the ability to invalidate caches to force metadata collection

This adds the discussed hack to touch `clippy_lints/src/lib.rs` to invalidate cargos cache and force metadata collection. I've decided to use the [`filetime`](https://github.com/alexcrichton/filetime) crate instead of the touch command to make is cross-platform and just cleaner in general. Looking at the maintainers I would say that it's a save crate to use xD.

---

cc: #7172 I know this ID without looking it up now... This is not good

changelog: none

r? `@flip1995`
@bors
Copy link
Contributor

bors commented May 25, 2021

💥 Test timed out

@flip1995
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented May 25, 2021

⌛ Testing commit e3a1ae7 with merge 6d50cff...

@bors
Copy link
Contributor

bors commented May 25, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 6d50cff to master...

@bors bors merged commit 6d50cff into rust-lang:master May 25, 2021
@xFrednet xFrednet deleted the 7172-trick-cargos-caching-for-collection branch July 28, 2021 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants