On Wed, Jun 03, 2015 at 09:56:21AM -0500, Jonathon Jongsma wrote: > On Tue, 2015-06-02 at 16:29 +0200, Christophe Fergeau wrote: > > --- > > virt-viewer.spec.in | 8 +++++++- > > 1 file changed, 7 insertions(+), 1 deletion(-) > > > > diff --git a/virt-viewer.spec.in b/virt-viewer.spec.in > > index dfc819c..b75ac45 100644 > > --- a/virt-viewer.spec.in > > +++ b/virt-viewer.spec.in > > @@ -104,7 +104,13 @@ autoreconf -if > > %define govirt_arg --with-ovirt > > %endif > > > > -%configure %{spice_arg} %{gtk_arg} %{govirt_arg} --with-buildid=%{release} --disable-update-mimedb > > +%if 0%{epoch} > > +%define buildid %{epoch}:%{release} > > +%else > > +%define buildid %{release} > > +%endif > > + > > +%configure %{spice_arg} %{gtk_arg} %{govirt_arg} --with-buildid=%{buildid} --disable-update-mimedb > > %__make %{?_smp_mflags} > > > > > > > It doesn't seem that virt_viewer_compare_buildid() currently handles the > epoch? It'd be good to add a test case for that. Yes, it does not handle it, I initially added that patch as it makes sense, and forgot to avoid sending it /o\ Christophe
Attachment:
pgpdKjpDO8338.pgp
Description: PGP signature
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list