Comment 43 for bug 250820

Revision history for this message
Colin Watson (cjwatson) wrote :

Thanks, this is largely fine, assuming that the .changes will be attached in the real thing. As Matt said, a blank line would help. Also:

  * We are used to reading changelog trailer lines as " -- name <e-mail> date" (note leading space, and two spaces between e-mail and date; it would be helpful to match that.
  * Using the Maintainer for the changelog trailer line is weird, because it doesn't correspond to what would be in the real changelog. I suggest using Changed-By for this instead (Bhavani Shankar in this case), and including a Maintainer: line for ubuntu-motu. Matt summarised the possibilities in comments 16 and 17 to this bug, which could be turned into test cases.
  * It would be better for the synthesised changelog trailer line to be just below the first changelog block, rather than right at the end. As it is, it looks as if ubuntu-motu (or Bhavani) was responsible for version 0.8.14-2, which isn't the case; that trailer line belongs with version 0.8.17-11ubuntu1.