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

Support --jobs on publish argument for build parallelism #2860

Closed
adamcrume opened this issue Jul 12, 2016 · 4 comments
Closed

Support --jobs on publish argument for build parallelism #2860

adamcrume opened this issue Jul 12, 2016 · 4 comments
Labels
E-easy Experience: Easy

Comments

@adamcrume
Copy link

cargo build supports a --jobs argument for controlling the build parallelism, and this is very useful for controlling the parallelism of non-Rust builds for *-sys crates. cargo publish should support the same option.

@alexcrichton alexcrichton changed the title Support --jobs argument for build parallelism Support --jobs on publish argument for build parallelism Jul 12, 2016
@alexcrichton
Copy link
Member

Ah good point! This should be relatively easy to do, so I'm tagging with E-easy. This largely just needs to modify src/bin/publish.rs and then plumb the -j option all the way down to where the crate is actually verified. Feel free to ask questions if someone needs help implementing this!

@alexcrichton alexcrichton added the E-easy Experience: Easy label Jul 12, 2016
@mattscamp
Copy link
Contributor

mattscamp commented Jul 13, 2016

I will take it. Just to confirm @alexcrichton, we want to pipe to here?

bors added a commit that referenced this issue Jul 13, 2016
@alexcrichton
Copy link
Member

Indeed! (although it looks like you've already done it)

@brson
Copy link
Contributor

brson commented Jul 22, 2016

Fixed. Thanks @mattscamp !

@brson brson closed this as completed Jul 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-easy Experience: Easy
Projects
None yet
Development

No branches or pull requests

4 participants