[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kwrite-devel
Subject:    Re: Review Request: Use ##Alerts in python.xml (syntax hl)
From:       Dominik Haumann <dhdev () gmx ! de>
Date:       2009-07-08 10:29:54
Message-ID: 200907081229.55233.dhdev () gmx ! de
[Download RAW message or body]

On Tuesday 07 July 2009, Matthew Woehlke wrote:
> Dominik Haumann wrote:
> > On Tuesday 07 July 2009, Matthew Woehlke wrote:
> >> Dominik Haumann wrote:
> >>> On Monday 06 July 2009, Matthew Woehlke wrote:
> >>>> Dominik Haumann wrote:
> >>>>> -----------------------------------------------------------
> >>>>> This is an automatically generated e-mail. To reply, visit:
> >>>>> http://reviewboard.kde.org/r/896/#review1431
> >>>>> -----------------------------------------------------------
> >>>>>
> >>>>> Ship it!
> >>>>>
> >>>>>
> >>>>> looks good, ok to commit :)
> >>>>
> >>>> Thanks; committed to trunk.
> >>>>
> >>>> Think this is worth backporting to 4.3? It's technically a "feature"
> >>>> but it's pretty tiny (and no string changes or anything like that
> >>>> AFAIK).
> >>>
> >>> If you think it's worth backporting go ahead :)
> >>
> >> Thanks :-). I figure it's a nice feature for little cost, seems a
> >> shame for it to get stuck waiting for 4.4. But before I do...
> >>
> >> I saw your reply to the commit twice, but didn't see a reply to my
> >> response; maybe a mailer problem on one of our ends? Question was, is
> >> "1.100" safe as the new version number ("1.100" > "1.99"), or do I
> >> need to use something else?
> >
> > make it n.nn, i.e 2.00 :)
>
> Bleh, ##Alerts is *not* a major version bump :-).

This is not about major and minor anyway. It's simply about being able to 
distinguish versions. Imo it should simply have been an int.

Dominik
_______________________________________________
KWrite-Devel mailing list
KWrite-Devel@kde.org
https://mail.kde.org/mailman/listinfo/kwrite-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic