mbox series

[bug#66889,0/2] Unbundle igraph dependencies.

Message ID cover.1698878710.git.david.elsing@posteo.net
Headers show
Series Unbundle igraph dependencies. | expand

Message

David Elsing Nov. 1, 2023, 10:53 p.m. UTC 
These patches unbundle everything from igraph and update it to 0.10.7.
As newer SuiteSparse versions always use an integer width of 64, I set
the same in igraph. Would it be better to create a package variant of
suitesparse-cxsparse for igraph with an integer width depending on the
architecture? Upstream doesn't recommend unbundling CXSparse for that
reason.

David Elsing (2):
  gnu: Add pcg-c.
  gnu: igraph: Update to 0.10.7.

 gnu/local.mk                                  |   1 +
 gnu/packages/c.scm                            |  55 +++++++++
 gnu/packages/graph.scm                        | 114 +++++++++++++-----
 .../igraph-fix-varargs-integer-size.patch     |  39 ++++++
 4 files changed, 182 insertions(+), 27 deletions(-)
 create mode 100644 gnu/packages/patches/igraph-fix-varargs-integer-size.patch

Comments

Ludovic Courtès Dec. 2, 2023, 10:58 a.m. UTC | #1 
Hi David,

David Elsing <david.elsing@posteo.net> skribis:

> These patches unbundle everything from igraph and update it to 0.10.7.
> As newer SuiteSparse versions always use an integer width of 64, I set
> the same in igraph. Would it be better to create a package variant of
> suitesparse-cxsparse for igraph with an integer width depending on the
> architecture? Upstream doesn't recommend unbundling CXSparse for that
> reason.

I don’t know what would be better, but what you did here LGTM.

> David Elsing (2):
>   gnu: Add pcg-c.
>   gnu: igraph: Update to 0.10.7.

Finally applied, thanks!

Ludo’.