Comment 2 for bug 289268

Revision history for this message
Dan Watkins (oddbloke) wrote :

Hi pdf23ds,

Thanks for the patch! The normal way to submit changes to bzr is via the mailing list, in an attachment to an email with a subject starting with '[MERGE]' (e.g. "[MERGE][Bug #289263] Allow setting of default external diff program"). You can use a patch, as you have here, or you can use 'bzr send'.

One thing that is required of any code going into bzrlib is testing. For this change, I suggest you look in bzrlib/tests/test_config.py and bzrlib/tests/blackbox/test_diff.py to see what is done there.

Dan