search

flux-framework / flux-accounting

bank/accounting interface for the Flux resource manager
https://flux-framework.readthedocs.io/projects/flux-accounting/en/latest/index.html
GNU Lesser General Public License v3.0
3 stars 10 forks source link

repo: add `pkg.m4`, checks for flux-core libs #441

Closed cmoussa1 closed 7 months ago

cmoussa1 commented 7 months ago

Problem

The flux-accounting repo does not check for certain flux-core libraries that it will eventually need to use, such as the hostlist library.

This PR adds pkg.m4 to the config/ directory which will enable flux-accounting to check for flux-core libraries while it is building. It uncomments the PKG_CHECK_MODULES calls in ax_flux_core.m4 so that flux-accounting actually checks for these libraries, and finally adds output to configure.ac to show the location of flux-core's hostlist library.

cmoussa1 commented 7 months ago

Thanks for reviewing @grondo! Will set MWP here

codecov[bot] commented 7 months ago

Codecov Report

Merging #441 (522f3b1) into master (b6f7a7e) will not change coverage. The diff coverage is n/a.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #441 +/- ## ======================================= Coverage 83.30% 83.30% ======================================= Files 23 23 Lines 1545 1545 ======================================= Hits 1287 1287 Misses 258 258 ```