I don’t know about you, but I like diff -p . Having used it for years, I can read these diffs like a text, while diffs without -p often need to have the original file opened side by side, just to get enough of the context.
Loving diff -p so much, I want to see it everywhere (evil laughter). Alas, in bzr only diff command can easily use -p, just run it as
bzr diff --diff-options=-p or store it as an alias in the
Actually, for an alias there is a better, although more verbose, alternative:
diff = "diff --diff-options='-F ^[[:alpha:]$_].*[^:]$'"
Unlike simple -p it will not think that a word ending with a semicolon (like a label or, say,
private:) is a “C function name”.
But the problem is — only
bzr diff can be tuned this way. Bzr email plugin still sends diffs without function names. And
bzr gdiff does not show them. And, of course, all other bzr commands —
bzr commit, for example, or
bzr unshelve --preview,
bzr log --show-diff and others — they are still as unfriendly as before.
I was solving it on a case by case basis — added a
post_commit_diffoptions configuration option to the bzr-email plugin, then a command line option to
bzr gdiff. But then it occurred to me that I can attack the problem at its core!
And here it is: get this piece of the python code , save it as the
~/.bazaar/plugins/diffoptions/__init__.py file. Then you can add, say,
diff_options = -u -F ^[[:alpha:]$_].*[^:]$
bazaar.conf file and it will magically change everything. All bzr commands, bzr plugins — everything that generate diffs will use these options, and you can have nice C function names on all diff chunks, no matter what plugin or core bzr command generates them.
I wonder, whether bzr folks would like to get that as a core feature. They can do it properly, without hacks that I had to resort to.
1) If you haven’t heard about it before, -p option adds C function name to every diff chunk:
=== modified file 'sql-common/my_time.c'
--- sql-common/my_time.c 2010-08-27 14:12:44 +0000
+++ sql-common/my_time.c 2011-02-28 21:24:19 +0000
@@ -727,6 +727,9 @@ <strong>void my_init_time(void)</strong>
my_time.hour= (uint) l_time->tm_hour;
my_time.minute= (uint) l_time->tm_min;
my_time.second= (uint) l_time->tm_sec;
+ my_time.neg= 0;
+ my_time.second_part= 0;
+ my_time.time_type= MYSQL_TIMESTAMP_DATETIME;
my_system_gmt_sec(&my_time, &my_time_zone, &not_used);
2) This is the complete text of the bzr diffoptions plugin. It works for me with bzr 2.2.4, but I didn’t try it with any other bzr version.
"""global bzr diff options"""
version_info = (0, 0, 1)
from bzrlib import branch
from bzrlib import diff
from bzrlib import errors
root = tree.id2path(tree.get_root_id())
br = branch.Branch.open_containing(root)
diff_options = br.get_config().get_user_option('diff_options')
if diff_options is not None:
opts = diff_options.split()
def diff_file(olab, olines, nlab, nlines, to_file, path_encoding=None):
diff.external_diff(olab, olines, nlab, nlines, to_file, opts)
except (errors.NoSuchId, NotImplementedError):
old_init = diff.DiffText.__init__
def new_init(self, old_tree, new_tree, to_file, path_encoding='utf-8',
old_label='', new_label='', differ=diff.internal_diff):
if differ == diff.internal_diff:
differ = get_differ(old_tree) or get_differ(new_tree) or diff.internal_diff
old_init(self, old_tree, new_tree, to_file, path_encoding,
old_label, new_label, differ)
diff.DiffText.__init__ = new_init