Skip to content

Commit

Permalink
doc: correct vcbuild options for windows testing
Browse files Browse the repository at this point in the history
Corrected parameter for running tests on Windows. Without the corrected
parameters, Windows users encounter an error about failing to sign the
build, "Failed to sign exe", which can be discouraging to new Windows
community members.

PR-URL: nodejs#10686
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Bartosz Sosnowski <bartosz@janeasystems.com>
  • Loading branch information
jboarman authored and joaocgreis committed Feb 9, 2017
1 parent 5b98fd0 commit 6034bdc
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
10 changes: 5 additions & 5 deletions BUILDING.md
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ Prerequisites:
To run the tests:

```text
> .\vcbuild test
> .\vcbuild nosign test
```

To test if Node.js was built correctly:
Expand Down Expand Up @@ -172,7 +172,7 @@ $ ./configure --with-intl=small-icu --download=all
##### Windows:

```text
> .\vcbuild small-icu download-all
> .\vcbuild nosign small-icu download-all
```

The `small-icu` mode builds with English-only data. You can add full
Expand All @@ -195,7 +195,7 @@ $ ./configure --with-intl=full-icu --download=all
##### Windows:

```text
> .\vcbuild full-icu download-all
> .\vcbuild nosign full-icu download-all
```

#### Building without Intl support
Expand All @@ -212,7 +212,7 @@ $ ./configure --with-intl=none
##### Windows:

```text
> .\vcbuild intl-none
> .\vcbuild nosign intl-none
```

#### Use existing installed ICU (Unix / OS X only):
Expand Down Expand Up @@ -251,7 +251,7 @@ First unpack latest ICU to `deps/icu`
as `deps/icu` (You'll have: `deps/icu/source/...`)

```text
> .\vcbuild full-icu
> .\vcbuild nosign full-icu
```

## Building Node.js with FIPS-compliant OpenSSL
Expand Down
6 changes: 3 additions & 3 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -164,19 +164,19 @@ $ ./configure && make -j8 test
Windows:

```text
> vcbuild test
.\vcbuild nosign test
```

(See the [BUILDING.md](./BUILDING.md) for more details.)

Make sure the linter is happy and that all tests pass. Please, do not submit
patches that fail either check.

Running `make test`/`vcbuild test` will run the linter as well unless one or
Running `make test`/`.\vcbuild nosign test` will run the linter as well unless one or
more tests fail.

If you want to run the linter without running tests, use
`make lint`/`vcbuild jslint`.
`make lint`/`.\vcbuild nosign jslint`.

If you are updating tests and just want to run a single test to check it, you
can use this syntax to run it exactly as the test harness would:
Expand Down

0 comments on commit 6034bdc

Please sign in to comment.