build: fix ARM build

Several fixes needed:
1. Don't pass in the platform to ChakraCore-
   it'll figure out the right SDK version
2. Disable intl on ARM if cross-compiling
3. Set the default SDK version if compiling ARM to
   be v10.0 since Node doesn't seem to compile for
   Windows on ARM with the older SDKs.

PR-URL: #265
Reviewed-By: Kunal Pathak <kunal.pathak@microsoft.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>

Author: kfarnung

configure

@@ -787,6 +787,9 @@ def configure_arm(o):
 
   o['variables']['arm_fpu'] = options.arm_fpu or arm_fpu
 
+  if flavor == 'win':
+    o['msvs_windows_sdk_version'] = 'v10.0'
+
 
 def configure_mips(o):
   can_use_fpu_instructions = (options.mips_float_abi != 'soft')

deps/chakrashim/chakracore.gyp

@@ -4,7 +4,6 @@
     'library%': 'static_library',   # build chakracore as static library or dll
     'component%': 'static_library', # link crt statically or dynamically
     'chakra_dir%': 'core',
-    'msvs_windows_target_platform_version_prop': '',
     'icu_args%': '',
     'icu_include_path%': '',
     'linker_start_group%': '',
@@ -19,8 +18,6 @@
       ['target_arch=="x64"', { 'Platform': 'x64' }],
       ['target_arch=="arm"', {
         'Platform': 'arm',
-        'msvs_windows_target_platform_version_prop':
-          '/p:WindowsTargetPlatformVersion=$(WindowsTargetPlatformVersion)',
       }],
       ['OS!="win"', {
         'icu_include_path': '../<(icu_path)/source/common'
@@ -113,7 +110,6 @@
                 '/p:Configuration=$(ConfigurationName)',
                 '/p:RuntimeLib=<(component)',
                 '/p:AdditionalPreprocessorDefinitions=COMPILE_DISABLE_Simdjs=1',
-                '<(msvs_windows_target_platform_version_prop)',
                 '/m',
                 '<@(_inputs)',
               ],

vcbuild.bat

@@ -143,6 +143,13 @@ if "%i18n_arg%"=="intl-none" set configure_flags=%configure_flags% --with-intl=n
 if "%i18n_arg%"=="without-intl" set configure_flags=%configure_flags% --without-intl
 if "%engine%"=="chakracore" set configure_flags=%configure_flags% --without-bundled-v8&set chakra_jslint=deps\chakrashim\lib
 
+if "%target_arch%"=="arm" (
+  if "%PROCESSOR_ARCHITECTURE%" NEQ "ARM" (
+    echo Skipping building ARM with Intl on a non-ARM device
+    set configure_flags=%configure_flags% --without-intl
+  )
+)
+
 if defined config_flags set configure_flags=%configure_flags% %config_flags%
 
 if not exist "%~dp0deps\icu" goto no-depsicu