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

windows build #196

Merged
merged 4 commits into from
Oct 10, 2012
Merged

windows build #196

merged 4 commits into from
Oct 10, 2012

Conversation

booo
Copy link
Contributor

@booo booo commented Sep 26, 2012

Seems like we fixed the windows build. I hope I did not broke any other build :) Linux works here on 0.8.1. Any mac users out there?

#165

Best Regards
Philipp

This should fix the following warning on windows:

..\src\binding.cc(117): warning C4800: 'int' : forcing value to bool 'true' or
'false' (performance warning)

Spotted in issue brianc#165.
@brianc
Copy link
Owner

brianc commented Sep 29, 2012

I really appreciate this. I'm on my honeymoon right now so I cannot test this until I get home. As soon as I get back I'll test, merge, and release new version. 1 week.

@3124mike
Copy link

3124mike commented Oct 3, 2012

I can't build this on MacOs 10.5.8. I haven't installed pg before, so I'm not sure if this new build is the problem. I get:

:~ mike$ npm install pg
npm http GET https://registry.npmjs.org/pg
npm http 304 https://registry.npmjs.org/pg
npm http GET https://registry.npmjs.org/generic-pool/1.0.12
npm http 304 https://registry.npmjs.org/generic-pool/1.0.12

> pg@0.8.4 install /Users/mike/node_modules/pg
> node-gyp rebuild || (exit 0)

  CXX(target) Release/obj.target/binding/src/binding.o
  SOLINK_MODULE(target) Release/binding.node
i686-apple-darwin9-g++-4.0.1: -install_name only allowed with -dynamiclib
make: *** [Release/binding.node] Error 1
gyp ERR! build error 
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/Cellar/node/0.8.11/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:219:23)
gyp ERR! stack     at ChildProcess.EventEmitter.emit (events.js:96:17)
gyp ERR! stack     at Process._handle.onexit (child_process.js:678:10)
gyp ERR! System Darwin 9.8.0
gyp ERR! command "node" "/usr/local/Cellar/node/0.8.11/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/mike/node_modules/pg
gyp ERR! node -v v0.8.11
gyp ERR! node-gyp -v v0.6.11
gyp ERR! not ok 
pg@0.8.4 node_modules/pg
└── generic-pool@1.0.12

I get the same thing with npm -g install pg.

@3124mike
Copy link

3124mike commented Oct 7, 2012

Despite these build errors, pg seems to be working okay.

@brianc
Copy link
Owner

brianc commented Oct 10, 2012

I tested linux and os x builds last night. They pass! Thanks - will release new version this evening. 👍

brianc pushed a commit that referenced this pull request Oct 10, 2012
@brianc brianc merged commit d715515 into brianc:master Oct 10, 2012
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

Successfully merging this pull request may close these issues.

None yet

3 participants