Various small fixes and tweaks #30

Closed
wants to merge 5 commits into
from

Projects

None yet

2 participants

@eliask
Contributor
eliask commented Mar 24, 2013

This correctly passes -j/--jobs to Make, adds --load-average as another way to control Make paralleliism, and changes the default to not installing npm (as it is not needed normally). Plus a few small tweaks to wording and such.

eliask added some commits Mar 24, 2013
@eliask eliask Add working options for running Make instances in parallel
This removes the use of the non-working JOBS environment variable and
passes the arguments -j/--jobs to make directly. Additionally, the
Make option --load-average is now also supported.
7f82002
@eliask eliask Skip stand-alone npm installation by default
Node.js has included npm for a while now and it makes little sense to
install it separately by default. Hence, --without-npm has been
changed to --with-npm along with the related logic.

Associated parameter 'sanity checks' that don't make sense anymore
have been dropped as well.
bd267f5
@eliask eliask Small changes to parameter name and description wording 40a0d91
@eliask eliask Use strip() with --requirements instead of replacing just the newlines ce69864
@eliask eliask Bump the version number 5b09915
@ekalinin ekalinin closed this in dce2201 Mar 26, 2013
@ekalinin
Owner

@eliask Thanks for great patch. Applied.

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