Thread: meson: pkgconfig difference

meson: pkgconfig difference

From
Peter Eisentraut
Date:
I think there is a tiny typo in src/interfaces/ecpg/ecpglib/meson.build:

diff --git a/src/interfaces/ecpg/ecpglib/meson.build 
b/src/interfaces/ecpg/ecpglib/meson.build
index dba9e3c3d9..da8d304f54 100644
--- a/src/interfaces/ecpg/ecpglib/meson.build
+++ b/src/interfaces/ecpg/ecpglib/meson.build
@@ -57,7 +57,7 @@ pkgconfig.generate(
    description: 'PostgreSQL libecpg library',
    url: pg_url,
    libraries: ecpglib_so,
-  libraries_private: [frontend_shlib_code, thread_dep],
+  libraries_private: [frontend_stlib_code, thread_dep],
    requires_private: ['libpgtypes', 'libpq'],
  )

This makes it match the other libraries.

Without this change, we get

Libs.private: ... -lpgport_shlib -lpgcommon_shlib

which seems wrong.



Re: meson: pkgconfig difference

From
Andres Freund
Date:
Hi,

On January 31, 2023 11:40:52 PM PST, Peter Eisentraut <peter.eisentraut@enterprisedb.com> wrote:
>I think there is a tiny typo in src/interfaces/ecpg/ecpglib/meson.build:
>
>diff --git a/src/interfaces/ecpg/ecpglib/meson.build b/src/interfaces/ecpg/ecpglib/meson.build
>index dba9e3c3d9..da8d304f54 100644
>--- a/src/interfaces/ecpg/ecpglib/meson.build
>+++ b/src/interfaces/ecpg/ecpglib/meson.build
>@@ -57,7 +57,7 @@ pkgconfig.generate(
>   description: 'PostgreSQL libecpg library',
>   url: pg_url,
>   libraries: ecpglib_so,
>-  libraries_private: [frontend_shlib_code, thread_dep],
>+  libraries_private: [frontend_stlib_code, thread_dep],
>   requires_private: ['libpgtypes', 'libpq'],
> )
>
>This makes it match the other libraries.
>
>Without this change, we get
>
>Libs.private: ... -lpgport_shlib -lpgcommon_shlib
>
>which seems wrong.

Ugh, yes, that's wrong. Do you want me to apply the fix?

Regards,

Andres

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.



Re: meson: pkgconfig difference

From
Peter Eisentraut
Date:
On 01.02.23 08:55, Andres Freund wrote:
> Hi,
> 
> On January 31, 2023 11:40:52 PM PST, Peter Eisentraut <peter.eisentraut@enterprisedb.com> wrote:
>> I think there is a tiny typo in src/interfaces/ecpg/ecpglib/meson.build:
>>
>> diff --git a/src/interfaces/ecpg/ecpglib/meson.build b/src/interfaces/ecpg/ecpglib/meson.build
>> index dba9e3c3d9..da8d304f54 100644
>> --- a/src/interfaces/ecpg/ecpglib/meson.build
>> +++ b/src/interfaces/ecpg/ecpglib/meson.build
>> @@ -57,7 +57,7 @@ pkgconfig.generate(
>>    description: 'PostgreSQL libecpg library',
>>    url: pg_url,
>>    libraries: ecpglib_so,
>> -  libraries_private: [frontend_shlib_code, thread_dep],
>> +  libraries_private: [frontend_stlib_code, thread_dep],
>>    requires_private: ['libpgtypes', 'libpq'],
>> )
>>
>> This makes it match the other libraries.
>>
>> Without this change, we get
>>
>> Libs.private: ... -lpgport_shlib -lpgcommon_shlib
>>
>> which seems wrong.
> 
> Ugh, yes, that's wrong. Do you want me to apply the fix?

I've done it now.