Skip to content

Support both --bin and --lib together in cargo new #5433

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

Closed
wants to merge 2 commits into from

Conversation

kornelski
Copy link
Contributor

Fixes #5431

@rust-highfive
Copy link

r? @matklad

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

@matklad
Copy link
Member

matklad commented May 2, 2018

We've discussed this at the Cargo meeting, and the conclusion was that, while today src/lib.rs + src/main.rs is the default pattern for applications, we are actually not sure that this is the proper way to structure code, and not just a tradition and a historical accident.

So, we are not ready to promote this pattern even more, until we decide that this is indeed a good pattern to promote. The discussion about that happens at rust-lang/api-guidelines#167.

Let's keep this PR open, in case that that discussion ends quickly and in favor of the current pattern!

@alexcrichton
Copy link
Member

I'm gonna close this in favor of rust-lang/api-guidelines#167 for now as it's been awhile, but we can of course reopen if the discussion comes to a close!

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