-
Notifications
You must be signed in to change notification settings - Fork 2
Open
Description
In loader.py, we include config_autogen.h as follows:
ogdf-python/src/ogdf_python/loader.py
Line 55 in f487b2c
| cppyy.include("ogdf/basic/internal/config_autogen.h") |
and use the include directories defined as follows:
ogdf-python/src/ogdf_python/loader.py
Lines 32 to 46 in f487b2c
| if "OGDF_INSTALL_DIR" in os.environ: | |
| INSTALL_DIR = os.path.expanduser(os.getenv("OGDF_INSTALL_DIR")) | |
| cppyy.add_include_path(os.path.join(INSTALL_DIR, "include")) | |
| cppyy.add_library_path(os.path.join(INSTALL_DIR, "lib")) # TODO windows? | |
| if "OGDF_BUILD_DIR" in os.environ: | |
| BUILD_DIR = os.path.expanduser(os.getenv("OGDF_BUILD_DIR")) | |
| cppyy.add_include_path(os.path.join(BUILD_DIR, "include")) | |
| cppyy.add_include_path(os.path.join(BUILD_DIR, "..", "include")) # TODO not canonical | |
| cppyy.add_library_path(BUILD_DIR) | |
| try: | |
| wheel_inst_dir = importlib_resources.files("ogdf_wheel") / "install" | |
| if wheel_inst_dir.is_dir(): | |
| cppyy.add_library_path(str(wheel_inst_dir / "lib")) | |
| cppyy.add_library_path(str(wheel_inst_dir / "bin")) | |
| cppyy.add_include_path(str(wheel_inst_dir / "include")) |
However, because of ogdf/ogdf#253, config_autogen.h now is not anymore in ...include/ogdf/basic/ (like the other header files) but in ...include/ogdf-release/ogdf/basic/ or ...include/ogdf-debug/ogdf/basic/. So we need an additional include directory. Otherwise using ogdf-python with the newest ogdf fails.
Metadata
Metadata
Assignees
Labels
No labels