On Sat, Dec 31, 2022 at 10:54:27AM -0700, Todd C. Miller wrote:
> In POSIX mode, df(1) does not honor the BLOCKSIZE environment
> variable.
> 
> Any comments on the wording?
> 
>  - todd
> 

hi

> Index: bin/df/df.1
> ===================================================================
> RCS file: /cvs/src/bin/df/df.1,v
> retrieving revision 1.48
> diff -u -p -u -r1.48 df.1
> --- bin/df/df.1       10 Aug 2016 19:46:43 -0000      1.48
> +++ bin/df/df.1       31 Dec 2022 17:50:39 -0000
> @@ -100,6 +100,9 @@ with the possibly stale statistics that 
>  .It Fl P
>  Print out information in a stricter format designed to be parsed
>  by portable scripts.
> +The
> +.Ev BLOCKSIZE
> +environment variable is ignored when this option is specified.

you could shorten it to "BLOCKSIZE is ignored...". up to you.

>  .It Fl t Ar type
>  Indicate the actions should only be taken on
>  file systems of the specified
> @@ -127,11 +130,12 @@ the last option given overrides the othe
>  .It Ev BLOCKSIZE
>  If the environment variable
>  .Ev BLOCKSIZE
> -is set, and the
> -.Fl h
> +is set, and neither of the

theoretically, "neither" refers to two items. so maybe "none of"?

> +.Fl h ,
> +.Fl k ,
>  or
> -.Fl k
> -options are not specified, the block counts will be displayed in units of 
> that
> +.Fl P
> +options are specified, the block counts will be displayed in units of that
>  size block.
>  .El
>  .Sh EXIT STATUS
> 

again we could shorten it. the BLOCKSIZE text already looks a bit weird:
the first clause is just redundant, and the emphasis seems to be how to
avoid it. maybe:

        Display block counts in units of size
        .Dv BLOCKSIZE .
        Ignored if any of the
        .Fl h , k
        or
        .Fl P
        options are specified.

sth like that?

jmc

Reply via email to