Just checked in.
Thanks Erich.
Cheers,
Bernard
On 3/4/07, Erich Focht <[EMAIL PROTECTED]> wrote:
> On Sunday 04 March 2007 11:46, Bernard Li wrote:
> > Guys:
> >
> > I plan to check the following patch in so that we can develop for test
> > releases of Fedora Core correctly:
> >
> > Index: lib/OSCAR/OCA/OS_Detect/Fedora.pm
> > ===================================================================
> > --- lib/OSCAR/OCA/OS_Detect/Fedora.pm (revision 5713)
> > +++ lib/OSCAR/OCA/OS_Detect/Fedora.pm (working copy)
> > @@ -48,6 +48,8 @@
> >
> > if ($release_string =~ /Fedora Core release (\d+)/) {
> > $fc_release = $1;
> > + } elsif ($release_string =~ /Fedora release (\d+).91
> > \(Rawhide\)/) { # Fedora Core test releases
> > + $fc_release = $1+1;
>
> Okay, but why just for 6.91 ??? Make it generic and do something like
> + } elsif ($release_string =~ /Fedora release (\d+)\.9\d \(Rawhide\)/) { #
> Fedora Core test releases
>
>
> BTW, don't forget to escape the dot.
>
> Regards,
> Erich
>
>
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Oscar-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/oscar-devel
