[Top][All Lists]
[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: |
Pádraig Brady |
Subject: |
Re: [PATCH] core-count: A new program to count the number of cpu cores |
Date: |
Thu, 05 Nov 2009 12:08:27 +0000 |
User-agent: |
Thunderbird 2.0.0.6 (X11/20071008) |
Giuseppe Scrivano wrote:
> Hi Pádraig,
>
> Pádraig Brady <address@hidden> writes:
>
>> 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.
cool
> 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?
Well --available and --all are mutually exclusive and related.
That fact is obvious if they're parameters to a single option.
But I do take your point that --count is a bit redundant,
and I don't see nproc getting many other options, so OK
leave them as separate options.
I'll hope to commit this soon.
cheers,
Pádraig.
- Re: [PATCH] core-count: A new program to count the number of cpu cores, (continued)
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Bruno Haible, 2009/11/01
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Pádraig Brady, 2009/11/01
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Bruno Haible, 2009/11/04
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Giuseppe Scrivano, 2009/11/04
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Paolo Bonzini, 2009/11/04
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Pádraig Brady, 2009/11/04
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Giuseppe Scrivano, 2009/11/04
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Pádraig Brady, 2009/11/05
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Giuseppe Scrivano, 2009/11/05
- Re: [PATCH] core-count: A new program to count the number of cpu cores,
Pádraig Brady <=
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Giuseppe Scrivano, 2009/11/05
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Pádraig Brady, 2009/11/06
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Jim Meyering, 2009/11/06
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Erik Auerswald, 2009/11/06
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Giuseppe Scrivano, 2009/11/05
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Bruno Haible, 2009/11/04
- Re: [PATCH] core-count: A new program to count the number of cpu cores, Paolo Bonzini, 2009/11/05
Re: [PATCH] core-count: A new program to count the number of cpu cores, Pádraig Brady, 2009/11/03
Re: [PATCH] core-count: A new program to count the number of cpu cores, Pádraig Brady, 2009/11/03