The test case and regression potential belong in the description of this bug, not the patch.
> * debian/patches/1-fix-data-corruption-bug.patch: LP #666494
It's still missing a colon after the LP, and it could use a description. I'd do this: * 1-fix-data-corruption-bug.patch: Backport typo bugfix patch. LP: #666494
> -- nutznboltz <email address hidden>
I'd prefer a real name.
> +## Description: data corruption when using GlusterFS with certain options set. > +## Origin/Author: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=639 > +## TEST CASE: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=639 > +## Regression potential: none known
I'd remove the second two lines of that, and remove "/Author", you are giving the origin not the author.
The test case and regression potential belong in the description of this bug, not the patch.
> * debian/ patches/ 1-fix-data- corruption- bug.patch: LP #666494
It's still missing a colon after the LP, and it could use a description. I'd do this: corruption- bug.patch: Backport typo bugfix patch. LP: #666494
* 1-fix-data-
> -- nutznboltz <email address hidden>
I'd prefer a real name.
> +## Description: data corruption when using GlusterFS with certain options set. bugs.gluster. com/cgi- bin/bugzilla3/ show_bug. cgi?id= 639 bugs.gluster. com/cgi- bin/bugzilla3/ show_bug. cgi?id= 639
> +## Origin/Author: http://
> +## TEST CASE: http://
> +## Regression potential: none known
I'd remove the second two lines of that, and remove "/Author", you are giving the origin not the author.