coreutils
[Top][All Lists]
Advanced

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

Re: [PATCH] Support for --size in du


From: Bernhard Voelker
Subject: Re: [PATCH] Support for --size in du
Date: Mon, 21 Jan 2013 01:46:55 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130105 Thunderbird/17.0.2

On 01/20/2013 11:55 AM, Pádraig Brady wrote:
> On 01/19/2013 10:47 PM, Bernhard Voelker wrote:
>> [...]But increasing compatibility to BSD is tempting.

Just FYI: it seems that only FreeBSD's du supports
the -t option (but not the long --threshold) while
OpenBSD does not:

FreeBSD:
  http://www.freebsd.org/cgi/man.cgi?query=du
  https://github.com/freebsd/freebsd/blob/master/usr.bin/du/du.c
OpenBSD:
  http://www.openbsd.org/cgi-bin/man.cgi?query=du
  http://www.openbsd.org/cgi-bin/cvsweb/src/usr.bin/du/du.c

>> If I'd be asked to decide between --size and --threshold then
>> the latter is the winner: a) BSD compat, b) the abbreviation
>> of the option better fits into the already existing ones,
>> see --size vs. --si, while --th is (currently) fine.
>>
>> I'll change the patch towards --threshold then. WDYT?
> 
> +1

Thanks. Here it is.

I did not alias --threshold to a short -t option ... as I
remembered that we should be reluctant to add short options
which are not officially standardised.

Have a nice day,
Berny

Attachment: du-threshold.patch
Description: Text Data


reply via email to

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