Re: [igraph] [PATCH] Two important packaging improvements for igraph

Jerome BENOIT <[email protected]>
Newsgroups gmane.linux.debian.devel.medical
Organization ReZoZeR
Message-ID <[email protected]>
Hello Aymeric, thank for the report. However it would be more appropriate and more efficient to fill a bug report.

Concerning the first issue,
the generated files are put in the `/usr/include/<ARCHTRIPLE>/igraph` purposely because they are generated on a specific arch.
Note that the dev- package installs a pkgconf(1) meta data file which we want to use. You may want to have a look to the Makfile_s
in the examples folders. These are used by the autotests, so they should work.

Concerning the second issue,
I was not aware. I will have a closer look this week-end.

Best wishes,
Jerome

On 06/03/2025 18:02, Aymeric Agon-Rambosson wrote:
> 
> Hello everyone, and sorry for the duplicated receptions,
> 
> There are a couple of issues in the debian packaging of igraph.
> 
> First, in the package libigraph-dev, some of header files (4 of them, to be specific, the ones that are generated by preprocessing), are put in the directory /usr/include/x86_64-linux-gnu/igraph, instead of /usr/include/igraph/.
> 
> This is a problem, since they expect to be in the same directory (igraph.h includes igraph_version.h with the following statement #include "igraph_version.h"). This makes the library basically unusable for development (trying to compile any C source file that includes igraph.h fails immediately).
> 
> Patching d/libigraph-dev.install solves the issue.
> 
> The second issue is about the info file. The rename from igraph-docs to igraph breaks the references inside the info file itself. For instance, in the current state in unstable, the dir file in /usr/share/info carries the wrong file name (igraph-docs).
> 
> I had proposed some time ago a sed command to make the replacement, it turns out it was not wide enough. I have a patch to d/rules that widens the sed command accordingly, and solves the issue.
> 
> Here are the relevant patches, feel free to adapt.
> 
> Best,
> 
> Aymeric
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.