Avoid recompiling remote native node modules (#112644)

* avoid recompiling remote native node modules

fixes #112320

* fix build

* update cache

* empty

* chore: remove rebuild step

* chore: update salt

* product compile should use same CC CXX

Co-authored-by: deepak1556 <hop2deep@gmail.com>

build/azure-pipelines/linux/product-build-linux.yml

@@ -76,12 +76,19 @@ steps:
 
   - script: |
       set -e
-      if [ -z "$CC" ] || [ -z "$CXX" ]
-      then
+      export npm_config_arch=$(NPM_ARCH)
+
+      if [ -z "$CC" ] || [ -z "$CXX" ]; then
         export CC=$(which gcc-5)
         export CXX=$(which g++-5)
       fi
-      export npm_config_arch=$(NPM_ARCH)
+
+      if [ "$VSCODE_ARCH" == "x64" ]; then
+        export VSCODE_REMOTE_CC=$(which gcc-4.8)
+        export VSCODE_REMOTE_CXX=$(which g++-4.8)
+        export VSCODE_REMOTE_NODE_GYP=$(which node-gyp)
+      fi
+
       for i in {1..3}; do # try 3 times, for Terrapin
         yarn --frozen-lockfile && break
         if [ $i -eq 3 ]; then
@@ -103,16 +110,6 @@ steps:
       vstsFeed: "npm-vscode"
     condition: and(succeeded(), ne(variables['CacheRestored'], 'true'))
 
-  - script: |
-      set -e
-      export CC=$(which gcc-4.8)
-      export CXX=$(which g++-4.8)
-      export npm_config_node_gyp=$(which node-gyp)
-      cd remote && rm -rf node_modules/
-      yarn
-    displayName: Rebuild remote modules with gcc-4.8
-    condition: and(succeeded(), eq(variables['VSCODE_ARCH'], 'x64'))
-
   - script: |
       set -e
       node build/azure-pipelines/mixin