[PATCH] [Bug 209281] Re: Windows diff apps don't understand symlinks created by Cygwin bzr diff --using
Matt McClure
mlm at aya.yale.edu
Mon May 26 18:35:43 BST 2008
Ian,
On Thu, May 22, 2008 at 2:47 AM, Ian Clatworthy <
ian.clatworthy at canonical.com> wrote:
> IMO, this is almost ready to merge. It still needs a test and
> two tweaks:
>
> 1. osutils.host_os_dereferences_symlinks() is a public function so
> it needs a test added to tests/test_osutils.py. I'm ok if the
> test simply checks that the function is present, i.e. it can be
> called without throwing an exception.
>
> 2. In test_diff.py, result=proc.stdout.read() needs spaces either
> side of the = character.
>
> 3. It's fixing a bug so please add an entry into NEWS.
>
> As a final comment, sending diffs to the list is OK but sending
> merge directives is better. Here's one way to do that:
>
> bzr send -o ../59302.x.patch
> (switch to your mail tool and attach the file)
See the attached merge directive. I believe it addresses the items above.
Matt
-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/bazaar/attachments/20080526/0f8af1b7/attachment-0001.htm
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug.209281.8.patch
Type: application/octet-stream
Size: 23037 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20080526/0f8af1b7/attachment-0001.obj
More information about the bazaar
mailing list