This PR changes astcanopy build system to the following:
Use cmake / other build tools to build libastcanopy
Change pylibastcanopy's library discovery via find_package
Undo some custom build wrangling in #45
Additionally, this PR restructures the cpp project layout following C++ header layout convention (<ast_canopy/ast_canopy.hpp>).
This PR also removes cuda-tookit dependency for ast_canopy. Partially contributing to #53.
One question left unanswered is that ast-canopy recipe explicitly needs {{ compiler('c') }} in run/host. While cuda-nvcc already depends on some host compilers, nvcc needs the gcc binary to correctly function, and gcc_linux-64 package doesn't provide this binary. This makes me feel that cuda-nvcc package recipe is incomplete. But I'm not sure, and it should not be blocking this PR. See this CI job for more detail.
isVoid
commented
3 months ago
This PR changes astcanopy build system to the following:
find_packageAdditionally, this PR restructures the cpp project layout following C++ header layout convention (
<ast_canopy/ast_canopy.hpp>).This PR also removes
cuda-tookitdependency forast_canopy. Partially contributing to #53.One question left unanswered is that ast-canopy recipe explicitly needs
{{ compiler('c') }}in run/host. Whilecuda-nvccalready depends on some host compilers, nvcc needs thegccbinary to correctly function, andgcc_linux-64package doesn't provide this binary. This makes me feel thatcuda-nvccpackage recipe is incomplete. But I'm not sure, and it should not be blocking this PR. See this CI job for more detail.Fixes #52