- 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!
** Changed in: ldapscripts (Ubuntu)
Assignee: (unassigned) => awen (andreas-wenning)
Status: Confirmed => In Progress
--
scripts dont accept gnu long options ( eg --help)
https://bugs.launchpad.net/bugs/177051
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
--
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
