From 087544342f271c325f78aeeaba8891643c5eef76 Mon Sep 17 00:00:00 2001
From: Jinho Bang <zino@chromium.org>
Date: Mon, 20 Nov 2017 02:40:28 +0900
Subject: [PATCH] Remove platform-specific commands

No longer need the platform-specific commands.

ISSUE=#193
---
 binding.gyp              | 86 +++++++++-------------------------------
 bootstrap/bacardi.sh     |  3 +-
 bootstrap/command/tsc    |  2 +-
 generator/generator.gypi | 28 -------------
 tsconfig.json            |  8 ++--
 5 files changed, 26 insertions(+), 101 deletions(-)
 delete mode 100644 generator/generator.gypi

diff --git a/binding.gyp b/binding.gyp
index 209be88..0ffcffd 100644
--- a/binding.gyp
+++ b/binding.gyp
@@ -13,10 +13,6 @@
 # limitations under the License.
 
 {
-  'includes': [
-    'generator/generator.gypi',
-  ],
-
   'variables': {
     'bacardi_command%': './bacardi',
     'conditions': [
@@ -31,30 +27,12 @@
       'target_name': 'bacardi',
       'dependencies': [
         'idl',
+        '<!(<(bacardi_command) node -p "require(\'node-addon-api\').gyp")',
       ],
-      'conditions': [
-        ['OS!="win"',
-        {
-          'dependencies': [
-            '<!@(./bootstrap/command/node -p \'require("node-addon-api").gyp\')',
-          ],
-          'include_dirs': [
-            './',
-            '<@(SHARED_INTERMEDIATE_DIR)',
-            '<!@(./bootstrap/command/node -p \'require("node-addon-api").include\')',
-          ],
-        }],
-        ['OS=="win"',
-        {
-          'dependencies': [
-            '<!(third_party\\node\\node.exe -p "require(\'node-addon-api\').gyp")',
-          ],
-          'include_dirs': [
-            './',
-            '<@(SHARED_INTERMEDIATE_DIR)',
-            '<!@(third_party\\node\\node.exe -p "require(\'node-addon-api\').include")',
-          ],
-        }],
+      'include_dirs': [
+        './',
+        '<@(SHARED_INTERMEDIATE_DIR)',
+        '<!@(<(bacardi_command) node -p "require(\'node-addon-api\').include")',
       ],
       'sources': [
         '<!@(<(bacardi_command) list_cpp_files --silent)',
@@ -71,34 +49,20 @@
         {
           'action_name': 'tsc',
           'inputs': [
-            '<@(generator_files)',
           ],
           'outputs': [
             '<@(PRODUCT_DIR)/generator',
           ],
-          'conditions': [
-            ['OS!="win"',
-            {
-              'action': [
-                '<@(PRODUCT_DIR)/../../bootstrap/command/tsc',
-                '<@(_inputs)',
-                '--outDir',
-                '<@(_outputs)',
-                ],
-            }],
-            ['OS=="win"',
-            {
-              'action': [
-                '<@(PRODUCT_DIR)/../../third_party/node/node.exe <@(PRODUCT_DIR)/../../node_modules/typescript/bin/tsc --lib es2015',
-                '<@(_inputs)',
-                '--outDir',
-                '<@(_outputs)',
-                ],
-            }],
+          'action': [
+            '<(module_root_dir)/<(bacardi_command)',
+            'tsc',
+            '--outDir',
+            '<@(_outputs)',
           ],
         },
       ],
     },
+
     {
       'target_name': 'idl',
       'type': 'none',
@@ -114,27 +78,13 @@
           'outputs': [
             '<!@(<(bacardi_command) list_generated_cpp_files --silent)',
           ],
-          'conditions': [
-            ['OS!="win"',
-            {
-              'action': [
-                '<@(PRODUCT_DIR)/../../bootstrap/command/node',
-                '<@(PRODUCT_DIR)/generator/main.js',
-                '<(module_root_dir)',
-                '<@(SHARED_INTERMEDIATE_DIR)',
-                '<@(_inputs)',
-              ],
-            }],
-            ['OS=="win"',
-            {
-              'action': [
-                '<@(PRODUCT_DIR)/../../third_party/node/node.exe',
-                '<@(PRODUCT_DIR)/generator/main.js',
-                '<(module_root_dir)',
-                '<@(SHARED_INTERMEDIATE_DIR)',
-                '<@(_inputs)',
-              ],
-            }],
+          'action': [
+            '<(module_root_dir)/<(bacardi_command)',
+            'node',
+            '<@(PRODUCT_DIR)/generator/main.js',
+            '<(module_root_dir)',
+            '<@(SHARED_INTERMEDIATE_DIR)',
+            '<@(_inputs)',
           ],
         },
       ],
diff --git a/bootstrap/bacardi.sh b/bootstrap/bacardi.sh
index 8fac8de..860c207 100644
--- a/bootstrap/bacardi.sh
+++ b/bootstrap/bacardi.sh
@@ -40,8 +40,9 @@ if [ ! -f .last_update ] || [ package.json -nt .last_update ]; then
   npm install
 fi
 
+sub_command=$(basename $(echo $1 | sed 's/\\/\//g'))
 for command in $(ls $(bootstrap_command_path)); do
-  if [ "$1" = "$command" ]; then
+  if [ "$sub_command" = "$command" ]; then
     shift
     $(bootstrap_command_path)/$command $@
     exit
diff --git a/bootstrap/command/tsc b/bootstrap/command/tsc
index cac2574..cc7e91f 100755
--- a/bootstrap/command/tsc
+++ b/bootstrap/command/tsc
@@ -14,4 +14,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-tsc --lib es2015 $@
+tsc $@
diff --git a/generator/generator.gypi b/generator/generator.gypi
deleted file mode 100644
index bbabba0..0000000
--- a/generator/generator.gypi
+++ /dev/null
@@ -1,28 +0,0 @@
-# Copyright (c) 2017 The Bacardi Authors.
-#
-# Licensed under the Apache License, Version 2.0 (the 'License');
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an 'AS IS' BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-{
-  'variables': {
-    'generator_files': [
-      'base/file.ts',
-      'parser/idl_definition.ts',
-      'parser/idl_definition_factory.ts',
-      'parser/idl_identifier.ts',
-      'parser/idl_interface.ts',
-      'parser/parser.ts',
-      'reader/simple_reader.ts',
-      'main.ts',
-    ],
-  },
-}
diff --git a/tsconfig.json b/tsconfig.json
index 45794dc..b7c2362 100644
--- a/tsconfig.json
+++ b/tsconfig.json
@@ -3,10 +3,12 @@
   "compilerOptions": {
     "baseUrl": ".",
     "experimentalDecorators": true,
-    "noImplicitAny": true,
-    "noImplicitReturns": true,
+    "lib": ["es2015"],
     "preserveConstEnums": true,
     "sourceMap": true,
     "target": "es5"
-  }
+  },
+  "include": [
+    "generator/**/*"
+  ]
 }