[MERGE] add ghandle-patch

Daniel Schierbeck daniel.schierbeck at gmail.com
Mon Mar 3 15:43:05 GMT 2008


On Sat, 2008-02-23 at 02:43 -0500, Aaron Bentley wrote:
> Hi all,
> 
> This patch adds a new ghandle-patch command that shows a diff, and if
> the patch is a merge-directive, allows merging it.
> 
> It's a helper, meant to be launched from web browsers or email client,
> so it's general enough to handle plain patches as well as merge
> directives. It was inspired by bzr-merge-into-gui.

Hi Aaron, sorry for the long wait, I've been rather busy lately!

Thank you for this patch, it's a great addition to bzr-gtk. I've
reviewed it, and it seems to work great!

I have one issue with it, though -- it seems to make Compiz crash, right
after clicking the OK button on a successful merge. Don't really know
why, but we need to fix it before releasing. Nevertheless, I'll merge it
into mainline. If you could find the time to investigate the issue, that
would be really great.

Again, thank you for this contribution!


Cheers,
Daniel

bb:approve
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : https://lists.canonical.com/archives/bzr-gtk/attachments/20080303/36f37081/attachment.pgp 


More information about the bzr-gtk mailing list