Ubuntu
cryptsetup package

  1. Bug #164181
  2. Comment #12

Comment 12 for bug 164181

Revision history for this message
Cesare Tirabassi (norsetto) wrote :

@Bruno,

still just a couple of minor remarks:

1. Try to improve your changelog so that it is clear what does what/where/why. In this case it could be good to say:

* debian/patches:
  - add 07typos_fix.dpatch: Fix typos in man pages. (LP: #164181)

2. In DPATCH you should fill the header information. For instance, your email is <email address hidden> not bruno@bruno-desktop. Also, see where it says: ##DP: No description. ? We use that field to describe what the patch does and why, and eventually where is coming from. In this case is your patch, so just add a description (like ##DP: Fixes some typos in upstream's man pages).

If you have doubts, ask your mentor or on IRC (#ubuntu-motu).

Keep it up!