Skip to content

build: make Conan optional and require quiver - #11

Merged
mtao merged 3 commits into
mainfrom
fix/conan-options
Aug 21, 2026
Merged

build: make Conan optional and require quiver#11
mtao merged 3 commits into
mainfrom
fix/conan-options

Conversation

@mtao

@mtao mtao commented Apr 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep Meson with system packages/subprojects as the default build path
  • retain an optional, zipper-style Conan recipe that only provides dependencies
  • stop mirroring Meson feature options into Conan, avoiding stale-option failures
  • support opt-in Conan builds through setup.sh --conan [profile]
  • make quiver an unconditional dependency, matching geometry and visualization's existing API usage
  • remove dead non-quiver BoundingBox and AlternatingDigitalTree branches
  • simplify build documentation and default Python requirements

Testing

  • conan inspect .
  • ./setup.sh --build-dir build-review -Dvisualization=false -Dexamples=false
  • meson test -C build-review -v
  • all 7 enabled test suites pass

mtao added 2 commits April 13, 2026 15:18
PR #6 removed these three options from meson_options.txt but
conanfile.py still listed them in __OPTIONAL_FLAGS_WITH_DEPS__.
The generate() method pushed all entries as meson project options,
causing 'Unknown options: eltopo, igl, pngpp' on every Conan CI run.
@mtao mtao changed the title fix: remove eltopo/igl/pngpp from conanfile.py to match meson_options build: remove Conan and require quiver Aug 21, 2026
@mtao mtao changed the title build: remove Conan and require quiver build: make Conan optional and require quiver Aug 21, 2026
@mtao
mtao merged commit d21b55f into main Aug 21, 2026
0 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant