Ubuntu
efax-gtk package

  1. Bug #108746
  2. Comment #10

Comment 10 for bug 108746

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

Looks very good but still few minor issues:

1. Please add to the changelog the addition of dpatch in debian/control and debian/rules
2. The Menu section is "Transfer" not "Transfert"
3. It is always a good idea to add a description of the patch in the dpatch patch. The template is in the lines:

## All lines beginning with `## DP:' are a description of the patch.
## DP: No description.

4. Note also that you are using the (incorrect) email <chris@chabarrou> in the patch while it is <email address hidden> in the changelog (I think this is simply because you have not set the DEBEMAIL env variable).

When you have a new patch available, please unassign yourself, set it as confirmed and resubscribe u-u-s.