Skip to content
This repository has been archived by the owner on Jun 18, 2021. It is now read-only.

clang warning when using MacOS Sierra #22

Closed
AndyRWatson opened this issue Nov 15, 2016 · 8 comments
Closed

clang warning when using MacOS Sierra #22

AndyRWatson opened this issue Nov 15, 2016 · 8 comments

Comments

@AndyRWatson
Copy link

AndyRWatson commented Nov 15, 2016

when attempting to install on Node v6.7.0 via npm using MacOS 10.12.1 (Sierra) I get the following compiler warning

clang: warning: libstdc++ is deprecated; move to libc++ with a minimum deployment target of OS X 10.9

@gibfahn
Copy link
Member

gibfahn commented Nov 15, 2016

Node core issue here: nodejs/node#8521

@rnchamberlain
Copy link
Contributor

Possible work-around here: nodejs/nan#606

@rnchamberlain
Copy link
Contributor

Fix landed in github, not published in npm yet.

@JeremyIglehart
Copy link

This is all still way over my head - I'm still experiencing this. Any ETA when this will make it to NPM?

@rnchamberlain
Copy link
Contributor

Should be on npm in the next day or so. Quite a few changes have gone in and it's a new version, just need to do the project rename, see #35 #43

@JeremyIglehart
Copy link

Wow. Amazing work has been done here. You all work together better than most companies do! Hat's off to everyone on this project. It was encouraging to read through those issues. Thank you for al the hard work you're all working on.

@richardlau
Copy link
Member

We've decided in #56 to revert the change made by #31 and instead be consistent with Node.js and link to the same library as it does. That means that if you build with Node.js v6 you will get deprecation warnings (you will get the same deprecation warning if you build Node.js v6, see nodejs/node#8521 (comment)).

Node.js v7 and above have switched to using libc++, so you will not get the warnings with those.

@rnchamberlain
Copy link
Contributor

Closing this as per @richardlau comment above, fixed in Node 7 and later.

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

Successfully merging a pull request may close this issue.

5 participants