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

Match library properly if multiple crate-types are in use. #499

Merged
merged 1 commit into from
Nov 28, 2019

Conversation

jothan
Copy link
Contributor

@jothan jothan commented Nov 28, 2019

Noticed this in my crate rustyknife was flagged as "not a library".

Cargo.toml snippet:
[lib]
crate-type = ["lib", "cdylib"]

Noticed this, my crate rustyknife was flagged as "not a library".

Cargo.toml snippet:
[lib]
crate-type = ["lib", "cdylib"]
@jyn514
Copy link
Member

jyn514 commented Nov 28, 2019

Ah, this is what I was missing in #487 (comment).

Let me check this locally but it looks good!

@jyn514 jyn514 self-assigned this Nov 28, 2019
@jyn514 jyn514 merged commit cfec6ef into rust-lang:master Nov 28, 2019
@jyn514
Copy link
Member

jyn514 commented Nov 28, 2019

Looks great, thanks for the contribution!

@pietroalbini
Copy link
Member

Queued a rebuild for rustyknife 0.2.7!

@jothan
Copy link
Contributor Author

jothan commented Dec 9, 2019

Queued a rebuild for rustyknife 0.2.7!

The rebuild worked ! rustyknife is a library once again. 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.

3 participants