Alexander Belchenko wrote:
> I'm not sure is this bug should be marked as High or Critical. For me
> it's critical enough and for 0.92 it should be fixed.
>
I think it is a Critical regression.
The reason 'diff' is blowing up is because 'internal_diff' dies when you feed
it 2 texts which have no differences. Which should also be considered a bug.
It doesn't effect any actual committed changes, etc.
Oh, and we've already handled the O_BINARY issues in the Transport code. You
can use 'osutils.O_BINARY' which is set up to handle when os.O_BINARY is not
available.
I'll write up a patch.
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (Darwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Alexander Belchenko wrote:
> I'm not sure is this bug should be marked as High or Critical. For me
> it's critical enough and for 0.92 it should be fixed.
>
I think it is a Critical regression.
The reason 'diff' is blowing up is because 'internal_diff' dies when you feed
it 2 texts which have no differences. Which should also be considered a bug.
It doesn't effect any actual committed changes, etc.
Oh, and we've already handled the O_BINARY issues in the Transport code. You
can use 'osutils.O_BINARY' which is set up to handle when os.O_BINARY is not
available.
I'll write up a patch.
John
=:->
-----BEGIN PGP SIGNATURE----- enigmail. mozdev. org
BCYSNAAMRAjXXAJ dZPngKfOZwuDlrD VdmkiahW/ ++AJ4xC4n7 V0YK95Q= =
Version: GnuPG v1.4.7 (Darwin)
Comment: Using GnuPG with Mozilla - http://
iD4DBQFHHMOSJde
rO670Fi5LL3CvEq
=16ci
-----END PGP SIGNATURE-----