[MERGE][BUG #177809] bzr SOURCE DESTINATION tweaks

[Spencer Chastain]

Sorry for taking so long to get back to this.  Long story short: been made
team lead, been swamped in work, been scrubbing our processes and tools,
think I'll be using bzr at work beginning next week.

The gist of what I'm seeing is: take what I've done, move the functionality
out of the CLI code, and move it into the working tree code.  Is that
correct?

Sounds cool.  I don't know if I'll have the time to finish the changes
though.  I've been pretty swamped with my new responsibilities, and
home-computer time is at a premium these days.

I'll try to get to this if no one else is going to pick it up, but if
someone wants to finish it out, my feelings won't be hurt.  Just reply so
I'll know :)

Or if you just want to take my patch as-is, that wouldn't hurt my feelings
either ;)

Thanks,

--Spencer

On Thu, Apr 3, 2008 at 1:54 AM, Alexander Belchenko <bialix at ukr.net> wrote:

> Ian Clatworthy пишет:
>
>  Ian Clatworthy has voted tweak.
> > Status is now: Semi-approved
> > Comment:
> > Apologies for taking way too long to review this.
> >
> > I've explained in comments on the bug why I think this is actually ok,
> > despite originally agreeing with Alexander's view. The only bit I don't like
> > is that duplicate files are silently ignored by design, e.g.
> >
> >  bzr mv a a b/
> >
> > just works without a warning. It turns out that's consistent with add
> > and delete, so fair enough. I would like the help (docstring) expanded
> > though to mention this behaviour.
> >
> > For details, see:
> > http://bundlebuggy.aaronbentley.com/request/%3C64ab70ac0802041811u61752e66l3a04f025336390ae%40mail.gmail.com%3E
> >
>
> bb:abstain because this patch solve problem only at CLI level, but I think
> it's better to solve it at WorkingTree API level. Without solving it at WT
> API level each GUI application will duplicate this logic inside their own
> codebase.
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/bazaar/attachments/20080411/99f21843/attachment.htm