Code review comment for lp:~abentley/launchpad/ignore-md-diff

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

= Summary =
This patch stops launchpad from storing the diffs in merge directives,
fixing bug #453010.

== Proposed fix ==
bzr has a bug in its patch parsing, but we are only exposed to it when
we process diffs that we don't get directly from bzrlib. When we
receive diffs via email, there's a chance that the mail client will
munge the newlines, which exposes us to this bug.

We are no longer using this type of diff at all, so there's no need to
be retrieving it from the merge directive.

So this patch simply stops attempting to store the diff from the merge
directive. For the time being, this means that it will be generated by
the MergeProposalCreationJob, but eventually, we can shut that off too.

== Pre-implementation notes ==
This patch has no pre-implementation call.

== Implementation details ==
Nothing to add

== Tests ==
bin/test -v test_codehandler

== Demo and Q/A ==
Generate a merge directive. Convert its line endings to CRLF. Send it
as an attachment to <email address hidden>. Launchpad should not
report an OOPS (via email).

= Launchpad lint =

Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.

Linting changed files:
  lib/lp/code/mail/codehandler.py
  lib/lp/code/mail/tests/test_codehandler.py
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkrYqkkACgkQ0F+nu1YWqI360gCdGSh7GtIj/OjKgm1f76PkOZkk
piYAn1pfPDnOPUgeA7HJ4izjFyD+9pJ/
=xtLB
-----END PGP SIGNATURE-----