[PATCH] script: allow to pass any float value for CT_LOAD

[Samuel Martin]

Hi Yann,

2013/1/27 Yann E. MORIN <yann.morin.1998@free.fr>:
> Samuel, All,
>
> On Sunday 27 January 2013 Samuel Martin wrote:
>> # HG changeset patch
>> # User Samuel Martin <s.martin49@gmail.com>
>> # Date 1359292548 -3600
>> # Node ID d87c8d118f4b68b5ab297e956e5ae32149647bf8
>> # Parent  67df47c6ccd0c65a92c1d84813ad64f5d96dfd2b
>> script: allow to pass any float value for CT_LOAD
>>
>>
>> Signed-off-by: Samuel Martin <s.martin49@gmail.com>
>>
>> diff -r 67df47c6ccd0 -r d87c8d118f4b config/global/build-behave.in
>> --- a/config/global/build-behave.in   Mon Jan 21 23:03:18 2013 +0100
>> +++ b/config/global/build-behave.in   Sun Jan 27 14:15:48 2013 +0100
>> @@ -19,19 +19,15 @@
>>        Enter 0 to set automatically based on how many processors the host has.
>>
>>  config LOAD
>> -    int
>> +    string
>>      prompt "Maximum allowed load" if ! BACKEND
>> -    default 0
>> +    default "0"
>>      help
>>        Specifies that no new jobs should be started if there are others jobs
>>        running and the load average is at least this value.
>> -
>> -      Makes sense on SMP machines only.
>> -
>> -      Enter 0 to have no limit on the load average.
>> -
>> -      Note: only the integer part of the load is allowed here (you can't enter
>> -            0.75 for example).
>> +
>> +      Note: any float value is allowed and will be passed to the '-l' option
>> +            of make (for further details, refer to the make manpage).
>
> There's now no reference to how to not have limit on the load, since you
> removed the "Enter 0 to have no limit" line.
>
> What about making the default empty, and adding this to the help:
>     Leave to empty for no load limit.
>
> and...
>
>>  config USE_PIPES
>>      bool
>> diff -r 67df47c6ccd0 -r d87c8d118f4b scripts/crosstool-NG.sh.in
>> --- a/scripts/crosstool-NG.sh.in      Mon Jan 21 23:03:18 2013 +0100
>> +++ b/scripts/crosstool-NG.sh.in      Sun Jan 27 14:15:48 2013 +0100
>> @@ -516,7 +516,7 @@
>>      AUTO_JOBS=$((`getconf _NPROCESSORS_ONLN 2> /dev/null || echo 0` + 1))
>>      [ ${CT_PARALLEL_JOBS} -eq 0 ] && JOBSFLAGS="${JOBSFLAGS} -j${AUTO_JOBS}"
>>      [ ${CT_PARALLEL_JOBS} -gt 0 ] && JOBSFLAGS="${JOBSFLAGS} -j${CT_PARALLEL_JOBS}"
>> -    [ ${CT_LOAD} -ne 0 ] && JOBSFLAGS="${JOBSFLAGS} -l${CT_LOAD}"
>> +    [ ${CT_LOAD} != "0" ] && JOBSFLAGS="${JOBSFLAGS} -l${CT_LOAD}"
>
> ... change this to simply:
>     JOBSFLAGS="${JOBSFLAGS} -l${CT_LOAD}"
>
ok, I'll repost it.

Regards,

-- 
Samuel

--
For unsubscribe information see http://sourceware.org/lists.html#faq