Move native code, gypfile to subdir to fix #23.

electron-rebuild will blindly run any top-level binding.gyp, which will fail due to the missing dep, but also is a waste of time as this is N-API and Windows only, so there's no reason to ever rebuild it. Save such users by moving the binding.gyp the only way that works AFAIK, by providing a --cwd to prebuildify.

.gitignore

@@ -1,9 +1,9 @@
 .*/
 !/.idea/
 /.idea/workspace.xml
-/build/
+/binding/build/
+/binding/prebuilds/
 /lib/
-/prebuilds/
 node_modules/
 yarn-error.log*
 *.tgz

.npmignore

@@ -1,5 +1,5 @@
 /.idea/
 /.vs/
-/build/
+/binding/build/
 /examples/
 *.log

CHANGELOG.md

@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- [#23](https://github.com/simonbuchan/native-reg/issues/23) Don't break electron-rebuild by avoiding a top-level
+  binding.gyp.
+
 ## [v0.3.5] - 2021-03-18
 
 ### Added

package.json

@@ -18,8 +18,8 @@
     "lib",
     "src",
     "test",
-    "prebuilds",
-    "binding.gyp",
+    "binding/prebuilds",
+    "binding/binding.gyp",
     "*.cc"
   ],
   "scripts": {
@@ -29,8 +29,8 @@
     "clean": "rimraf build lib prebuilds",
     "build": "yarn build:js && yarn build:ia32 && yarn build:x64",
     "build:js": "tsc -p src",
-    "build:ia32": "prebuildify --napi --target [email protected] --arch ia32",
-    "build:x64": "prebuildify --napi --target [email protected] --arch x64",
+    "build:ia32": "prebuildify --napi --target [email protected] --cwd binding --arch ia32",
+    "build:x64": "prebuildify --napi --target [email protected] --cwd binding --arch x64",
     "test": "mocha --ui tdd"
   },
   "dependencies": {

src/index.ts

@@ -7,7 +7,7 @@ const types = require('util').types || {
 };
 
 const isWindows = process.platform === 'win32';
-const native = isWindows && require('node-gyp-build')(__dirname + '/..');
+const native = isWindows && require('node-gyp-build')(__dirname + '/../binding');
 
 // from winreg.h
 export enum HKEY {