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.

Sign up for GitHub

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

Jump to bottom

the command cargo build --target= panic #3224

Closed
gui1117 opened this issue Oct 24, 2016 · 1 comment · Fixed by #3229
Closed

the command cargo build --target= panic #3224

gui1117 opened this issue Oct 24, 2016 · 1 comment · Fixed by #3229
Labels
E-easy Experience: Easy

Comments

@gui1117
Copy link

gui1117 commented Oct 24, 2016

the command cargo build --target= panic
@alexcrichton
Copy link
Member

I believe the relevant panic is this one and should be relatively easy to remove, just need to punt upwards an error there!

@alexcrichton alexcrichton added the E-easy Experience: Easy label Oct 25, 2016
martinhath pushed a commit to martinhath/cargo that referenced this issue Oct 25, 2016
Add fix and test for rust-lang#3224
84fe070
@martinhath martinhath mentioned this issue Oct 25, 2016
Merged
bors added a commit that referenced this issue Oct 25, 2016
@bors
Auto merge of #3229 - martinhath:master, r=alexcrichton
ad47f0a 
Add fix and test for #3224

This commit fixes #3224, and adds a test for it.
First time contributor to Cargo here, so I'm sorry if I put stuff the wrong places :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
E-easy Experience: Easy
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add fix and test for #3224 martinhath/cargo
2 participants
@alexcrichton @gui1117