bug-bash
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Missing options in ulimit usage string


From: Jessica T McKellar
Subject: Missing options in ulimit usage string
Date: Sun, 14 Aug 2011 18:52:11 -0400 (EDT)
User-agent: Alpine 1.10 (DEB 962 2008-03-14)

Configuration Information [Automatically generated, do not change]:
Machine: x86_64
OS: linux-gnu
Compiler: gcc
Compilation CFLAGS: -DPROGRAM='bash' -DCONF_HOSTTYPE='x86_64' -DCONF_OSTYPE='linux-gnu' -DCONF_MACHTYPE='x86_64-unknown-linux-gnu' -DCONF_VENDOR='unknown' -DLOCALEDIR='/usr/local/share/locale' -DPACKAGE='bas$ uname output: Linux kid-charlemagne 2.6.38-10-generic #46-Ubuntu SMP Tue Jun 28 15:07:17 UTC 2011 x86_64 x86_64 x86_64 GNU/Linux
Machine Type: x86_64-unknown-linux-gnu

Bash Version: 4.2
Patch Level: 0
Release Status: release

Description:

`help ulimit` is missing the -b option in the usage string and is missing the -T option in both the usage string and the Options section of the help description. doc/FAQ in the bash 4.2 source tarball says both options were added in bash 4.0.

Repeat-By:

Run `help ulimit` and note that -b is listed in the Options section but not the usage string. Then look at doc/FAQ or the ulimit source and note that -T also exists but isn't documented.

Fix:

Attached is a patch to add usage documentation for -b and -T. I wasn't sure what to do about internationalization for the new text, so I left that alone for now.

Having the usage string and Options match is an improvement, but it's a little confusing that you get an "invalid option" message when you use an option that isn't available on your platform, e.g.:

$ ulimit -b
bash: ulimit: -b: invalid option
ulimit: usage: ulimit [-SHabcdefilmnpqrsTtuvx] [limit]

To minimally address this I added a note in the help documentation that not all options are available on all platforms, but a more precise error message when using an option that doesn't exist on your platform would be better.

Best regards,
-Jessica

Attachment: ulimit-opts.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]