Bug 72584

Summary: -isystem a -isystem b -isystem c is munged to -isystem a b c
Product: pkg-config Reporter: Alan W. Irwin <Alan.W.Irwin1234>
Component: srcAssignee: pkg-config
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: x86-64 (AMD64)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:

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.