[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

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



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


--
Jerome BENOIT | calculus+at-rezozer^dot*net
https://qa.debian.org/developer.php?login=calculus@rezozer.net
AE28 AE15 710D FF1D 87E5  A762 3F92 19A6 7F36 C68B


Reply to: