as many jobs as CPU's #35

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@seebees

if sysctl does not have a value for hw.ncpu it should return null, which is a valid value for -j

And if sysctl does not exist then the value should evaluate to null and something would pop out on stderr, but again -j should be happy.

I do not have a plethora of OS's to test on, so I only tested on lion. But... I did run the following:

$(sysctl -n hw.ncpu)
$(sysctl -n doesNotExist)
$(doesNotExist -n hw.ncpu)

And each one worked for me...

@seebees seebees as many jobs as CPU's
if sysctl does not have a value for hw.ncpu it should return null, which is a valid value for -j

And if sysctl does not exist then the value should evaluate to null and something would pop out on stderr, but again -j should be happy.
8a9e53b
@isaacs isaacs added a commit that closed this pull request May 8, 2012
@isaacs Default jobs to number of CPUs
Close #35
5bc6eac
@isaacs isaacs closed this in 5bc6eac May 8, 2012
@isaacs
Owner

Fixed in a slightly different way. Thanks for the suggestion :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment