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

Redesign features #1294

Closed
wants to merge 2 commits into from
Closed

Redesign features #1294

wants to merge 2 commits into from

Conversation

sfackler
Copy link
Member

Note

A crates.io update needs to be made before this can merge. I'm opening this now to get feedback.

Closes #1286.

r? @alexcrichton

return HashMap::new();
}

raw.as_ref().unwrap().iter().map(|(name, desc)| {
Copy link
Member

Choose a reason for hiding this comment

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

Could this unwrap be transformed into:

let raw = match *raw {
    Some(ref raw) => raw,
    None => return HashMap::new()
};

@alexcrichton
Copy link
Member

This looks all good to me, thanks @sfackler! I think that we're going to want to be more aggressive about a transition plan, however, instead of just breaking the old behavior. Would it be possible for Feature to be an enumeration of LegacyFeature and NewFeature or something like that for now? At the very least we should provide nice error messages nudging crates in the right direction for now (or perhaps warnings for the time being). Ideally we could have both systems working in tandem, although I'm not totally sure as to the feasibility of such.

@alexcrichton
Copy link
Member

I can also work more on the crates.io-half of the features here if you'd like, although it may have to wait until after alpha2 depending on what else needs to land.

@alexcrichton
Copy link
Member

Closing due to inactivity, but I'd still be interested in a change along these lines happening!

@sfackler
Copy link
Member Author

sfackler commented Sep 9, 2015

RIP

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.

RFC: Remove the implicit features associated with dependencies
2 participants