Don't use make -j by default, log debug level by default

Author: Zoran Simic

src/portable_python/__init__.py

@@ -539,8 +539,7 @@ def xenv_PKG_CONFIG_PATH(self):
             yield f"{self.deps_lib}/pkgconfig"
 
     def _do_run(self, program, *args, fatal=True, env=None):
-        logger = self._log_handler
-        return runez.run(program, *args, passthrough=logger, stdout=None, stderr=None, fatal=fatal, env=env, logger=logger or runez.UNSET)
+        return runez.run(program, *args, passthrough=self._log_handler, stdout=None, stderr=None, fatal=fatal, env=env)
 
     def run_configure(self, program, *args, prefix=None):
         """
@@ -558,11 +557,13 @@ def run_configure(self, program, *args, prefix=None):
 
     def run_make(self, *args, program="make", cpu_count=None):
         cmd = program.split()
-        if cpu_count is None:
-            cpu_count = multiprocessing.cpu_count()
+        if cpu_count and cpu_count < 0:
+            available = multiprocessing.cpu_count()
+            if available and available > 0:
+                cpu_count += available
 
-        if cpu_count and cpu_count > 3:
-            cmd.append("-j%s" % (cpu_count - 2))
+        if cpu_count and cpu_count > 1:
+            cmd.append("-j%s" % cpu_count)
 
         self._do_run(*cmd, *args)
 

src/portable_python/cli.py

@@ -14,19 +14,21 @@
 @runez.click.group()
 @runez.click.version()
 @runez.click.color()
-@runez.click.debug("-v")
 @runez.click.dryrun("-n")
 @click.option("--config", "-c", metavar="PATH", default="portable-python.yml", show_default=True, help="Path to config file to use")
+@click.option("--quiet", "-q", is_flag=True, help="Turn off DEBUG logging")
 @click.option("--target", "-t", hidden=True, help="For internal use / testing")
-def main(debug, config, target):
+def main(config, quiet, target):
     """
     Build (optionally portable) python binaries
     """
+    level = logging.INFO if quiet else logging.DEBUG
     runez.system.AbortException = SystemExit
     runez.log.setup(
-        debug=debug,
+        debug=not quiet,
+        level=level,
         console_format="%(levelname)s %(message)s",
-        console_level=logging.INFO,
+        console_level=level,
         default_logger=LOG.info,
         locations=None,
     )

src/portable_python/cpython.py

@@ -204,7 +204,7 @@ def _do_linux_compile(self):
 
         self.run_make(*make_args)
         # Don't parallelize `make install`, see https://github.com/python/cpython/issues/109796
-        self.run_make("install", f"DESTDIR={self.destdir}", cpu_count=0)
+        self.run_make("install", f"DESTDIR={self.destdir}")
 
     def _finalize(self):
         is_shared = self.setup.prefix or self.has_configure_opt("--enable-shared", "yes")

src/portable_python/external/__init__.py

@@ -13,8 +13,8 @@ def version(self):
         return self.cfg_version("0.3.2")
 
     def _do_linux_compile(self):
-        self.run_make("LIBINTL=NOOP", cpu_count=0)
-        self.run_make("LIBINTL=NOOP", f"DESTDIR={self.deps}", "prefix=/", "install", cpu_count=0)
+        self.run_make("LIBINTL=NOOP")
+        self.run_make("LIBINTL=NOOP", f"DESTDIR={self.deps}", "prefix=/", "install")
 
 
 class Toolchain(ModuleBuilder):

src/portable_python/external/xcpython.py

@@ -244,8 +244,8 @@ def _do_linux_compile(self):
             "--disable-docs",
             "--enable-portable-binary",
         )
-        self.run_make(cpu_count=0)
-        self.run_make("install", cpu_count=0)
+        self.run_make()
+        self.run_make("install")
 
 
 class Sqlite(ModuleBuilder):

tests/test_invoker.py

@@ -3,7 +3,7 @@
 
 
 def test_invoker(cli):
-    cli.run("-v", "inspect", "invoker", "-v", "-mall")
+    cli.run("inspect", "invoker", "-v", "-mall")
 
     # Invoker may not be completely clean, but it has to have at least one OK .so usage
     m = re.search(r"^\.so files: .+, (\d+) OK", cli.logged.stdout.contents(), re.MULTILINE)