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

error: invalid argument '-std=c++14' not allowed with 'C' #33923

Closed
ryandesign opened this issue Jul 31, 2023 · 3 comments
Closed

error: invalid argument '-std=c++14' not allowed with 'C' #33923

ryandesign opened this issue Jul 31, 2023 · 3 comments

Comments

@ryandesign
Copy link

What version of gRPC and what language are you using?

gRPC 1.48.0
Python

What operating system (Linux, Windows,...) and version?

OS X 10.9 but I don't think the issue is OS specific

What runtime / compiler are you using (e.g. python version or version of gcc)

Clang 16.0.6
Python 3.10.12

What did you do?

Build the grpcio python module

What did you expect to see?

Successful build

What did you see instead?

clang: warning: argument unused during compilation: '-stdlib=libc++' [-Wunused-command-line-argument]
error: invalid argument '-std=c++14' not allowed with 'C'
distutils.errors.CompileError: command '/opt/local/bin/clang-mp-16' failed with exit code 1

py310-grpcio.log

Anything else we should know about your project / environment?

I reported this to MacPorts > 2 years ago but nothing has happened so I'm reporting it to you now.

@XuanWang-Amos
Copy link
Contributor

Hi, looks like we do have a check here which suppose to ensure that the correct -std= will be used. Since we don't have a similar environment to reproduce, can you help check why the check is not working correctly on your side?

@mascguy
Copy link

mascguy commented Aug 17, 2023

Since MacPorts is still at version 1.48.x, we didn't have the benefit of the following fix:

PR 26480 - Respect CC variable in grpcio python build

But with that in place, everything is good now.

So I think we can close this issue. @ryandesign ?

@gnossen
Copy link
Contributor

gnossen commented Aug 23, 2023

@mascguy Thanks for confirming!

@gnossen gnossen closed this as completed Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants