coreutils
[Top][All Lists]
Advanced

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

Re: [PATCH] csplit doesn't honor the -m option as documented


From: Pádraig Brady
Subject: Re: [PATCH] csplit doesn't honor the -m option as documented
Date: Tue, 6 Oct 2015 11:25:17 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0

On 06/10/15 08:08, Ondrej Oprala wrote:
> Hi,
> just as subj. says, the option is documented and even shown in the usage 
> part of an info manual
> but not accepted.
> 
> This patch fixes that.

I wrote previously when reviewing --suppress-matched that:

  "I've removed the -m short option since we try to avoid them for new stuff.
   Also it gives us the flexibility in future to add a param to 
--suppress-matched
   to suppress X lines before/around/after the matched line, which could also 
be useful."

So getopt() didn't actually accept the -m option, nor was it documented in info 
docs.
So I've removed the extraneous mention in --help and man page with:

http://git.sv.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=ce50178

thanks,
Pádraig.



reply via email to

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