Damien Laniel <[EMAIL PROTECTED]> writes:

> As your decision has been to not support -i and -p flags, please remove them
> from the man/infos pages

In coreutils 5.2.1-2, the -i and -p flags are not listed in the
uname man page, which is mostly generated from the --help output.
Removing them from Info may be too much work, because of section
4 of the GFDL.  (/usr/share/doc/coreutils/copyright mentions
"removal of the doc/ subdirectory"; what's that?)

However, I think something should still be changed.  Currently,
uname -p or -i does not give the same error message as other
unsupported options:

[EMAIL PROTECTED]:~$ uname --processor
uname: unrecognized option `--processor'
Try `uname --help' for more information.
[EMAIL PROTECTED]:~$ uname -Z
uname: invalid option -- Z
Try `uname --help' for more information.
[EMAIL PROTECTED]:~$ uname -p
Try `uname --help' for more information.

This happens because "p" and "i" are still present in the string
of supported options that uname.c gives to getopt_long.  The
attached patch fixes that.

coreutils (5.2.1-2.kon.1) local; urgency=low

  * Local version!
  * src/uname.c (main): Don't recognize -i and -p at all.  (See: #193170)

 -- Kalle Olavi Niemitalo <[EMAIL PROTECTED]>  Sat, 30 Apr 2005 12:38:13 +0300

--- coreutils_5.2.1-2/debian/patches/21_uname-no-unknowns.diff	2005-04-30 12:21:04.000000000 +0300
+++ coreutils_5.2.1-2.kon.1/debian/patches/21_uname-no-unknowns.diff	2005-04-30 12:24:40.000000000 +0300
@@ -18,6 +19,15 @@
    -o, --operating-system   print the operating system\n\
  "), stdout);
        fputs (HELP_OPTION_DESCRIPTION, stdout);
+@@ -158,7 +154,7 @@
+ 
+   atexit (close_stdout);
+ 
+-  while ((c = getopt_long (argc, argv, "asnrvmpio", long_options, NULL)) != -1)
++  while ((c = getopt_long (argc, argv, "asnrvmo", long_options, NULL)) != -1)
+     {
+       switch (c)
+ 	{
 @@ -189,14 +185,6 @@
  	  toprint |= PRINT_MACHINE;
  	  break;

Attachment: pgp5ctN49SUev.pgp
Description: PGP signature

Reply via email to