On 4.3.2016 14:13, Tomas Babej wrote:
> On 03/01/2016 03:46 PM, Petr Spacek wrote:
>> Hello,
>>
>> Pylint: enable parallelism
>>
>> The config file specifies 8 cores but Pylint very quickly
>> ends up with 3 cores so do not worry about overwhelming your system.
> 
> I like the idea of the patch, however, on my single-CPU VM this causes
> additional overhead and lint ends up taking more time (+15%).
> 
>>From the pylint docs [1]:
> 
>  If the provided number is 0 then the number of CPUs will be used.
> 
> I'd suggest we use this value instead, to get the best of both worlds :)

Here you go.

Petr^2 Spacek


> Tomas
> 
> [1] https://docs.pylint.org/run.html
From 700579ca97956e85c7501a96003627d301f5ebb8 Mon Sep 17 00:00:00 2001
From: Petr Spacek <pspa...@redhat.com>
Date: Tue, 1 Mar 2016 15:42:48 +0100
Subject: [PATCH] Pylint: enable parallelism

The config file specifies 8 cores but Pylint very quickly
ends up with 3 cores so do not worry about overwhelming your system.
---
 pylintrc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pylintrc b/pylintrc
index 1003e49217ec9761ea5e17d3e965376a0cca1a32..2a4dd7b09ef8b7ab052fc12e382c88a7af2ed568 100644
--- a/pylintrc
+++ b/pylintrc
@@ -7,7 +7,7 @@ persistent=no
 load-plugins=pylint_plugins
 
 # Use multiple processes to speed up Pylint.
-jobs=1
+jobs=0
 
 [MESSAGES CONTROL]
 
-- 
2.5.0

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to