Comment 7 for bug 666494

Revision history for this message
Stefano Rivera (stefanor) wrote :

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.