Skip to content

Commit

Permalink
win,build: forward release_urlbase to configure
Browse files Browse the repository at this point in the history
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]>
  • Loading branch information
joaocgreis authored and MylesBorins committed Oct 26, 2016
1 parent 26e7374 commit 410e083
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions vcbuild.bat
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ set noperfctr=
set noperfctr_msi_arg=
set i18n_arg=
set download_arg=
set release_urls_arg=
set build_release=
set configure_flags=
set build_addons=
Expand Down Expand Up @@ -104,7 +103,7 @@ if "%config%"=="Debug" set configure_flags=%configure_flags% --debug
if defined nosnapshot set configure_flags=%configure_flags% --without-snapshot
if defined noetw set configure_flags=%configure_flags% --without-etw& set noetw_msi_arg=/p:NoETW=1
if defined noperfctr set configure_flags=%configure_flags% --without-perfctr& set noperfctr_msi_arg=/p:NoPerfCtr=1
if defined release_urlbase set release_urlbase_arg=--release-urlbase=%release_urlbase%
if defined release_urlbase set configure_flags=%configure_flags% --release-urlbase=%release_urlbase%
if defined download_arg set configure_flags=%configure_flags% %download_arg%

if "%i18n_arg%"=="full-icu" set configure_flags=%configure_flags% --with-intl=full-icu
Expand Down

0 comments on commit 410e083

Please sign in to comment.