Gentoo Logo
Gentoo Spaceship




Note: Due to technical difficulties, the Archives are currently not up to date. GMANE provides an alternative service for most mailing lists.
c.f. bug 424647
List Archive: gentoo-dev
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Headers:
To: gentoo-dev@g.o
From: Roy Marples <uberlord@g.o>
Subject: Re: RFC: sh versionator.eclass
Date: Tue, 02 Oct 2007 07:26:39 +0100
On Mon, 2007-10-01 at 20:22 -0400, Mike Frysinger wrote:
> "better readability" is a pretty subjective claim ...

Many things are subjective - one mans good is another mans bad.
/me shrugs

>  i'd say that changing 
> things from doing a case match against proper character classes to doing a 
> printf against arbitrary character ranges (which btw are not locale safe, so 
> i imagine you've introduced a regression there) is not "better".  but i'm not 
> the maintainer of said eclass.

That is why the case match is like so

[0-9]      Should be safe - every language should operate in base 10
-|.|_      Exact match
*          If it's not the above, treat it like a letter

So there shouldn't be any regressions with version numbers here as I've
cunningly avoided the locale trap.

> > > but I find interesting that
> > > you ripped Ciaran's copyright while leaving the "Prod ciaranm if you
> > > find something it can't handle" comment.
> >
> > I copied and pasted the comments for the most part. Sorry for leaving that
> > bit in. And I thought that the Gentoo Foundation had the copyright?
> 
> it does.  and if it doesnt, punt the file and rewrite it from scratch.

Well, this should constitute a rewrite from scratch.

Thanks

Roy

-- 
gentoo-dev@g.o mailing list


Replies:
Re: RFC: sh versionator.eclass
-- Mike Frysinger
Re: RFC: sh versionator.eclass
-- Duncan
References:
RFC: sh versionator.eclass
-- Roy Marples
Re: RFC: sh versionator.eclass
-- Fernando J. Pereda
Re: RFC: sh versionator.eclass
-- Roy Marples
Re: RFC: sh versionator.eclass
-- Mike Frysinger
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
Re: RFC: sh versionator.eclass
Next by thread:
Re: RFC: sh versionator.eclass
Previous by date:
Re: [gentoo-commits] gentoo-x86 commit in dev-lang/swi-prolog: ChangeLog swi-prolog-5.6.41.ebuild
Next by date:
Re: RFC: sh versionator.eclass


Updated Jun 17, 2009

Summary: Archive of the gentoo-dev mailing list.

Donate to support our development efforts.

Copyright 2001-2013 Gentoo Foundation, Inc. Questions, Comments? Contact us.