[Top][All Lists]

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

Re: coreutils: FIXMEs in 2013

From: Pádraig Brady
Subject: Re: coreutils: FIXMEs in 2013
Date: Fri, 29 Nov 2013 10:10:50 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2

On 11/29/2013 09:16 AM, Bernhard Voelker wrote:
> Hi Padraig,
> shouldn't we include these into the upcoming release?
>   $ git grep -n 'FIXME.*2013' | cat -n
>      1        src/df.c:246:  MEGABYTES_OPTION  /* FIXME: remove long opt in 
> Aug 2013 */

This has been undocumented for 12 years,
warned about for a year,
and unused in debian at least:*megabytes

Nuke it please.

>      2        src/extent-scan.c:32:   FIXME: remove in 2013, or whenever 
> we're pretty confident

The kernel never got this fix unfortunately.
Therefore the FIXME comment should be removed.

>      3        src/realpath.c:54:  {"strip", no_argument, NULL, 's' /* FIXME: 
> deprecate in 2013 or so */},

Actually debian still removes realpath(1) from coreutils
in preference for the separate realpath package.
I must log a bug for that.
So we can't remove yet.
In any case --strip is another way to think about the operation,
so I'd be inclined to leave it in indefinitely.
So please just remove the FIXME comment.

> I'm not sure about #2, but I'd go for #1 and #3.
> And there's an even older one regarding
> "info coreutils 'General output formatting'":
>   doc/coreutils.texi:7651:@c FIXME: remove in 2009, if Apple Terminal has 
> been fixed for long enough.
> I've no idea about Apple Terminals. Do you know?

No idea. Test case is here if anyone can try:


reply via email to

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