Comment 4 for bug 177051

- Please add an entry to the changelog mentioning that you changed the Maintainer field to match the DebianMaintainerField specification. Also note that you can use the script "update-maintainer" (from package "ubuntu-dev-tools") for this rutinary task (just execute it in the source directory and it will change tha Maintainer field and note it in the changelog).

- It is best practice when patching a package from Debian with no patch system to not add one (to minimize the diff to Debian; packages using cdbs are an exception to this) or use that one you think it's Maintainer would prefer. In this case it is evident that the authors prefers simple-patchsys (not only because it's cdbs' default system but because there is a commented out line for this in debian/rules). It isn't really important, but it wouldn't be bad to use simple-patchsys instead of dpatch because of this (and to avoid the additional build dependency).

- You might consider also taking "-h" as an option to show the help. Do you think it would be apropiate, too?

Please address this issues and subscribe ubuntu-universe-sponsors again once you have a new debdiff ready (well, or don't do so; I have subscribed to this bug and will upload your next debdiff ;)).

Thanks for contributing to Ubuntu!