[fix] use binary-pack module so we only need to npm install one extra…

… dependency for binary support
primus · Aug 19, 2013 · 712ba5c · 712ba5c
1 parent 71ced0b
commit 712ba5c
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/package.json b/package.json
@@ -42,7 +42,6 @@
     "ws": "0.4.x",
     "sockjs-client-node": "0.0.x",
     "request": "2.21.x",
-    "js-binarypack": "git://github.com/jcrugzz/js-binarypack.git#make-requireable",
+    "binary-pack": "0.0.x"
-    "binarypack": "0.0.x"
   }
 }
diff --git a/parsers.json b/parsers.json
@@ -7,6 +7,6 @@
     "server": "e-json"
   },
   "binary": {
-    "server": "binarypack js-binarypack"
+    "server": "binary-pack"
   }
 }
diff --git a/parsers/binary.js b/parsers/binary.js
@@ -1,7 +1,6 @@
 'use strict';
 
-var BinaryPack = require('binarypack')
+var BinaryPack = require('binary-pack');
-  , fs = require('fs');
 
 /**
  * Message encoder.
@@ -32,8 +31,8 @@ exports.decoder = function decoder(data, fn) {
 //
 exports.library = [
   'var BinaryPack = (function () {',
-  '  try { return require("binarypack"); }',
+  '  try { return require("binary-pack"); }',
   '  catch (e) {}',
-  fs.readFileSync(require.resolve('js-binarypack'), 'utf-8'),
+  BinaryPack.BrowserSource,
   '})();'
 ].join('\n');