build: include ICU by default for win dll option (!17547) · Merge requests · Rodrigo Test / Test Group-nodejs / node · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Rodrigo Muino Tomonari requested to merge github/fork/BethGriggs/include-icu-win-shared-lib into v4.x-staging Dec 08, 2017

Change the Windows .dll builds to include small-icu by default for parity with the regular and msi build.

I noticed this while running the tests against the windows shared library.

Checklist

make -j4 test (UNIX), or vcbuild test (Windows) passes
commit message follows commit guidelines

Affected core subsystem(s)

build