On Sun, Jan 20, 2013 at 02:35:27PM +0100, Benno Schulenberg wrote:
>
> Hello Sami,
>
> On Sun, Jan 20, 2013, at 12:25, Sami Kerola wrote:
> > + fprintf(stderr, _("%s from %s\n"),
> > + program_invocation_short_name, PACKAGE_STRING);
> >
> > should use UL version printing macro.
> >
> > + fprintf(stderr, UTIL_LINUX_VERSION);
>
> Ah. Okay. Then it is good to leave that out for now. I will provide a
> new patch for that later.
I have fixed this issue.
> > In the patch 0007 the renice(1) adds -V short option along side with
> > -v to print version. IMHO if the upper case option is added the lower
> > case equivalent should be dropped.
>
> I agree -v should be dropped for --version, but I didn't want to do that
> right away. Maybe one or two releases down the line?
Well, we have to care about backward compatibility. It seems better
to support (now undocumented) -v as well as new and documented -V.
Karel
--
Karel Zak <kzak@xxxxxxxxxx>
http://karelzak.blogspot.com
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
