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.

Sign up for GitHub

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

Jump to bottom

build: make ci test addons in test/addons #2428

Merged
merged 2 commits into from
Aug 25, 2015
Merged

build: make ci test addons in test/addons #2428

merged 2 commits into from
Aug 25, 2015

Commits on Aug 25, 2015

  1. build: make ci test addons in test/addons 

    Make `make test-addons` part of the `make test-ci` target.

Use order-only prerequisites to make generating and building the add-ons
concurrency-safe when $JOBS > 1 and fudge the dependency on $(NODE_EXE)
so that add-ons are only rebuilt when needed instead of all the time.

PR-URL: nodejs#2428
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Rod Vagg <rod@vagg.org>
    @bnoordhuis
    bnoordhuis committed Aug 25, 2015
    Configuration menu
    Copy the full SHA
    1aa9d3a View commit details
    Browse the repository at this point in the history

  2. tools: make add-on scraper print filenames 

    Make the tool that generates add-ons from doc/api/addons.markdown print
the names of the files it writes out.  Before this commit, it printed a
rather unhelpful "Done."

PR-URL: nodejs#2428
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Rod Vagg <rod@vagg.org>
    @bnoordhuis
    bnoordhuis committed Aug 25, 2015
    Configuration menu
    Copy the full SHA
    a2e4341 View commit details
    Browse the repository at this point in the history