[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.