Skip to content

Conversation

strega-nil
Copy link
Contributor

No description provided.

@rust-highfive
Copy link
Contributor

r? @pnkfelix

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

@aidanhs
Copy link
Contributor

aidanhs commented May 3, 2017

Hi @ubsan, looks like this has failed travis because the Cargo.lock needs updating as well?

@aidanhs aidanhs added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label May 3, 2017
@strega-nil
Copy link
Contributor Author

@aidanhs ah, dang

@strega-nil
Copy link
Contributor Author

Hopefully that fixes it.

Copy link
Contributor

Choose a reason for hiding this comment

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

nit: indentation is weird here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh, right, I use a different style from rustc.

@kennytm
Copy link
Member

kennytm commented May 4, 2017

Could some features from libsyntax_pos (a dependency of libsyntax) be removed as well?

/rust/src/libsyntax_pos$ git grep '\[feature'
lib.rs:26:#![feature(const_fn)]
lib.rs:27:#![feature(custom_attribute)]
lib.rs:28:#![feature(optin_builtin_traits)]
lib.rs:30:#![feature(rustc_private)]
lib.rs:31:#![feature(staged_api)]
lib.rs:32:#![feature(specialization)]

@strega-nil
Copy link
Contributor Author

strega-nil commented May 4, 2017

@kennytm it's a lot harder. The only one you can delete without changing quite a bit of code is specialization, which can just be deleted.

@kennytm
Copy link
Member

kennytm commented May 4, 2017

@ubsan I see, thanks. I thought they are pulled in also due to bitflags.

@Mark-Simulacrum Mark-Simulacrum added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels May 7, 2017
@Mark-Simulacrum
Copy link
Member

I think this is done, though commits could be squashed.

@strega-nil
Copy link
Contributor Author

@Mark-Simulacrum done

@nrc
Copy link
Member

nrc commented May 7, 2017

@bors: r+

@bors
Copy link
Collaborator

bors commented May 7, 2017

📌 Commit 0be8758 has been approved by nrc

@bors
Copy link
Collaborator

bors commented May 7, 2017

⌛ Testing commit 0be8758 with merge 9956e81...

bors added a commit that referenced this pull request May 7, 2017
Delete features which are easily removed, in libsyntax
@bors
Copy link
Collaborator

bors commented May 8, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: nrc
Pushing 9956e81 to master...

@bors bors merged commit 0be8758 into rust-lang:master May 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants