olive vs bzr-gtk

[Szilveszter Farkas]

> I would prefer to have just one integrated package.

That sounds reasonable.

> It's very obvious what something called 'bzr-gtk' would do (at least to
> Linux users), 'Olive' somewhat less so though it is a nice name.
> Jelmer's suggestion could work well.

I share Jelmer's view regarding this question. I think the name Olive
shouldn't disappear (maybe once we'll have a QT frontend, too).

> At a technical level I would recommend against having a package called
> 'gtk.*', as it seems like it would clash with Python's standard gtk
> library.  Could it be bzrgtk instead?

Currently the bzr-gtk code is under bzrlib.plugins.gtk.*, and Olive is
under olive.* (e.g. olive.frontend.gtk.*). So what you suggest is to
merge bzr-gtk's and Olive'd GTK code into bzrgtk.*?

> On my quick look through there does seem to be a lot of duplication of
> functions that are already provided inside bzrlib.  I'm sure Szilveszter
> added these for a reason, but I'd like very much if we could avoid
> having two implementations of operations like 'pull', etc.

This was already discussed both on the mailing list and on IRC. The
reason for having this wrapper was the lack of documented APIs (that
was 3 months ago, now I'm happy to see that the situation has
changed). Another reason was the thing you mentioned just below.

> The goal has always been that bzrlib should have an api that suits GUIs.
> At the moment many high-level operations like pull are only available in
> a function specific to the command line, but that can  be fixed, so that
> there is one layer which does strictly the command-line interpretation,
> and another which does the high-level operation.

I agree. This should be fixed (still many cmd_* have a huge code).

> So, Szilveszter, can you say something about how you came to have this,
> or what you would need in bzrlib to avoid having it in backend?

I'd be very happy if the problem above could be solved (cmd_* classes
shouldn't be more than 10-20 lines IMHO).

Cheers,
Szilveszter