From 1f44e78a8d6e0be7c0e780a9b8aa7eff45e4b760 Mon Sep 17 00:00:00 2001
From: Luis Blanco <blanco@ucoz-team.net>
Date: Tue, 30 Jan 2018 11:26:47 +0300
Subject: [PATCH] fix windows bats

---
 LICENSE      |  2 +-
 README.md    | 38 ++++++++++++++++++++------------------
 _del.bat     |  4 +++-
 _md.bat      |  3 +++
 index.js     |  1 +
 package.json |  2 +-
 6 files changed, 29 insertions(+), 21 deletions(-)
 create mode 100644 _md.bat

diff --git a/LICENSE b/LICENSE
index 1aef668..d37bd6d 100644
--- a/LICENSE
+++ b/LICENSE
@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2017 Luis Blanco
+Copyright (c) 2018 Luis Blanco
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
diff --git a/README.md b/README.md
index 39b4b1d..15b91ae 100644
--- a/README.md
+++ b/README.md
@@ -28,7 +28,7 @@ dependency packages.
 
 [index.js](#indexjs)
 
-[\_rd.bat / \_del.bat](#_rdbat--_delbat)
+[Windows BAT](#windows-bat)
 
 
 ---
@@ -38,11 +38,12 @@ dependency packages.
 
 ### binding.gyp
 
-* For Windows custom file/folder removers are present, you can put them into variables.
+* For Windows custom file/folder removers/creators are present, you can put them into variables.
 
 ```
 'variables': {
 	'_rd' : '<!(node -e "console.log(require(\'addon-tools-raub\')._rd)")',
+	'_md' : '<!(node -e "console.log(require(\'addon-tools-raub\')._md)")',
 	'_del' : '<!(node -e "console.log(require(\'addon-tools-raub\')._del)")',
 },
 ```
@@ -72,8 +73,9 @@ Unix systems and custom remover on Windows.
 		'<(module_root_dir)/build/Release/addon.node'
 	] } ],
 	[ 'OS=="win"', { 'action' : [
-		'<(_del) "<(module_root_dir)/build/Release/addon.*" && ' +
-		'<(_del) "<(module_root_dir)/build/Release/obj/addon/*.*"'
+		'<(_del)',
+		'<(module_root_dir)/build/Release/addon.*',
+		'<(module_root_dir)/build/Release/obj/addon/*.*'
 	] } ],
 ```
 
@@ -147,10 +149,7 @@ If you always copy your compiled addon to the `binary` directory, it will be eas
 		'conditions'  : [
 			[ 'OS=="linux"', { 'action': ['mkdir', '-p', 'binary'] } ],
 			[ 'OS=="mac"', { 'action': ['mkdir', '-p', 'binary'] } ],
-			[ 'OS=="win"', { 'action': [
-				'<(_rd) "<(module_root_dir)/binary" && ' +
-				'md "<(module_root_dir)/binary"'
-			] } ],
+			[ 'OS=="win"', { 'action': ['<(_md)', 'binary'] } ],
 		],
 	}],
 },
@@ -203,7 +202,7 @@ module.exports = require('./binary/addon');
 {
 	'variables': {
 		'_del'           : '<!(node -e "console.log(require(\'addon-tools-raub\')._del)")',
-		'_rd'            : '<!(node -e "console.log(require(\'addon-tools-raub\')._rd)")',
+		'_md'            : '<!(node -e "console.log(require(\'addon-tools-raub\')._md)")',
 		'EXT_LIB_include' : '<!(node -e "console.log(require(\'node-deps-EXT_LIB-raub\').include)")',
 		'EXT_LIB_bin'     : '<!(node -e "console.log(require(\'node-deps-EXT_LIB-raub\').bin)")',
 	},
@@ -275,10 +274,7 @@ module.exports = require('./binary/addon');
 				'conditions'  : [
 					[ 'OS=="linux"', { 'action': ['mkdir', '-p', 'binary'] } ],
 					[ 'OS=="mac"', { 'action': ['mkdir', '-p', 'binary'] } ],
-					[ 'OS=="win"', { 'action': [
-						'<(_rd) "<(module_root_dir)/binary" && ' +
-						'md "<(module_root_dir)/binary"'
-					] } ],
+					[ 'OS=="win"', { 'action': ['<(_md)', 'binary'] } ],
 				],
 			}],
 		},
@@ -330,8 +326,9 @@ module.exports = require('./binary/addon');
 						'<(module_root_dir)/build/Release/MY_ADDON.node'
 					] } ],
 					[ 'OS=="win"', { 'action' : [
-						'<(_del) "<(module_root_dir)/build/Release/MY_ADDON.*" && ' +
-						'<(_del) "<(module_root_dir)/build/Release/obj/MY_ADDON/*.*"'
+						'<(_del)',
+						'<(module_root_dir)/build/Release/MY_ADDON.*',
+						'<(module_root_dir)/build/Release/obj/MY_ADDON/*.*'
 					] } ],
 				],
 			}],
@@ -537,12 +534,13 @@ input `dir`.
 * `root` - where `'addon-tools-raub'` module is situated.
 * `include` - `'addon-tools-raub'` own 'include' directory.
 * `_rd` - the location of `'_rd.bat'` file.
+* `_md` - the location of `'_md.bat'` file.
 * `_del` - the location of `'_del.bat'` file.
 
 
 ---
 
-## \_rd.bat / \_del.bat
+## Windows BAT
 
 Windows-only utilities. Because in gyp any `/` on Windows is converted to `\`, it is
 impossible to put correct commands for file/directory removal. Those need such
@@ -550,6 +548,9 @@ parameters as `/Q`, but gyp makes them `\Q` which is inappropriate. So these fil
 simply contain their respective commands with all necessary parameters, avoiding any
 conflict with gyp.
 
+Also on Windows there is no `mkdir -p`, hence if directory exists you get an error
+trying to make it great again with `md`.
+
 ```
 ...
 [ 'OS=="mac"', { 'action' : [
@@ -558,7 +559,8 @@ conflict with gyp.
 	'<(module_root_dir)/build/Release/addon.node'
 ] } ],
 [ 'OS=="win"', { 'action' : [
-	'<(_del) "<(module_root_dir)/build/Release/addon.*" && ' +
-	'<(_del) "<(module_root_dir)/build/Release/obj/addon/*.*"'
+	'<(_del)',
+	'<(module_root_dir)/build/Release/addon.*',
+	'<(module_root_dir)/build/Release/obj/addon/*.*'
 ] } ],
 ```
diff --git a/_del.bat b/_del.bat
index 1cd56ed..2db053e 100644
--- a/_del.bat
+++ b/_del.bat
@@ -1 +1,3 @@
-del /f /q %*
+for %%x in (%*) do (
+	if exist %%x del /f /q %%x
+)
diff --git a/_md.bat b/_md.bat
new file mode 100644
index 0000000..fe7df83
--- /dev/null
+++ b/_md.bat
@@ -0,0 +1,3 @@
+for %%x in (%*) do (
+	if not exist %%x md %%x
+)
diff --git a/index.js b/index.js
index 5eeee42..972c3a2 100644
--- a/index.js
+++ b/index.js
@@ -50,6 +50,7 @@ module.exports = {
 	printNan() { require('nan'); },
 	
 	_rd  :`${thisDir}/_rd.bat`,
+	_md  :`${thisDir}/_md.bat`,
 	_del :`${thisDir}/_del.bat`,
 	
 };
diff --git a/package.json b/package.json
index c63f3bf..966170e 100644
--- a/package.json
+++ b/package.json
@@ -2,7 +2,7 @@
   "name": "addon-tools-raub",
   "author": "Luis Blanco <raubtierxxx@gmail.com>",
   "description": "A set of extra tools for Node.js addons",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "main": "index.js",
   "keywords": [
     "node",