[merge] update to external diff fixes

John Arbash Meinel john at arbash-meinel.com
Tue Aug 15 18:53:57 BST 2006


The attached patch is to close bug 56307.
https://launchpad.net/products/bzr/+bug/56307

Basically, now that we explicitly test that we can spawn diff and handle
binary files, we have to also support when LANG is not english.
So now we spawn diff with LANG=C set, and if it returns a bad error code
(retcode=2), we run it again with native LANG and report the error.

It includes a test that when using an external diff we report what diff
would naturally report. And I had Martin Pitt run it on his German
language Edgy machine, and it worked. (Dapper doesn't seem to have i18n
for diff)

John
=:->
-------------- next part --------------
A non-text attachment was scrubbed...
Name: external-diff-56307.patch
Type: text/x-patch
Size: 16782 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060815/bfdd9b19/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 254 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060815/bfdd9b19/attachment.pgp 


More information about the bazaar mailing list