Skip to content

Remove .git directory when downloading a crate from git(hub) #1223

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

Merged
merged 1 commit into from
Mar 23, 2022

Conversation

Kobzol
Copy link
Contributor

@Kobzol Kobzol commented Mar 21, 2022

The .git directory should not be kept there.

@Mark-Simulacrum
Copy link
Member

Do we capture the sha we downloaded into the directory currently? It might be nice to write a git-commit.txt or something with that information, so that there's an easy way to know what was downloaded.

@Kobzol
Copy link
Contributor Author

Kobzol commented Mar 22, 2022

Good idea, I added it.

Copy link
Member

@rylev rylev left a comment

Choose a reason for hiding this comment

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

Should we wait until all the currently open new benchmarks are merged and then add the git-commit.txt to them?

@Kobzol
Copy link
Contributor Author

Kobzol commented Mar 23, 2022

I assume that pretty much all of the (new) benchmarks currently in the repository are downloaded not from git(hub), but from crates.io, which makes it a bit redundant to add the commit to them I suppose.

The download from git is more useful for local profiling and downloading e.g. personal projects from github that are mentioned in rustc issues as regressions.

@rylev
Copy link
Member

rylev commented Mar 23, 2022

Ah good point! I'll merge then 😊

@rylev rylev merged commit 89ebbd3 into rust-lang:master Mar 23, 2022
@Kobzol Kobzol deleted the git-download branch March 24, 2022 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants