Skip to content

trpl: Clarify lib.rs vs main.rs #24860

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

Merged
merged 1 commit into from
Apr 27, 2015
Merged

Conversation

Manishearth
Copy link
Member

A user in IRC was having trouble because they used main.rs when they were trying
to migrate a library. The [[lib]] key is not easily found, and the main.rs/lib.rs
distinction doesn't seem to exist in trpl

r? @steveklabnik

Note that since we're creating an executable, we used `main.rs`. If we
want to make a compiled library instead, we should use `lib.rs`.
Custom file locations for the entry point can be specified
with a [`[[lib]]` or `[[bin]]`][crates-custom] key in the toml file described below.
Copy link
Member

Choose a reason for hiding this comment

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

TOML should be in all caps

@Manishearth Manishearth force-pushed the cargo-doc branch 2 times, most recently from eec378d to 38042d1 Compare April 27, 2015 13:11
@Manishearth
Copy link
Member Author

fixed.

@steveklabnik
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented Apr 27, 2015

📌 Commit 38042d1 has been approved by steveklabnik

@steveklabnik
Copy link
Member

Thanks! :)

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Apr 27, 2015
A user in IRC was having trouble because they used `main.rs` when they were trying
to migrate a library. The `[[lib]]` key is not easily found, and the `main.rs`/`lib.rs`
distinction doesn't seem to exist in trpl


r? @steveklabnik
bors added a commit that referenced this pull request Apr 27, 2015
@bors bors merged commit 38042d1 into rust-lang:master Apr 27, 2015
@Manishearth Manishearth deleted the cargo-doc branch September 3, 2015 17:46
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.

3 participants