Bug 72584 - -isystem a -isystem b -isystem c is munged to -isystem a b c
Summary: -isystem a -isystem b -isystem c is munged to -isystem a b c
Status: RESOLVED FIXED
Alias: None
Product: pkg-config
Classification: Unclassified
Component: src (show other bugs)
Version: unspecified
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: pkg-config
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-11 05:16 UTC by Alan W. Irwin
Modified: 2013-12-14 18:53 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Alan W. Irwin 2013-12-11 05:16:31 UTC
The following *.pc file illustrates the issue:

libdir=/home/software/plplot_svn/HEAD/comprehensive_test_disposeable/shared/install_tree/lib
includedir=/home/software/plplot_svn/HEAD/comprehensive_test_disposeable/shared/install_tree/include/plplot
drvdir=/home/software/plplot_svn/HEAD/comprehensive_test_disposeable/shared/install_tree/lib/plplot5.9.10/driversd

Name: PLplot Qt
Description: Scientific plotting library (Qt4 bindings, double precision)
Requires.private: plplotd
Version: 5.9.10
Libs: -L${libdir} -lplplotqtd -L/home/wine/newstart/build_script/install-linux/lib -lQtSvg -L/home/wine/newstart/build_script/install-linux/lib -lQtGui -L/home/wine/newstart/build_script/install-linux/lib -lQtCore
Cflags: -I${includedir} -isystem /home/wine/newstart/build_script/install-linux/include -isystem /home/wine/newstart/build_script/install-linux/include/QtSvg -isystem /home/wine/newstart/build_script/install-linux/include/QtGui -isystem /home/wine/newstart/build_script/install-linux/include/QtCore

The result for the -cflags option is

-isystem /home/wine/newstart/build_script/install-linux/include /home/wine/newstart/build_script/install-linux/include/QtSvg /home/wine/newstart/build_script/install-linux/include/QtGui /home/wine/newstart/build_script/install-linux/include/QtCore -I/home/software/plplot_svn/HEAD/comprehensive_test_disposeable/shared/install_tree/include/plplot

I assume the missing -isystem tags in that result are because pkg-config is being too enthusiastic about cleaning up repeat tags.  But in the -isystem case they should be left as they are.
Comment 1 Dan Nicholson 2013-12-14 18:53:36 UTC
Thanks for the report. Should be fixed in master. See commit abdbaba. This probably needs to be more robust against all the other -ifoo gcc options, but that's a project for another day.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.