After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 113607 - [build] CXXFLAGS vs. CPPFLAGS
[build] CXXFLAGS vs. CPPFLAGS
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins
git master
Other other
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-05-23 18:47 UTC by Ronald Bultje
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
well, it looks like this... Other plugins might need this too (avifile, ...) (1.33 KB, patch)
2003-05-29 22:38 UTC, Ronald Bultje 		none Details | Review

Description Ronald Bultje 2003-05-23 18:47:00 UTC 
I think a bug is the easiest way to do this.

My build in HEAD/CVS doesn't work for the c++ plugins. Reason is Brian
Cameron's recent changes that move libbla_la_CXXFLAGS to
libbla_la_CPPFLAGS, which causes the build to fail on my system. The FLAGS
are just not being set, so I don't have any -I.... lines during
compilation, so it fails. Changing it back to CXXFLAGS makes it work.
Examples of where this applies is the monkeyaudio plugin and modplug.

My system:
automake-1.6.3
autoconf-2.54c
gcc-2.96

We could add dual lines in each c++ plugin Makefile.am:
libbla_la_CPPFLAGS = ...
libbla_la_CXXFLAGS = $(libbla_la_CPPFLAGS)

But that's rather ugly. Anyone got a good solution here?
Comment 1 David Schleef 2003-05-29 22:21:14 UTC 
I like the ugly solution (with a comment).  It allows me to continue
using my old libtool.  :)
Comment 2 Ronald Bultje 2003-05-29 22:38:45 UTC 
Created attachment 16969 [details] [review]
well, it looks like this... Other plugins might need this too (avifile, ...)
Comment 3 Benjamin Otte (Company) 2003-05-30 13:04:50 UTC 
The old libtool was not changed for that reason. It was changed
because only the new one properly compiles c++ plugins.

FYI: That were my changes not Brians.
Comment 4 Ronald Bultje 2003-06-01 17:03:19 UTC 
I checked my fixes in. Sue me. ;).