Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix a number of bugs with cargo clean #2131

Merged
merged 1 commit into from
Nov 15, 2015

Conversation

alexcrichton
Copy link
Member

  • Clean out both host/target platforms if any are available.
  • Clean out build script output.
  • Add a --release flag to also be able to clean out release folder
  • Be sure to clean out benchmarks, tests, etc.
  • Cleaning out registry packages no longer panics

Closes #2121

@rust-highfive
Copy link

r? @wycats

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

@alexcrichton
Copy link
Member Author

r? @brson

@brson
Copy link
Contributor

brson commented Nov 13, 2015

@bors r+

@bors
Copy link
Contributor

bors commented Nov 13, 2015

📌 Commit 98ac931 has been approved by brson

@bors
Copy link
Contributor

bors commented Nov 13, 2015

⌛ Testing commit 98ac931 with merge 1e5fdc7...

@bors
Copy link
Contributor

bors commented Nov 13, 2015

💔 Test failed - cargo-mac-32

* Clean out both host/target platforms if any are available.
* Clean out build script output.
* Add a --release flag to also be able to clean out release folder
* Be sure to clean out benchmarks, tests, etc.
* Cleaning out registry packages no longer panics

Closes rust-lang#2121
@alexcrichton
Copy link
Member Author

@bors: r=brson 595ac1d

@bors
Copy link
Contributor

bors commented Nov 15, 2015

⌛ Testing commit 595ac1d with merge c41e05f...

bors added a commit that referenced this pull request Nov 15, 2015
* Clean out both host/target platforms if any are available.
* Clean out build script output.
* Add a --release flag to also be able to clean out release folder
* Be sure to clean out benchmarks, tests, etc.
* Cleaning out registry packages no longer panics

Closes #2121
@bors
Copy link
Contributor

bors commented Nov 15, 2015

@bors bors merged commit 595ac1d into rust-lang:master Nov 15, 2015
@alexcrichton alexcrichton deleted the cargo-clean branch January 20, 2016 18:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants