From adf2c0a4a4f0d3bccb42cd3bbce2ae26cbc4e985 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tim-Philipp=20M=C3=BCller?= Date: Thu, 17 Jul 2008 14:40:51 +0000 Subject: [PATCH] Only use -Wno-attributes (which is there to work around a bug in the taglib 1.5 headers) if the c++ compiler actually... Original commit message from CVS: * configure.ac:: * ext/taglib/Makefile.am:: Only use -Wno-attributes (which is there to work around a bug in the taglib 1.5 headers) if the c++ compiler actually supports it (#543255). --- ChangeLog | 8 ++++++++ configure.ac | 8 ++++++++ ext/taglib/Makefile.am | 4 ++-- 3 files changed, 18 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index de6c6e015f..4051f99999 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2008-07-17 Tim-Philipp Müller + + * configure.ac:: + * ext/taglib/Makefile.am:: + Only use -Wno-attributes (which is there to work around a + bug in the taglib 1.5 headers) if the c++ compiler actually + supports it (#543255). + 2008-07-17 Tim-Philipp Müller Patch by: Benoit Fouet diff --git a/configure.ac b/configure.ac index ddb17be2ff..da5214266f 100644 --- a/configure.ac +++ b/configure.ac @@ -904,6 +904,14 @@ AG_GST_CHECK_FEATURE(TAGLIB, [taglib tagging library], taglib, [ if test "x$HAVE_CXX" != "xyes"; then USE_TAGLIB=false AC_MSG_NOTICE([Not building taglib plugin: no C++ compiler found]) + else + dnl work around bug in taglib 1.5 headers, remove once there is a 1.6 + AS_CXX_COMPILER_FLAG([-Wno-attributes], [ + TAGLIB_CXXFLAGS="$TAGLIB_CFLAGS -Wno-attributes" + ], [ + TAGLIB_CXXFLAGS="$TAGLIB_CFLAGS" + ]) + AC_SUBST(TAGLIB_CXXFLAGS) fi ]) diff --git a/ext/taglib/Makefile.am b/ext/taglib/Makefile.am index b52916e7c8..ec01d4f97f 100644 --- a/ext/taglib/Makefile.am +++ b/ext/taglib/Makefile.am @@ -7,8 +7,8 @@ libgsttaglib_la_CFLAGS = \ $(TAGLIB_CFLAGS) libgsttaglib_la_CXXFLAGS = \ $(GST_PLUGINS_BASE_CFLAGS) \ - $(GST_CXXFLAGS) -Wno-attributes \ - $(TAGLIB_CFLAGS) + $(GST_CXXFLAGS) \ + $(TAGLIB_CXXFLAGS) libgsttaglib_la_LIBADD = \ $(GST_PLUGINS_BASE_LIBS) -lgsttag-$(GST_MAJORMINOR) \ $(GST_LIBS) \