Skip to content

Workspaces #1330

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

Closed
wants to merge 2 commits into from
Closed

Workspaces #1330

wants to merge 2 commits into from

Conversation

nrc
Copy link
Member

@nrc nrc commented Jan 5, 2018

This is mostly moving files around and adjusting paths in response. The idea is for Rustup to have a 'standard' workspaces layout, rather than the weird mix of one workspace plus a bunch of non-workspace crates.

It is failing a test locally (uninstall_self_delete_works), so I thought I'd get some extra eyes on it. It needs some high-level docs about how to build/test before it is ready to land.

r? @alexcrichton @Diggsey

@nrc
Copy link
Member Author

nrc commented Jan 5, 2018

Hmm, I think that test might fail anyway for me

@alexcrichton
Copy link
Member

@bors: r+

👍

@bors
Copy link
Contributor

bors commented Jan 5, 2018

📌 Commit cc043f3 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jan 5, 2018

⌛ Testing commit cc043f3 with merge 6e125ea...

bors added a commit that referenced this pull request Jan 5, 2018

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Workspaces

This is mostly moving files around and adjusting paths in response. The idea is for Rustup to have a 'standard' workspaces layout, rather than the weird mix of one workspace plus a bunch of non-workspace crates.

It is failing a test locally (`uninstall_self_delete_works`), so I thought I'd get some extra eyes on it. It needs some high-level docs about how to build/test before it is ready to land.

r? @alexcrichton  @Diggsey
@bors
Copy link
Contributor

bors commented Jan 5, 2018

💔 Test failed - status-appveyor

@nrc
Copy link
Member Author

nrc commented Jan 8, 2018

     Running target\i686-pc-windows-msvc\release\deps\rustup_msi-a90ae371926d022c.exe
running 0 tests
test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
if defined BUILD_MSI pushd src\rustup-win-installer && cargo build --release --target %TARGET% & popd
The system cannot find the path specified.
Command exited with code 1

@alexcrichton
Copy link
Member

I think src\rustup-win-installer no longer exists?

@bors
Copy link
Contributor

bors commented Jan 10, 2018

☔ The latest upstream changes (presumably #1313) made this pull request unmergeable. Please resolve the merge conflicts.

@bors
Copy link
Contributor

bors commented Feb 13, 2018

☔ The latest upstream changes (presumably #1354) made this pull request unmergeable. Please resolve the merge conflicts.

@nrc
Copy link
Member Author

nrc commented Apr 8, 2018

CLsoing since I haven't had time to rebase and it is now way out of date

@nrc nrc closed this Apr 8, 2018
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.

None yet

3 participants