flux-framework / dyad

DYAD: DYnamic and Asynchronous Data Streamliner
GNU Lesser General Public License v3.0
7 stars 5 forks source link

Makes Core into Core and Client and Exposes Client API to Users #120

Open ilumsden opened 1 week ago

ilumsden commented 1 week ago

This PR is based on feedback from #118. If we like it, it can supersede and replace #118.

This PR moves headers around and modifies the CMake to make DYAD's core and ctx libraries available to users. It also splits dyad_core.h and dyad_structures.h into multiple headers (an external header and an internal one) so that excess function definitions are not exposed to users. It also creates a new dyad/core/dyad.h header and dyad_client CMake target to refer to both libraries at once.

ilumsden commented 1 week ago

I rebased this PR on top of #116 for testing purposes. Please merge #116 before merging this PR.

ilumsden commented 1 week ago

I tested this using A4X, and it works as expected