reduce console.logs

This commit is contained in:
Luis Blanco 2018-02-02 09:45:35 +03:00
parent e5ba8d0105
commit d6a679d335
3 changed files with 32 additions and 23 deletions

View File

@ -42,9 +42,9 @@ dependency packages.
``` ```
'variables': { 'variables': {
'rm' : '<!(node -e "console.log(require(\'addon-tools-raub\').rm)")', 'rm' : '<!(node -e "require(\'addon-tools-raub\').rm()")',
'cp' : '<!(node -e "console.log(require(\'addon-tools-raub\').cp)")', 'cp' : '<!(node -e "require(\'addon-tools-raub\').cp()")',
'mkdir' : '<!(node -e "console.log(require(\'addon-tools-raub\').mkdir)")', 'mkdir' : '<!(node -e "require(\'addon-tools-raub\').mkdir()")',
}, },
``` ```
@ -102,8 +102,8 @@ module.exports = require('addon-tools-raub').paths(__dirname);
``` ```
{ {
'variables': { 'variables': {
'rm' : '<!(node -e "console.log(require(\'addon-tools-raub\').rm)")', 'rm' : '<!(node -e "require(\'addon-tools-raub\').rm()")',
'rem' : '<!(node -e "console.log(require(\'.\').rem)")', 'rem' : '<!(node -e "require(\'.\').rem()")',
}, },
'targets': [ 'targets': [
{ {
@ -175,11 +175,11 @@ module.exports = require('./binary/addon');
``` ```
{ {
'variables': { 'variables': {
'rm' : '<!(node -e "console.log(require(\'addon-tools-raub\').rm)")', 'rm' : '<!(node -e "require(\'addon-tools-raub\').rm()")',
'cp' : '<!(node -e "console.log(require(\'addon-tools-raub\').cp)")', 'cp' : '<!(node -e "require(\'addon-tools-raub\').cp()")',
'mkdir' : '<!(node -e "console.log(require(\'addon-tools-raub\').mkdir)")', 'mkdir' : '<!(node -e "require(\'addon-tools-raub\').mkdir()")',
'EXT_LIB_include' : '<!(node -e "console.log(require(\'node-deps-EXT_LIB-raub\').include)")', 'EXT_LIB_include' : '<!(node -e "require(\'node-deps-EXT_LIB-raub\').include()")',
'EXT_LIB_bin' : '<!(node -e "console.log(require(\'node-deps-EXT_LIB-raub\').bin)")', 'EXT_LIB_bin' : '<!(node -e "require(\'node-deps-EXT_LIB-raub\').bin()")',
}, },
'targets': [ 'targets': [
{ {
@ -496,7 +496,7 @@ input `dir`.
--- ---
## Crossplatform commands ## Cross-platform commands
Because of the differences between Windows and Unix command shells, often a whole Because of the differences between Windows and Unix command shells, often a whole
lot of conditions have to be introduced in **binding.gyp** file. Now some of lot of conditions have to be introduced in **binding.gyp** file. Now some of
@ -519,7 +519,7 @@ folder. This can possibly be bypassed by supplying `./-p` or something like this
``` ```
'variables': { 'variables': {
'mkdir' : '<!(node -e "console.log(require(\'addon-tools-raub\').mkdir)")', 'mkdir' : '<!(node -e "require(\'addon-tools-raub\').mkdir()")',
}, },
... ...
'action' : ['<(mkdir)', '-p', 'binary'], 'action' : ['<(mkdir)', '-p', 'binary'],
@ -533,8 +533,8 @@ be used on all platforms to remove single and multiple files and directories.
``` ```
'variables': { 'variables': {
'rm' : '<!(node -e "console.log(require(\'addon-tools-raub\').rm)")', 'rm' : '<!(node -e "require(\'addon-tools-raub\').rm()")',
'rem' : '<!(node -e "console.log(require(\'.\').rem)")', 'rem' : '<!(node -e "require(\'.\').rem()")',
}, },
... ...
'action' : ['<(rm)', '-rf', '<@(rem)'], 'action' : ['<(rm)', '-rf', '<@(rem)'],
@ -543,3 +543,11 @@ be used on all platforms to remove single and multiple files and directories.
### cp ### cp
For Windows the `/y` flag was embedded. For Windows the `/y` flag was embedded.
```
'variables': {
'cp' : '<!(node -e "require(\'addon-tools-raub\').cp()")',
},
...
'action' : ['<(cp)', 'a', 'b'],
```

View File

@ -26,6 +26,7 @@ const getPlatformDir = platform => {
const currentDir = prefixName(getPlatformDir(process.platform)); const currentDir = prefixName(getPlatformDir(process.platform));
const remDirs = names.map(prefixName).filter(n => n !== currentDir); const remDirs = names.map(prefixName).filter(n => n !== currentDir);
const isWindows = process.platform === 'win32';
const paths = dir => { const paths = dir => {
@ -33,14 +34,14 @@ const paths = dir => {
const binPath = `${dir}/${currentDir}`; const binPath = `${dir}/${currentDir}`;
if (process.platform === 'win32') { if (isWindows) {
process.env.path = `${process.env.path ? `${process.env.path};` : ''}${binPath}`; process.env.path = `${process.env.path ? `${process.env.path};` : ''}${binPath}`;
} }
return { return {
bin : binPath, bin() { console.log(binPath); },
rem : remDirs.map(k => `${dir}/${k}`).join(' '), rem() { console.log(remDirs.map(k => `${dir}/${k}`).join(' ')); },
include : `${dir}/include`, include() { console.log(`${dir}/include`); },
}; };
}; };
@ -50,12 +51,12 @@ module.exports = {
paths, paths,
root : thisDir, root() { return console.log(thisDir); },
include() { console.log(`${nanInclude} ${thisInclude}`); }, include() { console.log(`${nanInclude} ${thisInclude}`); },
mkdir : process.platform === 'win32' ? `${thisDir}/_mkdir.bat` : 'mkdir', mkdir() { return isWindows ? `${thisDir}/_mkdir.bat` : 'mkdir'; },
rm : process.platform === 'win32' ? `${thisDir}/_rm.bat` : 'rm', rm() { return isWindows ? `${thisDir}/_rm.bat` : 'rm'; },
cp : process.platform === 'win32' ? `${thisDir}/_cp.bat` : 'cp', cp() { return isWindows ? `${thisDir}/_cp.bat` : 'cp'; },
}; };

View File

@ -2,7 +2,7 @@
"name": "addon-tools-raub", "name": "addon-tools-raub",
"author": "Luis Blanco <raubtierxxx@gmail.com>", "author": "Luis Blanco <raubtierxxx@gmail.com>",
"description": "A set of extra tools for Node.js addons", "description": "A set of extra tools for Node.js addons",
"version": "0.0.5", "version": "0.0.6",
"main": "index.js", "main": "index.js",
"keywords": [ "keywords": [
"node", "node",