tools: don't fetch V8 deps in the source tree by richardlau · Pull Request #60883 · nodejs/node
The Node.js source tree already includes several V8 DEPS under `deps/v8/third_party` that are needed to build Node.js. Exclude these in `tools/v8/fetch_deps.py` to prevent "Conflicting directory" warnings when running the V8 CI.
labels
targos pushed a commit that referenced this pull request
The Node.js source tree already includes several V8 DEPS under `deps/v8/third_party` that are needed to build Node.js. Exclude these in `tools/v8/fetch_deps.py` to prevent "Conflicting directory" warnings when running the V8 CI. PR-URL: #60883 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
aduh95 pushed a commit that referenced this pull request
The Node.js source tree already includes several V8 DEPS under `deps/v8/third_party` that are needed to build Node.js. Exclude these in `tools/v8/fetch_deps.py` to prevent "Conflicting directory" warnings when running the V8 CI. PR-URL: #60883 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
RafaelGSS pushed a commit that referenced this pull request
The Node.js source tree already includes several V8 DEPS under `deps/v8/third_party` that are needed to build Node.js. Exclude these in `tools/v8/fetch_deps.py` to prevent "Conflicting directory" warnings when running the V8 CI. PR-URL: #60883 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
aduh95 pushed a commit that referenced this pull request
The Node.js source tree already includes several V8 DEPS under `deps/v8/third_party` that are needed to build Node.js. Exclude these in `tools/v8/fetch_deps.py` to prevent "Conflicting directory" warnings when running the V8 CI. PR-URL: #60883 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
aduh95 pushed a commit that referenced this pull request
The Node.js source tree already includes several V8 DEPS under `deps/v8/third_party` that are needed to build Node.js. Exclude these in `tools/v8/fetch_deps.py` to prevent "Conflicting directory" warnings when running the V8 CI. PR-URL: #60883 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
aduh95 pushed a commit that referenced this pull request
The Node.js source tree already includes several V8 DEPS under `deps/v8/third_party` that are needed to build Node.js. Exclude these in `tools/v8/fetch_deps.py` to prevent "Conflicting directory" warnings when running the V8 CI. PR-URL: #60883 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
aduh95 pushed a commit that referenced this pull request
The Node.js source tree already includes several V8 DEPS under `deps/v8/third_party` that are needed to build Node.js. Exclude these in `tools/v8/fetch_deps.py` to prevent "Conflicting directory" warnings when running the V8 CI. PR-URL: #60883 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
aduh95 pushed a commit that referenced this pull request
The Node.js source tree already includes several V8 DEPS under `deps/v8/third_party` that are needed to build Node.js. Exclude these in `tools/v8/fetch_deps.py` to prevent "Conflicting directory" warnings when running the V8 CI. PR-URL: #60883 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters