| Summary: | CPATH include paths are not removed from include flags | ||
|---|---|---|---|
| Product: | pkg-config | Reporter: | v4hn <me> |
| Component: | src | Assignee: | pkg-config |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | medium | CC: | nicholson |
| Version: | unspecified | ||
| Hardware: | Other | ||
| OS: | All | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: | working patch | ||
Pushed to master in commit 4ade8fc. Thanks for the patch! Thanks for merging! When is the next release planned? There have been a number of changes since 0.29.1 that I would like to see in an official project release. I'll be making a new release in the next couple days. |
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.
Created attachment 128687 [details] working patch Thanks for maintaining this module! While paths specified in `C_INCLUDE_PATH` and `CPLUS_INCLUDE_PATH` are automatically excluded from the output of include flags extracted by pkg-config, paths listed in `CPATH` are not. I see the following behavior over here: ``` v4hn ~/ros/indigo/core $ cat lib/pkgconfig/tf.pc prefix=/home/v4hn/ros/indigo/core Name: tf Description: Description of tf Version: 1.11.8 Cflags: -I/home/v4hn/ros/indigo/core/include Libs: -L/home/v4hn/ros/indigo/core/lib -ltf Requires: geometry_msgs message_filters message_runtime roscpp sensor_msgs std_msgs tf2_ros rosconsole v4hn ~/ros/indigo/core $ echo $CPATH /home/v4hn/ros/indigo/core/include v4hn ~/ros/indigo/core $ pkg-config --cflags tf -I/home/v4hn/ros/indigo/core/include ``` I would expect that pkg-config treats these three variables the same. I only noticed this because it creates unforeseen overlay problems in a software framework that heavily relies on pkgconfig. The attached patch is my proposed fix of this inconsistent behavior.