scripts dont accept gnu long options ( eg --help)

Patch forwarded to debian unstable: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=457251

This debdiff fixes the bug.

From the changelog:
  * Adding the --help option to all scripts; shows usage. (LP: #177051)
    - Added dpatch to debian/rules and Build-Depends
    - Added debian/patches/01-help-option.dpatch

Forgot to add "XSBC-Original-Maintainer". This is done now.

- 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!

Good idea to take both --help and -h arguments. I can see that one of the scripts (with all arguments optional) shows usage when using -h, so this is indeed appropriate.

Thanks for the feedback. Here is a new debdiff.

From the changelog:
  * Adding --help and -h to all scripts; shows usage. (LP: #177051)
    - Enabled simple-patchsys in debian/rules
    - Added debian/patches/help-options.patch
  * Modify Maintainer value to match the DebianMaintainerField
    specification.
  * Standards-Version changed to 3.7.3
    - Added Homepage field to debian/control

Uploaded. Thanks!

This bug was fixed in the package ldapscripts - 1.7.1-2ubuntu1

---------------
ldapscripts (1.7.1-2ubuntu1) hardy; urgency=low

  * Adding --help and -h to all scripts; shows usage. (LP: #177051)
    - Enabled simple-patchsys in debian/rules
    - Added debian/patches/help-options.patch
  * Modify Maintainer value to match the DebianMaintainerField
    specification.
  * Standards-Version changed to 3.7.3
    - Added Homepage field to debian/control

 -- Andreas Wenning <email address hidden> Sat, 23 Feb 2008 15:09:03 +0100