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

add crate type #13

Closed
wants to merge 1 commit into from
Closed

add crate type #13

wants to merge 1 commit into from

Conversation

@mahkoh
Copy link

mahkoh commented Aug 24, 2014

No description provided.

@SimonSapin
Copy link
Member

SimonSapin commented Aug 24, 2014

Why?

@mahkoh
Copy link
Author

mahkoh commented Aug 24, 2014

To build it without cargo.

@SimonSapin
Copy link
Member

SimonSapin commented Aug 24, 2014

Why?

@mahkoh
Copy link
Author

mahkoh commented Aug 24, 2014

@SimonSapin
Copy link
Member

SimonSapin commented Aug 24, 2014

Also, you can always run rustc --lib without Cargo.

@mahkoh
Copy link
Author

mahkoh commented Aug 24, 2014

It's --crate-type=lib.

@SimonSapin
Copy link
Member

SimonSapin commented Aug 24, 2014

Yeah, that.

@mahkoh
Copy link
Author

mahkoh commented Aug 24, 2014

You might as well remove crate_name if cargo is a hard dependency for rust-url.

@mahkoh mahkoh closed this Aug 24, 2014
@SimonSapin
Copy link
Member

SimonSapin commented Aug 24, 2014

Good point. I had to keep it at some point because of a Cargo bug, but that’s fixed now. 637a57a.

Cargo is not a hard dependency, you just need whatever other build system you use to pass the right flags to rustc, as Cargo does. (Namely --crate-type and --crate-name.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.