On Tue, 2021-03-02 at 02:25 +0900, QI Fuli wrote: > Add checking jq command since it is needed to validate tests > > Cc: Santosh Sivaraj <[email protected]> > Signed-off-by: QI Fuli <[email protected]> > Link: https://github.com/pmem/ndctl/issues/141 > --- > configure.ac | 6 ++++++ > 1 file changed, 6 insertions(+)
Hm, I think I prefer how you did it in v1. i.e. no configure.ac check. In my view, configure.ac tests are for the core things needed to /run/ ndctl on a system. Development unit tests can just continue to use check_prereq as you did. So I'll pick up v1 of this for now - if you want me to do something else please let me know! > > diff --git a/configure.ac b/configure.ac > index 5ec8d2f..839836b 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -65,6 +65,12 @@ fi > AC_SUBST([XMLTO]) > fi > > +AC_CHECK_PROG(JQ, [jq], [$(which jq)], [missing]) > +if test "x$JQ" = xmissing; then > + AC_MSG_ERROR([jq command needed to validate tests]) > +fi > +AC_SUBST([JQ]) > + > AC_C_TYPEOF > AC_DEFINE([HAVE_STATEMENT_EXPR], 1, [Define to 1 if you have statement > expressions.]) > > -- > 2.29.2 > _______________________________________________ > Linux-nvdimm mailing list -- [email protected] > To unsubscribe send an email to [email protected] _______________________________________________ Linux-nvdimm mailing list -- [email protected] To unsubscribe send an email to [email protected]
