Le 3 août 2024 20:53:24 GMT+02:00, James Lu <ja...@overdrivenetworks.com> a 
écrit :
>Control: tags -1 + pending
>
>Hi Aurélien,

Hi James,

>I will push a fix adding D: libp11-kit-dev to libqalculate-dev.

I won't prevent you from doing that and it's going to fix my build issue for 
plasma-workspace, so fine with me.

But I'm wondering if that's the correct thing to do. We used to be able to 
build fine without having libp11-kit-dev in our build dependencies (including 
transitive).

Given that, and as I cannot find any trace of any of the libp11-kit-dev headers 
in libqalculate's source, it looks more like a false positive to me and that 
the correct fix would be *not to have* libp11-kit in your pc file.


>Strangely, this part of the .pc file is templated[1] and hasn't changed in 3 
>years, so I think something has changed in the dependency stack.

Yes, I think so too but I cannot understand why either.

If I'm not mistaken looking a level deeper the pc file for libcurl4-gnutls-dev 
is strictly the same before and after your build that picked up libp11-kit-dev 
so it only makes it more mysterious to me.


Anyway I won't dig into this issue any further, if your fine with that probably 
unused build dependency, so be it.


Thank for looking into it !
--
Aurélien

Reply via email to