Re: [Tracker] Merging the indexer-split branch to TRUNK

[Martyn Russell <martyn imendio com>]

Michael Biebl wrote:
> 2008/9/25 Martyn Russell <martyn imendio com>:
> > Michael Biebl wrote:
> > > 2008/9/25 Martyn Russell <martyn imendio com>:
> > > > Hi all,
> > > >
> > > > After the hackfest this past weekend Jamie and I resolved the final
> > > > issues for the merge to go ahead with everyone's consent.
> > > >
> > > > So the plan is to prepare for the merge today and to do the actual merge
> > > > tomorrow.
> > > >
> > > > So far the planning consists of:
> > > >
> > > > 1. Merging all .po files over from TRUNK to the indexer-split branch.
> > > > 2. Fixing any remaining migration paths for binaries.
> > > > 3. Fixing database migration paths.
> > > > 4. Commit any outstanding patches (e.g. the IMAP uid=NULL patch).
> > > >
> > > > If there is anything here I have missed, please let me know.
> > > >
> > > > So far #1 and #2 are done. We are thinking about how to detect #3 and in
> > > > that case to force a reindex (which effectively removes the database
> > > > files and starts again). As for #4, this should be done some time today.
> > > Please remove this again from Makefile.am:
> > >
> > > # Clean up any old trunk files that might exist which we don't use now
> > > all-local:
> > >         rm -Rf $(bindir)/trackerd
> > >         rm -Rf $(bindir)/tracker-indexer
> > >         rm -Rf $(bindir)/tracker-thumbnailer
> > >         rm -Rf $(bindir)/tracker-extract
> > >         rm -Rf $(datadir)/tracker
> > >
> > >
> > > This is really ugly.
> > I don't really care how beautiful it is. I care about bug reports and
> > people complaining on this mailing list when they can't run the project
> > because of old binaries which conflict with new binaries. Jamie has
> > already ran into this. Without this there is *NO* migration path. Are
> > you really saying we shouldn't care about that?
>
> The vast majority of users will use via their favourite distribution,
> where the package management system will take care of it. So imho your
> point is almost moot.

"The vast majority of users" is not everyone. So my point is not moot.

> For people actually reading the mailing list and testing trunk, it
> should be enough to announce this issues via email.

Yes I agree. But people don't always do that.

> Right now I'm unable to actually *compile* tracker as unprivileged
> user, as it tries to remove /usr/share/tracker.

Right, I had a fix a while back but I was testing something else. I just
committed it. It should work now.

> Deleting stuff without notice is a no-no too, imho

This is not deleting without notice this is deleting OLDER project files
which can not co-exist if you want Tracker to work.

> If you are worried about users testing svn snapshots, make an install
> hook which issues a warning at the end of the install phase, if old
> binaries are found, but please don't silently nuke existing data.

This isn't silently doing it at all, it is at the end of the install
process and you can see what it is doing.

In addition to all this, Jamie has requested that we do this so others
don't just upgrade from their current TRUNK to TRUNK post indexer-split
merge and find nothing works for them.

Really, I don't like that we have to do this, but I don't like (even
more) users complaining and then having to explain and/or debug
something I could have avoided.

-- 
Regards,
Martyn