search

DUNE-DAQ / appfwk

DUNE DAQ Application Framework Repository (implementations that use interfaces in app-framework-base)
5 stars 5 forks source link

JCF: given changes due to DUNE-DAQ/daq-cmake#100, make sure that file… #266

Closed jcfreeman2 closed 1 year ago

jcfreeman2 commented 1 year ago

…ConfFacility and dbConfFacility link to cetlib so that all functions in ConfFacility.hpp have definitions

UPSHOT: there's no need to merge this until a version of daq-cmake which includes the PR DUNE-DAQ/daq-cmake#101 has been cut.

eflumerf commented 1 year ago

Shouldn't the cetlib targets be added to the INTERFACE cmake definition of duneconffacility so that users don't have to worry about it?

jcfreeman2 commented 1 year ago

Closing this PR because in https://github.com/DUNE-DAQ/daq-cmake/pull/101 cetlib gets automatically linked in to the plugin if available, thus the change here is no longer necessary.