Removed docker build from C++ build

.travis.yml

@@ -42,6 +42,7 @@ install:
 script:
   - if [ "$TEST" = "WASM" ]; then yarn lint; fi
   - if [ "$TEST" = "WASM" ]; then PSP_DOCKER=1 yarn build; fi
-  - if [ "$TEST" = "WASM" ]; then yarn test --docker --quiet; fi
-  - if [ "$TEST" = "CPP" ]; then docker build -f docker/cpp/Dockerfile .; fi
-  - if [ "$TEST" = "PYTHON" ]; then docker build -f docker/python/Dockerfile .; fi
+  - if [ "$TEST" = "WASM" ]; then PSP_DOCKER=1 yarn test --quiet; fi
+  - if [ "$TEST" = "CPP" ]; then PSP_DOCKER=1 yarn build_cpp; fi
+  - if [ "$TEST" = "CPP" ]; then PSP_DOCKER=1 yarn test_cpp; fi
+  - if [ "$TEST" = "PYTHON" ]; then PSP_DOCKER=1 yarn build_python; fi

package.json

@@ -38,13 +38,13 @@
         "build,test,bench": "npm run --silent build && npm run --silent test && npm run --silent bench",
         "build,bench": "npm run --silent build && npm run --silent bench",
         "build": "node scripts/build.js",
-        "build_cpp": "lerna run build:cpp ${PACKAGE:+--scope=@jpmorganchase/perspective} --stream",
+        "build_cpp": "node scripts/build_cpp.js",
         "build_python": "node scripts/build_python.js",
         "install_python": "cd python && python3 setup.py install",
         "bench": "node scripts/bench.js",
         "docs": "lerna run docs --silent --stream",
         "test": "node scripts/test.js",
-        "test_cpp": "lerna run test:cpp ${PACKAGE:+--scope=@jpmorganchase/perspective} --stream",
+        "test_cpp": "node scripts/test_cpp.js",
         "clean": "find obj -mindepth 1 -delete && lerna run clean ${PACKAGE:+--scope=@jpmorganchase/${PACKAGE}} --stream",
         "preclean:screenshots": "lerna exec -- mkdir -p screenshots",
         "clean:screenshots": "lerna run clean:screenshots ${PACKAGE:+--scope=@jpmorganchase/${PACKAGE}}",

packages/perspective/package.json

@@ -18,15 +18,12 @@
     "scripts": {
         "bench": "node ../../scripts/bench.js",
         "prebuild": "mkdir -p build && mkdir -p obj",
-        "compile:cpp": "mkdir -p ../../cppbuild && cd ../../cppbuild && cmake ../ -DPSP_WASM_BUILD=0 -DPSP_CPP_BUILD=1 -DPSP_CPP_BUILD_TESTS=1 && make -j${PSP_CPU_COUNT-8}",
         "cpp": "npm-run-all build:cpp test:cpp",
         "build": "npm-run-all build:webpack",
-        "build:cpp": "npm-run-all compile:cpp",
         "build:webpack": "npm-run-all -p build:webpack:*",
         "build:webpack:umd": "webpack --color --config src/config/perspective.config.js",
         "build:webpack:node": "webpack --color --config src/config/perspective.node.config.js",
         "docs": "documentation build src/js/perspective.js -f md --shallow > README.md && cp README.md ../../docs/perspective.md",
-        "test:cpp": "npm-run-all build:cpp && ../../cppbuild/test/psp_test",
         "test:build": "cp test/html/* build",
         "test:run": "jest --color --ci",
         "test": "npm-run-all test:build test:run",

scripts/build_cpp.js

@@ -0,0 +1,35 @@
+/******************************************************************************
+ *
+ * Copyright (c) 2017, the Perspective Authors.
+ *
+ * This file is part of the Perspective library, distributed under the terms of
+ * the Apache License 2.0.  The full license can be found in the LICENSE file.
+ *
+ */
+
+const execSync = require("child_process").execSync;
+
+const execute = cmd => execSync(cmd, {stdio: "inherit"});
+
+function docker(image = "emsdk") {
+    console.log(`-- Creating ${image} docker image`);
+    let cmd = "docker run --rm -it";
+    if (process.env.PSP_CPU_COUNT) {
+        cmd += ` --cpus="${parseInt(process.env.PSP_CPU_COUNT)}.0"`;
+    }
+    cmd += ` -v $(pwd):/usr/src/app/cpp -w /usr/src/app/cpp/cppbuild perspective/${image}`;
+    return cmd;
+}
+
+try {
+    execute("mkdir -p cppbuild");
+    if (process.env.PSP_DOCKER) {
+        execute(docker("cpp") + " cmake ../ -DPSP_WASM_BUILD=0 -DPSP_CPP_BUILD=1 -DPSP_CPP_BUILD_TESTS=1");
+        execute(docker("cpp") + " make -j${PSP_CPU_COUNT-8}");
+    } else {
+        execute("cmake ../ -DPSP_WASM_BUILD=0 -DPSP_CPP_BUILD=1 -DPSP_CPP_BUILD_TESTS=1");
+        execute("make -j${PSP_CPU_COUNT-8}");
+    }
+} catch (e) {
+    process.exit(1);
+}

scripts/test_cpp.js

@@ -0,0 +1,33 @@
+/******************************************************************************
+ *
+ * Copyright (c) 2017, the Perspective Authors.
+ *
+ * This file is part of the Perspective library, distributed under the terms of
+ * the Apache License 2.0.  The full license can be found in the LICENSE file.
+ *
+ */
+
+const execSync = require("child_process").execSync;
+
+const execute = cmd => execSync(cmd, {stdio: "inherit"});
+
+function docker(image = "emsdk") {
+    console.log(`-- Creating ${image} docker image`);
+    let cmd = "docker run --rm -it";
+    if (process.env.PSP_CPU_COUNT) {
+        cmd += ` --cpus="${parseInt(process.env.PSP_CPU_COUNT)}.0"`;
+    }
+    cmd += ` -v $(pwd):/usr/src/app/cpp -w /usr/src/app/cpp/cppbuild perspective/${image}`;
+    return cmd;
+}
+
+try {
+    execute("mkdir -p cppbuild");
+    if (process.env.PSP_DOCKER) {
+        execute(docker("cpp") + " ./test/psp_test");
+    } else {
+        execute("./test/psp_test");
+    }
+} catch (e) {
+    process.exit(1);
+}