Skip to content

Commit

Permalink
doc: update BUILDING.md
Browse files Browse the repository at this point in the history
Added note about vcbuild being included as batch to clarify
that it's not needed from msbuild tools or visual studio.

PR-URL: #8704
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ilkka Myller <ilkka.myller@nodefield.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
  • Loading branch information
rainabba authored and MylesBorins committed Feb 21, 2017
1 parent d61c181 commit d3c5bc1
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions BUILDING.md
Original file line number Diff line number Diff line change
Expand Up @@ -117,13 +117,13 @@ Prerequisites:
and tools which can be included in the global `PATH`.

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

To run the tests:

```text
> vcbuild test
> .\vcbuild 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 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 full-icu download-all
```

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

```text
> vcbuild intl-none
> .\vcbuild 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 full-icu
```

## Building Node.js with FIPS-compliant OpenSSL
Expand Down

0 comments on commit d3c5bc1

Please sign in to comment.