Skip to content

Commit c4d0ca4

Browse files
avivkellertargos
authored andcommittedOct 2, 2024
doc, build: fixup build docs
PR-URL: #54899 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
1 parent 3011927 commit c4d0ca4

File tree

1 file changed

+14
-11
lines changed

1 file changed

+14
-11
lines changed
 

Diff for: ‎BUILDING.md

+14-11
Original file line numberDiff line numberDiff line change
@@ -210,8 +210,7 @@ For use of AVX2,
210210
* llvm version 3.3 or higher
211211
* nasm version 2.10 or higher in Windows
212212

213-
Please refer to
214-
<https://www.openssl.org/docs/man1.1.1/man3/OPENSSL_ia32cap.html> for details.
213+
Please refer to <https://docs.openssl.org/1.1.1/man3/OPENSSL_ia32cap/> for details.
215214

216215
If compiling without one of the above, use `configure` with the
217216
`--openssl-no-asm` flag. Otherwise, `configure` will fail.
@@ -529,7 +528,7 @@ $ gdb /opt/node-debug/node core.node.8.1535359906
529528
[ASan](https://github.com/google/sanitizers) can help detect various memory
530529
related bugs. ASan builds are currently only supported on linux.
531530
If you want to check it on Windows or macOS or you want a consistent toolchain
532-
on Linux, you can try [Docker](https://www.docker.com/products/docker-desktop)
531+
on Linux, you can try [Docker](https://www.docker.com/products/docker-desktop/)
533532
(using an image like `gengjiawen/node-build:2020-02-14`).
534533

535534
The `--debug` is not necessary and will slow down build and testing, but it can
@@ -622,7 +621,11 @@ vcpkg owns zlib1.dll
622621
vcpkg integrate remove
623622
```
624623

625-
Refs: #24448, <https://github.com/microsoft/vcpkg/issues/37518>, [vcpkg](https://github.com/microsoft/vcpkg/)
624+
Refs:
625+
626+
1. <https://github.com/nodejs/node/issues/24448>
627+
2. <https://github.com/microsoft/vcpkg/issues/37518> / <https://github.com/microsoft/vcpkg/discussions/37546>
628+
3. [vcpkg](https://github.com/microsoft/vcpkg/)
626629

627630
#### Windows Prerequisites
628631

@@ -673,9 +676,9 @@ packages:
673676
* [NetWide Assembler](https://chocolatey.org/packages/nasm)
674677

675678
To install Node.js prerequisites using
676-
[Boxstarter WebLauncher](https://boxstarter.org/weblauncher), open
679+
[Boxstarter WebLauncher](https://boxstarter.org/weblauncher), visit
677680
<https://boxstarter.org/package/nr/url?https://raw.githubusercontent.com/nodejs/node/HEAD/tools/bootstrap/windows_boxstarter>
678-
with Edge browser on the target machine.
681+
with a supported browser.
679682

680683
Alternatively, you can use PowerShell. Run those commands from
681684
an elevated (Administrator) PowerShell terminal:
@@ -750,7 +753,7 @@ architecture supports \[arm, arm64/aarch64, x86, x86\_64].
750753

751754
## `Intl` (ECMA-402) support
752755

753-
[Intl](https://github.com/nodejs/node/blob/HEAD/doc/api/intl.md) support is
756+
[Intl](doc/api/intl.md) support is
754757
enabled by default.
755758

756759
### Build with full ICU support (all locales supported by ICU)
@@ -816,7 +819,7 @@ that works for both your host and target environments.
816819
### Build with a specific ICU
817820

818821
You can find other ICU releases at
819-
[the ICU homepage](http://site.icu-project.org/download).
822+
[the ICU homepage](https://icu.unicode.org/download).
820823
Download the file named something like `icu4c-**##.#**-src.tgz` (or
821824
`.zip`).
822825

@@ -847,7 +850,7 @@ From a tarball URL:
847850
#### Windows
848851

849852
First unpack latest ICU to `deps/icu`
850-
[icu4c-**##.#**-src.tgz](http://site.icu-project.org/download) (or `.zip`)
853+
[icu4c-**##.#**-src.tgz](https://icu.unicode.org/download) (or `.zip`)
851854
as `deps/icu` (You'll have: `deps/icu/source/...`)
852855

853856
```powershell
@@ -870,10 +873,10 @@ configure option:
870873
## Building Node.js with FIPS-compliant OpenSSL
871874

872875
Node.js supports FIPS when statically or dynamically linked with OpenSSL 3 via
873-
[OpenSSL's provider model](https://www.openssl.org/docs/man3.0/man7/crypto.html#OPENSSL-PROVIDERS).
876+
[OpenSSL's provider model](https://docs.openssl.org/3.0/man7/crypto/#OPENSSL-PROVIDERS).
874877
It is not necessary to rebuild Node.js to enable support for FIPS.
875878

876-
See [FIPS mode](./doc/api/crypto.md#fips-mode) for more information on how to
879+
See [FIPS mode](doc/api/crypto.md#fips-mode) for more information on how to
877880
enable FIPS support in Node.js.
878881

879882
## Building Node.js with external core modules

0 commit comments

Comments
 (0)
Please sign in to comment.