Skip to content

Commit 137b7cb

Browse files
joaocgreisorangemocha
authored andcommitted
win,build: forward release_urlbase to configure
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: #8430 Reviewed-By: jbergstroem - Johan Bergström <[email protected]>
1 parent c21458a commit 137b7cb

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

vcbuild.bat

+1-2
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ set noperfctr=
3333
set noperfctr_msi_arg=
3434
set i18n_arg=
3535
set download_arg=
36-
set release_urls_arg=
3736
set build_release=
3837
set enable_vtune_arg=
3938
set configure_flags=
@@ -108,7 +107,7 @@ if "%config%"=="Debug" set configure_flags=%configure_flags% --debug
108107
if defined nosnapshot set configure_flags=%configure_flags% --without-snapshot
109108
if defined noetw set configure_flags=%configure_flags% --without-etw& set noetw_msi_arg=/p:NoETW=1
110109
if defined noperfctr set configure_flags=%configure_flags% --without-perfctr& set noperfctr_msi_arg=/p:NoPerfCtr=1
111-
if defined release_urlbase set release_urlbase_arg=--release-urlbase=%release_urlbase%
110+
if defined release_urlbase set configure_flags=%configure_flags% --release-urlbase=%release_urlbase%
112111
if defined download_arg set configure_flags=%configure_flags% %download_arg%
113112
if defined enable_vtune_arg set configure_flags=%configure_flags% --enable-vtune-profiling
114113
if defined dll set configure_flags=%configure_flags% --shared

0 commit comments

Comments
 (0)