Re: [PATCH 00/17] pull: almost complete setterm refactoring

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

 



On Tue, May 20, 2014 at 10:11:49AM +0200, Benno Schulenberg wrote:
> 
> Some day Karel Zak wrote:
> > >     _(" -hbcolor <bright> <color>\n")
> > >
> > > and describe below in usage():
> > >
> > >    _(" <color> means black, blue, ...")
> 
> But but but... the word "bright" is not a placeholder, it must not
> be replaced by something else, as <color> by blue, for example.
> It is a literal argument, so should not take <>, and it is optional,
> so it should take [].  No

 Yep, fixed. Thanks!

 BTW, setterm still need our love:
 
  * usage() contains only options, but no any description

  * it would be nice to have long options (--term rather than -term)
    in the man page

  * nowhere (in man page) is info that --long options are supported
    since v2.25 (and for backward compatibility is better to use
    "-foo" rather than "--foo")


    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




[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux