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

Fix build on clang++ (macOS/freebsd) #168

Closed
wants to merge 1 commit into from
Closed

Conversation

@king6cong
Copy link

king6cong commented Feb 26, 2019

This change is Reviewable

@@ -71,6 +71,7 @@ fn cc_flags() -> Vec<&'static str> {

if is_apple || is_freebsd {
result.push("-stdlib=libc++");
result.push("-std=c++14");

This comment has been minimized.

@jbeich

jbeich Feb 26, 2019

FWIW, all supported FreeBSD versions ship Clang 6 or later which defaults to C++14 as is.

@nox
Copy link
Member

nox commented Sep 30, 2019

@king6cong Why was this needed?

@nox
Copy link
Member

nox commented Oct 8, 2019

Closing this, I don't think it is an issue anymore.

@nox nox closed this Oct 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.