Skip to content

Commit

Permalink
Specify igraph < 0.10.0
Browse files Browse the repository at this point in the history
igraph 0.10.0 has some API changes that we don't handle. We may be able
to support it through some #ifdef macros, as we already do for 0.9.x,
though the devs suggest this might be painful.

For now, having cmake bail out and explain that the igraph version is
incompatible is better than having a bunch of compiler errors.

See #44
  • Loading branch information
sporksmith committed Jan 25, 2023
1 parent fa98ebd commit ae3e386
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,10 @@ include(FindPkgConfig)
find_package(RT REQUIRED)
find_package(M REQUIRED)

pkg_check_modules(IGRAPH REQUIRED igraph)
# We don't yet support igraph 0.10.0+, which has some API changes.
# See https://github.com/shadow/tgen/issues/44
pkg_check_modules(IGRAPH REQUIRED igraph<0.10.0)

pkg_check_modules(GLIB REQUIRED glib-2.0)

## Parse out igraph version. Needed to work around breaking API changes in igraph.
Expand Down

0 comments on commit ae3e386

Please sign in to comment.