Re: [Lxr-dev] FW: [Lxr-commits] CVS: lxr/lib/LXR/Index Mysql.pm, 1.30, 1.31 Oracle.pm, 1.20, 1.21 DB.pm, 1.17, 1.18 Postgres.pm, 1.31, 1.32

[Malcolm B. <mal...@gm...>]

Hi,

On Sun, May 10, 2009 at 1:02 PM, Adrian Issott <adr...@ho...> wrote:
>
> I felt that rel wasn't as immediately
> understandable as releaseid so I changed all the columns to that. While
> doing a search and replace it struck me that the code would be cleaner if
> all the $release variables were renamed to $releaseid too to match up with
> the SQL so I've also done that which meant a whole lot of files have changed
> superficially.

releaseid is a better name - so this change looks fine to me.

>
> BTW, when we release v0.9.6, as result of the fix for bug 1209273, we're
> going to need to tell people that their existing databases won't work
> out-of-the-box with the new version ...

Yes, must remember to flag that up in the release notes and
announcement.  We've broken db compatibility in the past without
anyone complaining, so I guess it will be OK.

I suppose for extra points we could provide a migration script, but my
preference would be to only do that for stable versions (i.e. 1.0
onwards).

Malcolm