Skip to content

Commit e670732

Browse files
Julien Gillitjfontaine
Julien Gilli
authored andcommitted
build: enable small-icu support for build-release
On Windows, when building the "build-release" rule, enable both small-icu and download-all. This change also slightly refactors the build-release rule, mainly so that it's easier to read. The resulting MSI package was tested on Windows 7. Fixes #9099. PR: #9100 PR-URL: nodejs/node-v0.x-archive#9100 Reviewed-By: Steven R. Loomis <[email protected]> Reviewed-By: Timothy J Fontaine <[email protected]>
1 parent ddd40fd commit e670732

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

vcbuild.bat

+11-1
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ set noperfctr_arg=
3737
set noperfctr_msi_arg=
3838
set i18n_arg=
3939
set download_arg=
40+
set build_release=
4041

4142
:next-arg
4243
if "%1"=="" goto args-done
@@ -69,7 +70,7 @@ if /i "%1"=="small-icu" set i18n_arg=%1&goto arg-ok
6970
if /i "%1"=="full-icu" set i18n_arg=%1&goto arg-ok
7071
if /i "%1"=="intl-none" set i18n_arg=%1&goto arg-ok
7172
if /i "%1"=="download-all" set download_arg="--download=all"&goto arg-ok
72-
if /i "%1"=="build-release" set nosnapshot=1&set config=Release&set msi=1&set licensertf=1&goto arg-ok
73+
if /i "%1"=="build-release" set build_release=1&goto arg-ok
7374

7475
echo Warning: ignoring invalid command line option `%1`.
7576

@@ -82,6 +83,15 @@ goto next-arg
8283
if defined upload goto upload
8384
if defined jslint goto jslint
8485

86+
if defined build_release (
87+
set nosnapshot=1
88+
set config=Release
89+
set msi=1
90+
set licensertf=1
91+
set download_arg="--download=all"
92+
set i18n_arg=small-icu
93+
)
94+
8595
if "%config%"=="Debug" set debug_arg=--debug
8696
if "%target_arch%"=="x64" set msiplatform=x64
8797
if defined nosnapshot set nosnapshot_arg=--without-snapshot

0 commit comments

Comments
 (0)