Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

need to bump some deps for recent versions of Node? #17

Open
bixu opened this issue Feb 25, 2016 · 1 comment
Open

need to bump some deps for recent versions of Node? #17

bixu opened this issue Feb 25, 2016 · 1 comment

Comments

@bixu
Copy link
Contributor

bixu commented Feb 25, 2016

I'm looking into Manatee for Postgres HA at $job, and I've found I need to do this to build for Node 5 and above:

blake.irvin$ git diff
diff --git a/.gitignore b/.gitignore
index 5597b62..23c143d 100644
--- a/.gitignore
+++ b/.gitignore
@@ -13,3 +13,4 @@ smf/manifests/*.xml
 test/etc/zookeeperMgr.test.cfg
 abandoned
 devconfs
+.vagrant/
diff --git a/package.json b/package.json
index c49f7d2..8068e6a 100644
--- a/package.json
+++ b/package.json
@@ -21,7 +21,7 @@
     "dependencies": {
         "assert-plus": "0.1.5",
         "backoff": "1.2.0",
-        "bignum": "0.6.2",
+        "bignum": "0.11.0",
         "bunyan": "0.22.1",
         "cmdln": "3.2.0",
         "extsprintf": "1.3.0",
@@ -33,7 +33,7 @@
         "node-zookeeper-client": "0.2.1",
         "once": "1.3.0",
         "pg": "3.0.3",
-        "posix": "1.0.3",
+        "posix": "4.0.0",
         "posix-getopt": "1.0.0",
         "progbar": "0.1.0",
         "prompt": "0.2.13",

I've not yet tested these changes, but they do allow a successful npm install...

@bixu bixu changed the title need to bump some deps for recent versions of Node need to bump some deps for recent versions of Node? Feb 25, 2016
@davepacheco
Copy link
Contributor

Thanks for that. It would be good to know that it works, and also to check for the breaking changes to "posix" that they won't affect us. Looks like the only posix interface we're using is getpwnam().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants