[merge] win32- all tests pass but 1
Aaron Bentley
aaron.bentley at utoronto.ca
Sat Jul 1 08:16:01 BST 2006
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
John Arbash Meinel wrote:
> Aaron Bentley wrote:
> It may not even be required for the caller to close() the file handle,
> as long as _KnitData doesn't hold it open.
Since finalizer invocation isn't deterministic, I think that's a race
condition.
> These are the 'KnownFailures'. I would be okay with creating a new
> exception class, like TestSkippe, maybe TestKnownFailed, or something
> like that.
> I return here only because a plain return in setUp doesn't do anything.
Fine, I won't insist on doing a TestSkipped. Clearly it means something
different to you than it means to me.
>>> This code looks like it could fail because it's joining a url and a
>>> local filesystem path. Am I wrong about that?
>
> It could fail if the working tree was a non ascii path.
> I could easily call local_path_to_url() instead, but I was thinking we
> were actually asserting that you can use the cwd() in a simple file://
> url. I'd be happy to switch to a real url if you prefer.
I don't think it's right to assert you can use the cwd in a simple file
URL, because it's easy to imagine situations where people are in
non-ascii urls, and don't have any control over that.
> I'd be okay either way with the TestSkipped stuff. I was just trying to
> avoid Skip when we really mean
> NotGoingToRunBecauseItDoesntWorkAndItIsOldAndUnsupported.
But it's actually a case of NotGoingToRunBecausePlatformSucks.
Okay, I can live with returning for now. So you have my +1 if you
switch the cwd thing to a real URL.
Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.2 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFEpiEx0F+nu1YWqI0RAmtMAJ4oXiZJDoJ9J0yvHzLi1Z46Lfd9BQCgiaZH
s7z7iuFQJyPJlK+DZR8/snQ=
=MPV5
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list