Skip to content
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

Fix depending on git repos with workspaces #2938

Merged
merged 1 commit into from
Aug 1, 2016

Conversation

alexcrichton
Copy link
Member

When we're recursively walking over a git repository we can safely ignore any
workspace Cargo.toml files we find instead of generating an error.

@alexcrichton
Copy link
Member Author

r? @brson

@rust-highfive
Copy link

@alexcrichton: no appropriate reviewer found, use r? to override

@brson
Copy link
Contributor

brson commented Jul 31, 2016

@bors r+

Wonder what rust-highfive is confused about.

@bors
Copy link
Contributor

bors commented Jul 31, 2016

📌 Commit 1d6b6f6 has been approved by brson

@bors
Copy link
Contributor

bors commented Jul 31, 2016

⌛ Testing commit 1d6b6f6 with merge 6292654...

@bors
Copy link
Contributor

bors commented Jul 31, 2016

💔 Test failed - cargo-mac-64

When we're recursively walking over a git repository we can safely ignore any
workspace Cargo.toml files we find instead of generating an error.
@alexcrichton
Copy link
Member Author

@bors: r=brson

@bors
Copy link
Contributor

bors commented Aug 1, 2016

📌 Commit 9243f06 has been approved by brson

@bors
Copy link
Contributor

bors commented Aug 1, 2016

⌛ Testing commit 9243f06 with merge 1ceb0c9...

bors added a commit that referenced this pull request Aug 1, 2016
Fix depending on git repos with workspaces

When we're recursively walking over a git repository we can safely ignore any
workspace Cargo.toml files we find instead of generating an error.
@bors
Copy link
Contributor

bors commented Aug 1, 2016

☀️ Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: brson
Pushing 1ceb0c9 to master...

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.

4 participants