iLCSoft / LCIO

Linear Collider I/O
BSD 3-Clause "New" or "Revised" License
17 stars 34 forks source link

Use `sio` via git submodules #179

Closed tmadlener closed 7 months ago

tmadlener commented 7 months ago

Keeping the sio sources here and in the upstream repo in sync is potentially more work than it needs to be and it might be easier to fix issues like #178 in upstream and then directly pick them up here if SIO would be included as a git submodule here.

arummler commented 7 months ago

I would propose to avoid submodules; better to use FetchContent. Mixing git and cmake methods otherwise quickly becomes tricky. Particularly for downstream consumers.