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

vmware autotools m4 macros broken for c++ only flags from pkg-config #46

Open
bzed opened this issue Oct 23, 2015 · 2 comments
Open

vmware autotools m4 macros broken for c++ only flags from pkg-config #46

bzed opened this issue Oct 23, 2015 · 2 comments

Comments

@bzed
Copy link

bzed commented Oct 23, 2015

Looking at
https://github.com/vmware/open-vm-tools/blob/stable-10.0.x/open-vm-tools/m4/vmtools.m4#L157
which will also be called for C++ libraries - all cflags from pkg-config will be assigned to cppflags. So even flags which might be c++ compiler compatible only will go in there.
Imho pkg-config -cflags need to go into CFLAGS for C libraries and CXXFLAGS for c++ libraries. And then use the appropriate ones. Right now building fails as libsigx-2.0-dev ships with -std=gnu11 in cflags and your m4 macros assign that to CPPFLAGS which is used for c code, too...

@ravindravmw
Copy link
Contributor

Bernd, you are right. We are aware of the issue in the macro.

BTW, I have not seen this change in cflags for libsigc++-2.0 v2.6.1. Any idea in what version of the library this change is going to show up?

Thanks for letting us know about this upcoming change in libsigx-2.0, we will fix it.

@oliverkurth
Copy link
Contributor

See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=800399 . I think it's a Debian specific change.

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

3 participants