2 nit picks, other than that, assuming this is tested it looks fine.
The headers you're putting on the patch are described at http://dep.debian.net/deps/dep3/
so:
a.) add 'Origin: upstream'
you can probably remove 'Author:' here (i realize we're probably not consistent in other patches on this).
b.) the bug number you referenced is a private bug (1374600), while there is a public bug (bug 1378441). Seems to make sense to reference the public bug.
c.) reference the ubuntu bug in the changelog message.
2 nit picks, other than that, assuming this is tested it looks fine. dep.debian. net/deps/ dep3/
The headers you're putting on the patch are described at http://
so:
a.) add 'Origin: upstream'
you can probably remove 'Author:' here (i realize we're probably not consistent in other patches on this).
b.) the bug number you referenced is a private bug (1374600), while there is a public bug (bug 1378441). Seems to make sense to reference the public bug.
c.) reference the ubuntu bug in the changelog message.
so make those changes, and i'll upload for you.