summaryrefslogtreecommitdiff
path: root/vcbuild.bat
Commit message (Collapse)AuthorAgeFilesLines
* build: change nosign flag to sign and flips logicJoe Doyle2016-12-181-8/+9
| | | | | | | | | | | | | | Makes the default build on Windows not try to sign the node.exe binary after a build. Instead the 'sign' flag now indicates that the binary should be signed. The 'nosign' flag is left as a noop. Reviewed-By: Nikolai Vavilov <vvnicholas@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Bartosz Sosnowski <bartosz@janeasystems.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: João Reis <reis@janeasystems.com> PR-URL: https://github.com/nodejs/node/pull/10156
* test: move tick-processor tests to own directoryRich Trott2016-11-091-0/+1
| | | | | | | | | | | | | | | | The tick-processor tests are inherently non-deterministic. They therefore have false negatives from time to time. They also sometimes leave extra processes running. Move them to their own directory until these issues are sorted. Note that this means that the tests will not be run in CI. Like the inspector tests and other tests, they will have to be run manually when they are wanted. PR-URL: https://github.com/nodejs/node/pull/9506 Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Matthew Loring <mattloring@google.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
* win,build: try multiple timeservers when signingRod Vagg2016-10-191-2/+2
| | | | | | PR-URL: https://github.com/nodejs/node/pull/9155 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: João Reis <reis@janeasystems.com>
* win,tools: ignore linting for line breaksJoão Reis2016-10-071-1/+1
| | | | | | | | | | | | Line breaks on Windows should be CRLF, but Node also supports LF. Hence, do not check line breaks on Windows, when running vcbuild jslint. Fixes: https://github.com/nodejs/node/issues/6912 PR-URL: https://github.com/nodejs/node/pull/8785 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Bartosz Sosnowski <bartosz@janeasystems.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* build: run cctests as part of test-ci targetBen Noordhuis2016-10-031-3/+4
| | | | | | | Enable the cctests on the CI now that they know how to write TAP output. PR-URL: https://github.com/nodejs/node/pull/8034 Reviewed-By: James M Snell <jasnell@gmail.com>
* build: remove VS 2013 switch from vcbuild.batBen Noordhuis2016-09-231-31/+5
| | | | | | | | | | | Support for Visual Studio 2013 has officially been dropped, remove the build option for that compiler. PR-URL: https://github.com/nodejs/node/pull/8067 Refs: https://github.com/nodejs/node/issues/7484 Refs: https://github.com/nodejs/node/pull/8049 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Joao Reis <reis@janeasystems.com>
* inspector: introduce a smoke testEugene Ostroukhov2016-09-191-3/+4
| | | | | | | This test executes a simple debug session over the inspector protocol. PR-URL: https://github.com/nodejs/node/pull/8429 Reviewed-By: ofrobots - Ali Ijaz Sheikh <ofrobots@google.com>
* win,build: forward release_urlbase to configureJoão Reis2016-09-091-2/+1
| | | | | | | | | | | | | The RELEASE_URLBASE environment variable is used in releases as a prefix for links in the process.release object. The Makefile picks it and forwards it to configure, but vcbuild.bat did not. Hence, in Windows, Node releases have a correct process.release because it uses the default URL, but nightlies, RCs and so on do not, breaking node-gyp. This enables native modules to be built with such versions of Node. PR-URL: https://github.com/nodejs/node/pull/8430 Reviewed-By: jbergstroem - Johan Bergström <bugs@bergstroem.nu>
* win,build: exit when addons fail to buildJoão Reis2016-09-081-0/+4
| | | | | PR-URL: https://github.com/nodejs/node/pull/8412 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* win,build: skip finding VS when not neededJoão Reis2016-09-081-0/+2
| | | | | PR-URL: https://github.com/nodejs/node/pull/8412 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* win,build: fail on invalid option in vcbuildJoão Reis2016-09-081-1/+2
| | | | | PR-URL: https://github.com/nodejs/node/pull/8412 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* tools: enable caching for jslint taskRich Trott2016-08-301-1/+1
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/8296 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
* build: add missing files to zip and 7z packagesRichard Lau2016-08-251-0/+12
| | | | | | | | | | Add nodevars.bat, node_etw_provider.man and node_perfctr_provider.man to the zip and 7z packages. PR-URL: https://github.com/nodejs/node/pull/8069 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: joaocgreis - João Reis <reis@janeasystems.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
* build: do not lint src dir for JS errorsRich Trott2016-08-181-2/+2
| | | | | | | | | | | There are no JavaScript files in the `src` directory. It can be safely omitted from the JavaScript linting step. PR-URL: https://github.com/nodejs/node/pull/8128 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: James M Snell <jasnell@gmail.com>
* tools: update to ESLint 3.2.2Rich Trott2016-08-091-1/+1
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/7999 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
* build: windows sharedlib supportStefan Budeanu2016-08-051-0/+3
| | | | | | | | | | | | | | Added "dll" option to vcbuild.bat Insure that Unix SO name is not used on Windows (i.e. produce a .dll file) Insure that Node and its V8 dependency link against the Visual C++ Runtime dynamically. Requires backported V8 patch, see PR 7802. Ref: https://github.com/nodejs/node/pull/7802 PR-URL: https://github.com/nodejs/node/pull/7487 Reviewed-By: Alexis Campailla <alexis@janeasystems.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
* tools: increase lint coverageRich Trott2016-07-131-2/+2
| | | | | | | | | | | | | Extend linting to tools/license2rtf.js and any other JS that gets added to the `tools` directory by default. This incidentally simplifies lint invocation. PR-URL: https://github.com/nodejs/node/pull/7647 Reviewed-By: Michaël Zasso <mic.besace@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
* win,build: add creation of zip and 7z packageBartosz Sosnowski2016-05-311-2/+57
| | | | | | | | | | | | | | | | Add a step in vcbuild.bat to create a minimal package including node and npm that can be used as an alternative to the MSI. Compress the node.pdb file as zip and 7z. All files are uploaded as part of build-release. Reviewed-By: Joao Reis <reis@janeasystems.com> Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Alexis Campailla <orangemocha@nodejs.org> PR-URL: https://github.com/nodejs/node/pull/5995 Fixes: https://github.com/nodejs/build/issues/299 Fixes: https://github.com/nodejs/node/issues/5696
* test: run known_issues tests in CIRich Trott2016-05-061-3/+3
| | | | | | | | | Add `known_issues` tests to `make test` and `make test-ci` targets and their equivalents on Windows. PR-URL: https://github.com/nodejs/node/pull/6559 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>
* tools: Check in tools for shrinking ICU size, change default to small-icuSteven R. Loomis2016-05-041-1/+3
| | | | | | | | | | | | | | * Change configure default to "small-icu" (Intl on, English only) * add "--without-intl" and "vcbuild without-intl" options, equivalent to --with-intl=none * update BUILDING.md with above changes * Checks in tools that generate the deps/icu-small source directory from ICU source * Tools and process for updating ICU documented in tools/icu/README.md Fixes: https://github.com/nodejs/node/issues/3476 PR-URL: https://github.com/nodejs/node/pull/6088 Reviewed-By: James M Snell <jasnell@gmail.com>
* tools: add tests for the doctoolIan Kronquist2016-04-291-2/+2
| | | | | | | | | | | | | | * Test the toHTML function in html.js. Check that given valid markdown it produces the expected html. One test case will prevent regressions of #5873. * Check that when given valid markdown toJSON produces valid JSON with the expected schema. * Add doctool to the list of built in tests so it runs in CI. PR-URL: https://github.com/nodejs/node/pull/6031 Fixes: https://github.com/nodejs/node/issues/5955 Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: Rich Trott <rtrott@gmail.com>
* tools: improve js linterBrian White2016-04-151-1/+8
| | | | | | | | | | | | This commit switches from the eslint command-line tool to a custom tool that uses eslint programmatically in order to perform linting in parallel and to display linting results incrementally instead of buffering them until the end. Fixes: https://github.com/nodejs/node/issues/5596 PR-URL: https://github.com/nodejs/node/pull/5638 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
* win,build: build and test add-ons on test-ciBogdan Lobor2016-03-291-4/+30
| | | | | | | | | | | Added build-addons task, it allows to build and test native addons during test-ci task. Basically it should work in same way like Makefile "build-addons" task. Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: João Reis <reis@janeasystems.com> PR-URL: https://github.com/nodejs/node/pull/5886 Fixes: https://github.com/nodejs/node/issues/2537
* build: remove lint/dotfiles from release tarballJohan Bergström2016-03-161-0/+6
| | | | | | | | | | | | | | | | This removes the ability to run linting from the source tarball, rationale being that developers should use a git clone instead. Also, fix the path of removing artifacts from the openssl dependency since it now lives in `deps/openssl/openssl/`. Tarballs shrink with ~10%. PR-URL: https://github.com/nodejs/node/pull/5695 Fixes: https://github.com/nodejs/node/issues/5618 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: thefourtheye <thechargingvolcano@gmail.com> Reviewed-By: James Snell <jasnell@gmail.com>
* win,build: support Visual C++ Build Tools 2015João Reis2016-03-111-1/+3
| | | | | | | Invoke MSBuild specifying the target platform as generated by Gyp. Reviewed-By: James M Snell <jasnell@gmail.com> PR-URL: https://github.com/nodejs/node/pull/5627
* tools: support testing known issuescjihrig2016-03-041-0/+1
| | | | | | | | | | | | | This commit adds a known_issues directory to the test directory for scripts that reproduce known bugs. Since these scripts are expected to fail, it also adds a --expect-fail flag to test.py which reports tests as successful when they fail. Refs: https://github.com/nodejs/testing/issues/18 PR-URL: https://github.com/nodejs/node/pull/5528 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Wyatt Preul <wpreul@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
* tools: enable linting for benchmarksRich Trott2016-03-031-1/+1
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/5517 Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Michaël Zasso <mic.besace@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* build: remove --quiet from eslint invocationfiredfox2016-03-021-1/+1
| | | | | | | | | | | | All eslint rules are configured to report as errors. Remove useless --quiet flag from eslint invocation in Makefile and vcbuild.bat. PR-URL: https://github.com/nodejs/node/pull/5519 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Michaël Zasso <mic.besace@gmail.com> Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
* build: skip msi build if WiX is not foundTsarevich Dmitry2016-02-161-1/+1
| | | | | | | | | | If none of VS2013/VS2015 WiX installs are found on a builder machine, skip node build and notify user about. PR-URL: https://github.com/nodejs/node/pull/5220 Reviewed-By: João Reis <reis@janeasystems.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rod Vagg <rod@vagg.org>
* tools: apply linting to doc toolsRich Trott2016-02-011-1/+1
| | | | | | | | | Apply eslint rules to `tools/doc`. PR-URL: https://github.com/nodejs/node/pull/4973 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
* build: add option to select VS versionjulien.waechter2016-01-141-1/+6
| | | | | | | | This changes vcbuild.bat to accept a new parameter (vc2015 or vc2013) to select the version of Visual Studio to use. PR-URL: https://github.com/nodejs/node/pull/4645 Reviewed-By: João Reis <reis@janeasystems.com>
* tools: update eslint configMichaël Zasso2016-01-131-1/+1
| | | | | | | | | | The no-reserved-keys rule doesn't exist anymore and we don't need ES3 compatibility. escape and unescape are now known by eslint. --reset flag was removed and it is now the default behavior. PR-URL: https://github.com/nodejs/io.js/pull/2286 Reviewed-By: Roman Reiss <me@silverwind.io>
* build: fix and refactor VTune config in vcbuild.batRod Vagg2016-01-131-5/+6
| | | | | | PR-URL: https://github.com/nodejs/node/pull/4192 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com>
* src: refactor vcbuild configure args creationRod Vagg2015-12-081-14/+14
| | | | | | | | | | | remove a bunch of variables and rely on %configure_flags% where possible, also allow for an external %config_flags% variable to supply additional arguments to configure to match the behaviour of the Makefile PR-URL: https://github.com/nodejs/node/pull/3399 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: João Reis <reis@janeasystems.com>
* build: update signtool description, add urlRod Vagg2015-11-301-2/+2
| | | | | | PR-URL: https://github.com/nodejs/node/pull/4011 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Alexis Campailla <orangemocha@nodejs.org>
* build,src: add Intel Vtune profiling supportChunyang Dai2015-11-251-2/+5
| | | | | | | | | | | | | | | | | | This feature supports the Intel Vtune profiling support for JITted JavaScript on IA32 / X64 / X32 platform. The advantage of this profiling is that the user / developer of NodeJS application can get the detailed profiling information for every line of the JavaScript source code. This information will be very useful for the owner to optimize their applications. This feature is a compile-time option. For windows platform, the user needs to pass the following parameter to vcbuild.bat: "enable-vtune" For other OS, the user needs to pass the following parameter to ./configure command: "--enable-vtune-profiling" PR-URL: https://github.com/nodejs/node/pull/3785 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* build: Intl: deps: bump ICU to 56.1 (GA)Steven R. Loomis2015-10-081-0/+5
| | | | | | | | | | | | | | | | | | | | * ICU 56 was just released yesterday. Update to it. * Notable changes: Unicode 8, CLDR 28, 2-3x number format perf, 20% improvement in Collator startup * more at http://site.icu-project.org/download/56 or in #2917 Also: * cleanup out/**/*.d and deps/icu on "make clean" * cleanup deps/icu on "vcbuild clean" When building from an non-clean directory, it's important to run `make clean` or `vcbuild clean` to remove the existing ICU 55 from the deps path before building. Fixes: https://github.com/nodejs/node/issues/2917 PR-URL: https://github.com/nodejs/node/pull/3281 Reviewed-By: James M Snell <jasnell@gmail.com>
* tools: apply linting to custom rules codeRich Trott2015-10-061-1/+1
| | | | | | | | | | | Apply our eslint rules to the code for our custom eslint rules. Eslint Inception! Maybe. I never saw that movie... PR-URL: https://github.com/nodejs/node/pull/3195 Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Roman Reiss <me@silverwind.io>
* build: remote commands on staging in single sessionRod Vagg2015-09-081-2/+1
| | | | | PR-URL: https://github.com/nodejs/node/pull/2717 Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
* build: fix borked chmod call for release uploadsRod Vagg2015-09-021-1/+1
| | | | | PR-URL: https://github.com/nodejs/node/pull/2645 Reviewed-By: thefourtheye - Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
* build: set file permissions before uploadingRod Vagg2015-09-011-0/+1
| | | | | PR-URL: https://github.com/nodejs/node/pull/2623 Reviewed-By: jbergstroem - Johan Bergström <bugs@bergstroem.nu>
* build: change staging directory on new serverRod Vagg2015-09-011-5/+5
| | | | | | | | new server has "nodejs" and "iojs" directories, upload to the appropriate one PR-URL: https://github.com/nodejs/node/pull/2623 Reviewed-By: jbergstroem - Johan Bergström <bugs@bergstroem.nu>
* build: fix bad cherry-pick for vcbuild.bat build-releaseRod Vagg2015-09-011-5/+5
| | | | | | PR-URL: https://github.com/nodejs/node/pull/2625 Reviewed-By: orangemocha - Alexis Campailla <orangemocha@nodejs.org> Reviewed-By: joaocgreis - João Reis <reis@janeasystems.com>
* test: pass args to test-ci via env variableAlexis Campailla2015-08-271-1/+1
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/2424 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: João Reis <reis@janeasystems.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
* test: support flaky tests in test-ciAlexis Campailla2015-08-271-0/+1
| | | | | | | | | | | | | | Adding support for specifying flaky test mode to the test runner: - via an environment variable FLAKY_TESTS for Makefile - via an argument ignore-flaky for vcbuild.bat Ported from https://github.com/joyent/node/commit/2d2494cf140c38327218a087593ff2177a9d0ec9 PR-URL: https://github.com/nodejs/node/pull/2424 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: João Reis <reis@janeasystems.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
* win: fix custom actions on Visual Studio != 2013Julien Gilli2015-08-241-1/+27
| | | | | | | | | | | | | | | | | | | | | | This is a port of joyent/node@16bcd68dc5f449c331499ced22aa626dbb2cdefc . Original commit message: The original change that added support for running custom actions during the install process (joyent/node@e7c84f82c730bddb7450dc56065aab11b93ffd30) assumed that Visual Studio 2013 is used to generate the installer file. However, that is not always the case, and older versions of Visual Studio should allow users to generate Windows installer files. This change makes the custom actions visual studio project use the visual studio version that is found by vcbuild.bat. Reviewed-By: João Reis <reis@janeasystems.com> PR-URL: https://github.com/joyent/node/pull/25569 PR-URL: https://github.com/nodejs/node/pull/2365 Reviewed-By: Alexis Campailla <alexis@janeasystems.com>
* node: rename from io.js to nodecjihrig2015-08-231-12/+12
| | | | | | | | | | | This commit replaces instances of io.js with Node.js, based on the recent convergence. There are some remaining instances of io.js, related to build and the installer. Fixes: https://github.com/nodejs/node/issues/2361 PR-URL: https://github.com/nodejs/node/pull/2367 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: João Reis <reis@janeasystems.com>
* build: intl: converge from joyent/nodeSteven R. Loomis2015-08-131-1/+14
| | | | | | | | | | | | | | | | | | | | * turn on small-icu by default for builds (Makefile+Windows) * add license info from ICU http://source.icu-project.org/repos/icu/icu/trunk/license.html All text pasted. Long lines wrapped. (original is HTML.) Port from joyent/node of: * https://github.com/nodejs/node/issues/26 * port of joyent/node 67f87a743dee4951374dd240c25e31cd507e6c5f * https://github.com/joyent/node/pull/9038 * Merge from joyent/node 70d04e7f35667381aa6e9d76f56ea962c019d5bc * Merge from joyent/node 6168fe6720650052728f3e78a495b723f0b41ce3 * merge from joyent/node e67073256d847be085be7049cd1d6ebd8d86ac62 PR-URL: https://github.com/nodejs/node/pull/2264 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rod Vagg <rod@vagg.org> Fixes: https://github.com/nodejs/node/issues/238
* src: introduce process.release objectRod Vagg2015-08-041-0/+2
| | | | | | PR-URL: https://github.com/nodejs/io.js/pull/2154 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
* build: fix vcbuild merge issuesAlexis Campailla2015-07-101-12/+0
| | | | | | | | | | configure was getting called twice. We also erroneously introduced support for VS2010, and were picking 2010 before other versions. PR-URL: https://github.com/nodejs/io.js/pull/2131 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Johan Bergström <bugs@bergstroem.nu>