[PATCH] [Bug 209281] Re: Windows diff apps don't understand symlinks created by Cygwin bzr diff --using

Matt McClure mlm at aya.yale.edu
Sun May 4 20:00:25 BST 2008


Is the attached patch better with respect to test_prepare_files?

I understand that you've been concerned that my use of newname2 was
superfluous.  However, it serves two purposes.  First, it addresses a
preexisting failure on Windows.  Second, it allows test_prepare_files to
continue succeeding on Cygwin after my changes to the code under test in
this thread.  Previously, the final call to _prepare_files succeeded on
Cygwin and other platforms with symlinks for the wrong reasons.

I hope this version of the patch makes it clearer why newname2 is
necessary.  Before the change, oldname2 and newname2 refer to nonexistent
files in the final call to _prepare_files in test_prepare_files.  I think
_prepare_files requires that its old_path and new_path arguments refer to a
single existing file, in the old and new trees respectively, at its file_id
argument.  This version of my change makes oldname2 and newname2 exist at


On Tue, Apr 29, 2008 at 10:47 AM, Matt McClure <mlm at aya.yale.edu> wrote:
> On Sun, Apr 27, 2008 at 4:10 PM, Aaron Bentley <aaron at aaronbentley.com> wrote:
>>  You haven't removed the tree/newname2.
> Without newname2, test_prepare_files fails on Windows (and on Cygwin
> with my patch to the code under test).  Maybe what I've done isn't the
> right fix, but the test is broken before my patch.  See
> https://bugs.launchpad.net/bzr/+bug/212289 .  Could you suggest a
> better fix?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug.209281.7.patch
Type: application/octet-stream
Size: 4568 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20080504/c9857a50/attachment.obj 

More information about the bazaar mailing list