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 684621 - patch to handle CXXFLAGS in libxslt.m4
patch to handle CXXFLAGS in libxslt.m4
Status: RESOLVED INCOMPLETE
Product: libxslt
Classification: Platform
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: Daniel Veillard
libxml QA maintainers
Depends on:
Blocks:
 
 
Reported: 2012-09-22 13:50 UTC by Pacho Ramos
Modified: 2018-04-21 14:42 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
libxslt.m4-libxslt-1.1.26.patch (1.39 KB, patch)
2012-09-22 13:50 UTC, Pacho Ramos 		none Details | Review

Description Pacho Ramos 2012-09-22 13:50:31 UTC 
Created attachment 224976 [details] [review]
libxslt.m4-libxslt-1.1.26.patch

Hello

In Gentoo we are carrying this old patch for a long time to make libxslt.m4 honor CXXFLAGS, could it be committed to master?

Thanks
Comment 1 Nick Wellnhofer 2016-04-29 14:16:17 UTC 
Why is this patch needed exactly? The libxslt build shouldn't use any C++ compiler flags.
Comment 2 Mart Raudsepp 2018-04-21 14:42:02 UTC 
Not sure, I made a new version of our package not do that now. Lets just see if there will be any issues from not having it.
My best guess would be that maybe in the 2012 era, there was a AC_LANG([C++]) or some such in configure.ac, or in some other cases a C++ compiler could have been called in those tests. The patch was introduced with no good details, just as part of "version bump" and I think there's no point chasing it more. Lets just see how it goes. Closing this bug accordingly and will reopen with more info, if it turns out necessary.