Merge pull request #74 from saadtazi/bug/73-ctl-c-not-exiting

properly exit on ctl-c
saadtazi · Dec 12, 2016 · eb4862d · eb4862d
2 parents 9971ffa + b0fc41a
commit eb4862d
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/lib/firefox_profile.js b/lib/firefox_profile.js
@@ -136,13 +136,14 @@ function FirefoxProfile(options) {
 
   // delete on process.exit()...
   var self = this;
-  this.onExit = function() {
+  this.onExit = function(evt) {
     if (self._deleteOnExit) {
       self._cleanOnExit();
     }
+    process.exit(evt === 'SIGINT' ? 130 : 0);
   };
   ['exit', 'SIGINT'].forEach(function(event) {
-    process.addListener(event, self.onExit);
+    process.addListener(event, self.onExit.bind(self, event));
   });
 }
 
@@ -154,9 +155,7 @@ function deleteParallel(files, cb) {
   });
 }
 
-
 FirefoxProfile.prototype._copy = function(profileDirectory, cb) {
-  var self = this;
   fs.copy(profileDirectory, this.profileDir, {
     clobber: true,
     filter: isNotLockFile