pkgconfig: Add '-I${includedir}' to Cflags #57
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Don't just specify
-I${includedir}/cupsfilters
,but also
-I${includedir}
in the Cflagsin the pkgconfig file.
This way, includes prefixed with the "cupsfilters/" directory path also work.
This e.g. makes the libppd build work when
libcupsfilters is installed in a non-default path
(e.g. configured with
--prefix=$HOME/temp/cpdb
)and libppd is configured to use that one
(e.g. by setting env variable
PKG_CONFIG_PATH=$HOME/temp/cpdb/lib/pkgconfig/
during configure).
Without this commit, libppd's configure would
succeed in the above scenario, but the actual build using
make
would then fail to find the cupsfilters headers:(This is with libppd [1] git master as of commit
327b61cf5ec3bc74677e3b56fde4c8c4e9acdcb7).
[1] https://github.com/OpenPrinting/libppd