[Debian-science-sagemath] arb: headers with generic names

[Ximin Luo]

Jerome BENOIT:
> On 10/09/18 19:13, Erik Bray wrote:
>> On Mon, Sep 10, 2018 at 7:12 AM Ximin Luo <infinity0 at debian.org> wrote:
>>>
>>> Julien Puydt:
>>>> [..]
>>>>
>>>> Upstream, then, as there's nothing I can do about it in Debian...
>>>>
>>>
>>> Well, we can install on-purpose into a Debian-specific directory like /usr/include/arb instead of /usr/include. Then all the packages that uses arb will have to add -I /usr/include/arb to their compiler flags, or #include <arb/xxx.h> instead of #include <xxx.h>.
>>>
>>> In fact I swear we were doing this already, and that we were happy to maintain this patch from upstream because what upstream does is so ridiculous. Did this get reverted or something?
>>>
>>> Of course the best solution is to have upstream adopt best practises, but here the situation IMO is too ridiculous to tolerate, we should just fix it in Debian for now, and keep maintaining the patch - which should not be that bad, just mechanical renames.
>>
>> Rather than patching the sources, wouldn't it make more sense to just
>> add `-I/usr/include/arb` where necessary?  This can go in a .pc file
>> for arb.
> 
> A .pc file would be very fine, but arb is at the end of a chain: gmp, mpfr, flint, arb
> gmp, mpfr, and flint do not come with a .pc 
> 
> For this reason, I guess arb must follows its predecessors.
> gmp and mpfr have basically one header, flint a bunch of them gathered in a folder.
> For this respect arb is more like flint: its headers may be gathered in a folder.
> 
> Otherwise, I think that submitted a patch to the upstream author is the best approach.
> 

Putting them in a separate folder solves the problem of co-installation, but does not solve the problem of potentially-conflicting "#include <>" directives. To solve this problem the sources (and anyone depending on them) do have to be patched, ideally by upstream.

X

-- 
GPG: ed25519/56034877E1F87C35
GPG: rsa4096/1318EFAC5FBBDBCE
https://github.com/infinity0/pubkeys.git