expose optparse API publicly #702
Conversation
|
Hm, does that mean we should do manpages for libflux-optparse? 😨 |
|
Maybe someday. IMHO not in this PR. |
|
Looks like mpich.org is back up so I restarted the Travis build for this PR |
|
Thanks! Still no progress on travis-ci/apt-package-safelist#406 it seems. |
Build optparse as an installed shared library.
Install library to ${libdir}/libflux-optparse.so.
Install header to ${includedir}/flux/optparse.h
so it can be included with <flux/optparse.h>.
Note: also add src/include/flux/optparse.h wrapper,
so core sources can incdlue <flux/optparse.h> as well.
Add -lflux-optparse to flux-core.pc so that framework projects can use optparse with minimal fuss.
garlick
force-pushed
the
libflux-optparse
branch
from
6439d49
to
7d286fd
Compare
|
Just rebased on current master and squashed the one incremental commit. |
|
The coverage test got stuck between the two cron tests :-(. There was no information I could glean from the travis logs so I just restarted it. Since everything else looked fine, is this ready to merge? |
|
Yep
|
This PR moves optparse from a static library internal to flux-core to a public shared library, installed as
${libdir}/libflux-optparse.soand${includedir}/flux/optparse.h.This allows flux subcommands built outside of the core source tree to use the same option handling code, which should promote a more uniform user interface across framework components.