Changeset d142ec5

Timestamp:

May 18, 2018, 3:59:52 PM (6 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, aaron-thesis, arm-eh, ast-experimental, cleanup-dtors, deferred_resn, demangler, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, pthread-emulation, qualifiedEnum, with_gc

Children:

2472a19, ccdab45

Parents:

c7d8100c

Message:

Fixed handling of jobs parameter in test.py

Location:

src/tests

Files:

• pybin/tools.py (modified) (2 diffs)
• test.py (modified) (1 diff)

src/tests/pybin/tools.py

@@ -168 +168 @@
 def jobCount( options, tests ):
         # check if the user already passed in a number of jobs for multi-threading
-        make_flags = os.environ.get('MAKEFLAGS')
-        make_jobs_fds = re.search("--jobserver-(auth|fds)=\s*([0-9]+),([0-9]+)", make_flags) if make_flags else None
-        if make_jobs_fds :
-                tokens = os.read(int(make_jobs_fds.group(2)), 1024)
-                options.jobs = len(tokens)
-                os.write(int(make_jobs_fds.group(3)), tokens)
+        if not options.jobs:
+                make_flags = os.environ.get('MAKEFLAGS')
+                force = bool(make_flags)
+                make_jobs_fds = re.search("--jobserver-(auth|fds)=\s*([0-9]+),([0-9]+)", make_flags) if make_flags else None
+                if make_jobs_fds :
+                        tokens = os.read(int(make_jobs_fds.group(2)), 1024)
+                        options.jobs = len(tokens)
+                        os.write(int(make_jobs_fds.group(3)), tokens)
+                else :
+                        options.jobs = multiprocessing.cpu_count()
         else :
-                options.jobs = multiprocessing.cpu_count()
+                force = True
 
         # make sure we have a valid number of jobs that corresponds to user input
@@ -182 +186 @@
                 sys.exit(1)
 
-        return min( options.jobs, len(tests) ), True if make_flags else False
+        return min( options.jobs, len(tests) ), force
 
 # setup a proper processor pool with correct signal handling

src/tests/test.py

@@ -96 +96 @@
         parser.add_argument('--all', help='Run all test available', action='store_true')
         parser.add_argument('--regenerate-expected', help='Regenerate the .expect by running the specified tets, can be used with --all option', action='store_true')
-        parser.add_argument('-j', '--jobs', help='Number of tests to run simultaneously', type=int, default='8')
+        parser.add_argument('-j', '--jobs', help='Number of tests to run simultaneously', type=int)
         parser.add_argument('--list-comp', help='List all valide arguments', action='store_true')
         parser.add_argument('-I','--include', help='Directory of test to include, can be used multiple time, All  if omitted', action='append')