Re: [RFC virt-viewer 10/12] spec: Take Epoch into account in package buildid

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux