diff mbox series

pkg-config: use the correct replacements for libdir/includedir

Message ID 20200419075201.1161001-1-eschwartz@archlinux.org (mailing list archive)
State New, archived
Headers show
Series pkg-config: use the correct replacements for libdir/includedir | expand

Commit Message

Eli Schwartz April 19, 2020, 7:52 a.m. UTC
They are defined pkg-config variables for a reason, let's reuse them as
is the intended usage of pkg-config. This ensures various pkg-config
features continue to work as expected.

Signed-off-by: Eli Schwartz <eschwartz@archlinux.org>
---
 libtirpc.pc.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Steve Dickson April 20, 2020, 5:47 p.m. UTC | #1
Hello,

On 4/19/20 3:52 AM, Eli Schwartz wrote:
> They are defined pkg-config variables for a reason, let's reuse them as
> is the intended usage of pkg-config. This ensures various pkg-config
> features continue to work as expected.
Just curious... What pkg-config feature does this fix?

steved.

> 
> Signed-off-by: Eli Schwartz <eschwartz@archlinux.org>
> ---
>  libtirpc.pc.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libtirpc.pc.in b/libtirpc.pc.in
> index 38034c5..863950f 100644
> --- a/libtirpc.pc.in
> +++ b/libtirpc.pc.in
> @@ -9,4 +9,4 @@ Requires:
>  Version: @PACKAGE_VERSION@
>  Libs: -L@libdir@ -ltirpc
>  Libs.private: -lpthread
> -Cflags: -I@includedir@/tirpc
> +Cflags: -I${includedir}/tirpc
>
Eli Schwartz April 20, 2020, 6:24 p.m. UTC | #2
On 4/20/20 1:47 PM, Steve Dickson wrote:
> Hello,
> 
> On 4/19/20 3:52 AM, Eli Schwartz wrote:
>> They are defined pkg-config variables for a reason, let's reuse them as
>> is the intended usage of pkg-config. This ensures various pkg-config
>> features continue to work as expected.
> Just curious... What pkg-config feature does this fix?

--define-variable might be used to remap paths.

But I see I forgot to git add -p all changes. :(
diff mbox series

Patch

diff --git a/libtirpc.pc.in b/libtirpc.pc.in
index 38034c5..863950f 100644
--- a/libtirpc.pc.in
+++ b/libtirpc.pc.in
@@ -9,4 +9,4 @@  Requires:
 Version: @PACKAGE_VERSION@
 Libs: -L@libdir@ -ltirpc
 Libs.private: -lpthread
-Cflags: -I@includedir@/tirpc
+Cflags: -I${includedir}/tirpc