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

Unquote crate names #23777

Closed
wants to merge 1 commit into from
Closed

Unquote crate names #23777

wants to merge 1 commit into from

Conversation

richo
Copy link
Contributor

@richo richo commented Mar 27, 2015

I believe this needs a snapshot to land.

@rust-highfive
Copy link
Collaborator

r? @nrc

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

@richo
Copy link
Contributor Author

richo commented Mar 27, 2015

Just noticed #23738 right on time :)

@nrc
Copy link
Member

nrc commented Mar 27, 2015

@bors: r+ 70e61013b1fb0aa6208162efa0ffefa8a1a6ec55

@@ -12,7 +12,7 @@

// pretty-expanded FIXME #23616

extern crate "xcrate-trait-lifetime-param" as other;
extern crate xcrate-trait-lifetime-param as other;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are hyphens in unquoted crate names here and some other places.

@alexcrichton
Copy link
Member

@bors: r-

I think @petrochenkov is right in that this won't land as-is. Can you also be sure to run all the tests as well?

@richo
Copy link
Contributor Author

richo commented Mar 27, 2015

Sure, sorry I PR'd this and then went to bed. I wasn't expecting it to r+ until that snapshot landed. I'll poke today.

@alexcrichton
Copy link
Member

I've also got a PR where I'm removing parser support, so I can try to just roll this into that as well.

@richo
Copy link
Contributor Author

richo commented Mar 27, 2015

Reran the script to only unquote non-hyphenated crates. r? @alexcrichton

(I think this still needs snaps to land, didn't see if they did)

@rust-highfive rust-highfive assigned alexcrichton and unassigned nrc Mar 27, 2015
@alexcrichton
Copy link
Member

Thanks! I've rolled this into #23786 where I took care of the rest of the tests and also handled the compiler support.

@richo
Copy link
Contributor Author

richo commented Mar 27, 2015

\o/ thanks

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.

5 participants