No need to override child_process.fork

We already support asar in Node mode.

diff --git a/atom/common/lib/asar.coffee b/atom/common/lib/asar.coffee
index 38ac2fc..e3e0f6f 100644 (file)
--- a/atom/common/lib/asar.coffee
+++ b/atom/common/lib/asar.coffee
@@ -298,4 +298,3 @@ exports.wrapFsWithAsar = (fs) ->
   overrideAPISync process, 'dlopen', 1
   overrideAPISync require('module')._extensions, '.node', 1
   overrideAPISync fs, 'openSync'
-  overrideAPISync child_process, 'fork'

diff --git a/spec/asar-spec.coffee b/spec/asar-spec.coffee
index 222faf9..39aa1e6 100644 (file)
--- a/spec/asar-spec.coffee
+++ b/spec/asar-spec.coffee
@@ -342,11 +342,6 @@ describe 'asar package', ->
           done()
         child.send 'message'
 
-      it 'throws ENOENT error when can not find file', ->
-        p = path.join fixtures, 'asar', 'a.asar', 'not-exist'
-        throws = -> child_process.fork p
-        assert.throws throws, /ENOENT/
-
       it 'supports asar in the forked js', (done) ->
         file = path.join fixtures, 'asar', 'a.asar', 'file1'
         child = child_process.fork path.join(fixtures, 'module', 'asar.js')