On Mon, Feb 21, 2011 at 10:44:13AM -0700, Eric Blake wrote:
* libvirt.spec.in (Requires): Add gnutls-utils, for virt-pki-validate. Suggested by Daniel P. Berrange. ---
-if [ ! -x $CERTOOL ] +if [ ! -x "$CERTOOL" ] then echo "Could not locate the certtool program" echo "make sure the gnutls-utils (or gnutls-bin) package is installed"
ACK, though we should have an RPM dep on gnutls-utils to make sure this is always present !
Like so?
libvirt.spec.in | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/libvirt.spec.in b/libvirt.spec.in index a29693a..b5c9fd9 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -418,6 +418,8 @@ Requires: ncurses Requires: nc # Needed by libvirt-guests init script. Requires: gettext +# Needed by virt-pki-validate script. +Requires: gnutls-utils %if %{with_sasl} Requires: cyrus-sasl # Not technically required, but makes 'out-of-box' config
Yeah that looks good Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|