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 :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list