bug-coreutils
[Top][All Lists]
Advanced

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

Re: [PATCH] core-count: A new program to count the number of cpu cores


From: Giuseppe Scrivano
Subject: Re: [PATCH] core-count: A new program to count the number of cpu cores
Date: Thu, 05 Nov 2009 12:22:49 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1.50 (gnu/linux)

Hi Pádraig,

Pádraig Brady <address@hidden> writes:

> Sorry about all the interface discussions, but interface
> to coreutils commands is very important given the size of the audience
> and the affect it has on all future shell scripts etc.

it is fine, except the interface and the program name there was not
anything to discuss ;-)


> How about removing the --current option as it's now redundant.
> A whole option to do nothing seems wrong. Also I think "available" is
> more understandable than "current". Perhaps the various (future) sets
> might be better as an option parameter. I.E how about:
>
>   nproc --count={all,available}
>
> with --count=available being the default operation.

Right, "available" looks better than "current", I'll change it.

I would avoid this interface, as it causes longer commands while using
two separate options can be shortened by getopt.  I don't think that
"--count" improves the readability (considering that nproc can't do
anything else) so much to sacrifice that possibility.  What do you
think?


Regards,
Giuseppe




reply via email to

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