Re: [PATCHv3 3/3] debian: Update volume-id for Squeeze

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

 



On Wed, Jan 02, 2013 at 06:58:56PM +0200, Zeeshan Ali (Khattak) wrote:
> On Tue, Jan 1, 2013 at 6:00 PM, Fabiano Fidêncio <fabiano@xxxxxxxxxxxx> wrote:
> > ---
> >  data/oses/debian.xml.in | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/data/oses/debian.xml.in b/data/oses/debian.xml.in
> > index fc0f6cc..349ba24 100644
> > --- a/data/oses/debian.xml.in
> > +++ b/data/oses/debian.xml.in
> > @@ -187,7 +187,7 @@
> >      <media arch="i386" live="true">
> >        <url>http://live.debian.net/cdimage/release/current/i386/iso-hybrid/debian-live-6.0.6-i386-gnome-desktop.iso</url>
> >        <iso>
> > -        <volume-id>Debian squeeze 20111014-08:13</volume-id>
> > +        <volume-id>Debian squeeze \d{8}-\d{2}:\d{2}</volume-id>
> >          <system-id>LINUX</system-id>
> >        </iso>
> >        <kernel>install/vmlinuz</kernel>
> > @@ -196,7 +196,7 @@
> >      <media arch="x86_64" live="true">
> >        <url>http://live.debian.net/cdimage/release/current/amd64/iso-hybrid/debian-live-6.0.6-amd64-gnome-desktop.iso</url>
> >        <iso>
> > -        <volume-id>Debian squeeze 20111014-08:11</volume-id>
> > +        <volume-id>Debian squeeze \d{8}-\d{2}:\d{2}</volume-id>
> 
> I thought we agreed that there is no need for date recognition here.
> i-e "Debian squeeze" should be both specific and generic enough?
> 
> I think thats also what teuf meant on IRC:
> 
> <teuf> [12:17:02] fidencio: even a truncated regex (ie stop before the
> date) would be good I think

For what it's worth, I mentioned this because it makes the regex simpler,
but the more specific matching from this patch would have been fine with me
too.

Christophe

Attachment: pgpcmU0QARVkK.pgp
Description: PGP signature

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux