From 410e083d7c5349fa3fc549f4ae4c96b0dfd97a55 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Reis?= Date: Wed, 7 Sep 2016 02:26:01 +0100 Subject: [PATCH] win,build: forward release_urlbase to configure MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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: https://github.com/nodejs/node/pull/8430 Reviewed-By: jbergstroem - Johan Bergström --- vcbuild.bat | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/vcbuild.bat b/vcbuild.bat index 15975257ed2b29..77c0d01a14f486 100644 --- a/vcbuild.bat +++ b/vcbuild.bat @@ -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= @@ -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