Cleanup to librustc::session and related code#53073
Conversation
|
r? @pnkfelix (rust_highfive has picked a reviewer for you, use r? to override) |
d9f994e to
49c6e33
Compare
|
@bors r+ |
|
📌 Commit 49c6e33ee236b6ebbfcda57b4fc103bd235a1351 has been approved by |
|
🔒 Merge conflict This pull request and the master branch diverged in a way that cannot be automatically merged. Please rebase on top of the latest master branch, and let the reviewer approve again. How do I rebase?Assuming
You may also read Git Rebasing to Resolve Conflicts by Drew Blessing for a short tutorial. Please avoid the "Resolve conflicts" button on GitHub. It uses Sometimes step 4 will complete without asking for resolution. This is usually due to difference between how Error message |
49c6e33 to
b3510b0
Compare
|
@bors r=pnkfelix |
|
📌 Commit b3510b08c559cbccb740e61030e8a97b7dd292a2 has been approved by |
b3510b0 to
488726c
Compare
|
Dropped the last commit to try and avoid conflicts @bors r=pnkfelix |
|
📌 Commit 488726c094a645388752b0d09971ecb8743ce9dd has been approved by |
488726c to
a8e19be
Compare
|
@bors r=pnkfelix |
|
📌 Commit a8e19be has been approved by |
Cleanup to librustc::session and related code No functional changes, just some cleanup. This also creates the `rustc_fs_util` crate, but I can remove that change if desired. It felt a little odd to force crates to depend on librustc for some fs utilities; and also seemed good to generally keep the size of librustc lower (for compile times); fs_util will compile in parallel with essentially the first crate since it has no dependencies beyond std.
|
☀️ Test successful - status-appveyor, status-travis |
No functional changes, just some cleanup.
This also creates the
rustc_fs_utilcrate, but I can remove that change if desired. It felt a little odd to force crates to depend on librustc for some fs utilities; and also seemed good to generally keep the size of librustc lower (for compile times); fs_util will compile in parallel with essentially the first crate since it has no dependencies beyond std.