[MERGE] [0.90] Disable patch verification (broken for CRLF files)

John Arbash Meinel john at arbash-meinel.com
Mon Aug 13 21:55:07 BST 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Aaron Bentley wrote:
> Aaron Bentley wrote:
>> Aaron Bentley wrote:
> 
>>> We recently discovered that patch verification is broken for CRLF files
>>> (and probably CR files, too).  The fix appeared simple, but I've run
>>> into problems testing it, so I think the safest things it to disable it
>>> for now.  I'll get a fix in before 0.91.
>>> I think it is important to get some kind of fix into 0.90, because it
>>> will be an extremely visible bug for projects using non-LF source files
>>> and bundles/merge-directives.
> 
> Can someone please review this?
> 
> Aaron
> 
> http://bundlebuggy.aaronbentley.com/request/%3C46C05772.3060906%40utoronto.ca%3E

What does it actually cause when merging a "broken" bundle?

I can see that you return "inapplicable", but I don't know if that means
it aborts, or does something else. (I think it means that you are saying
it verification isn't really possible for this object.)

If I'm correct then:

BB:approve

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFGwMUrJdeBCYSNAAMRAq27AKC6GEcdDFA5I1JRzvNcZlAjPmPFswCg2hsC
CldImv1GqRtm4f0dFUHUvgU=
=N0WL
-----END PGP SIGNATURE-----



More information about the bazaar mailing list