On 31/07/14 15:18, Robert Schiele wrote: > intltool is needed for nls _and_ polkit, thus the check needs to be > changed to do the test whenever one of them is enables. > > Without this build fails when configured with > --disable-nls --enable-polkit > --- > configure.ac | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure.ac b/configure.ac > index 43b6eef..0802cfc 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -62,7 +62,7 @@ AS_IF([test x"$intltool_found" != xyes], > ]) > > AM_NLS > -AS_IF([test x"$enable_nls" != xno], [ > +AS_IF([test x"$enable_nls" != xno -o "x$enable_polkit" != xno], [ > # intltoolize greps for '^(AC|IT)_PROG_INTLTOOL', so it needs to be on > its own line > IT_PROG_INTLTOOL([0.40.0]) > ])
Isn't this scenario already covered by this existing configure.ac test... AS_IF([test -z "$INTLTOOL_POLICY_RULE"], [ # If intltool is not available, provide a dummy rule to fail generation of %.policy files with a meaningful error message INTLTOOL_POLICY_RULE='%.policy: %.policy.in ; @echo " ITMRG " $@ && echo "*** intltool support required to build target $@" && false' AC_SUBST(INTLTOOL_POLICY_RULE) ]) ...? _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel