Hello community, here is the log from the commit of package cvise for openSUSE:Factory checked in at 2020-11-06 23:45:16 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/cvise (Old) and /work/SRC/openSUSE:Factory/.cvise.new.11331 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "cvise" Fri Nov 6 23:45:16 2020 rev:19 rq:846460 version:1.8.0+git.20201106.e7821e8 Changes: -------- --- /work/SRC/openSUSE:Factory/cvise/cvise.changes 2020-10-30 11:50:07.673838043 +0100 +++ /work/SRC/openSUSE:Factory/.cvise.new.11331/cvise.changes 2020-11-06 23:46:13.647133603 +0100 @@ -1,0 +2,16 @@ +Fri Nov 06 12:45:17 UTC 2020 - mli...@suse.cz + +- Update to version 1.8.0+git.20201106.e7821e8: + * Remove superfluous print. + * Fix killing of timed out processes. + * Save test when it reaches a timeout. + * Move timeout to WARN. + * Fix timeout default. + * Support timestamps for a duration >24 hours. + +------------------------------------------------------------------- +Fri Nov 6 12:44:39 UTC 2020 - Martin Liška <mli...@suse.cz> + +- Use llvm11. + +------------------------------------------------------------------- Old: ---- cvise-1.8.0+git.20201029.d32ab1b.tar.xz New: ---- cvise-1.8.0+git.20201106.e7821e8.tar.xz ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ cvise.spec ++++++ --- /var/tmp/diff_new_pack.haEKdz/_old 2020-11-06 23:46:14.303132343 +0100 +++ /var/tmp/diff_new_pack.haEKdz/_new 2020-11-06 23:46:14.307132336 +0100 @@ -17,7 +17,7 @@ Name: cvise -Version: 1.8.0+git.20201029.d32ab1b +Version: 1.8.0+git.20201106.e7821e8 Release: 0 Summary: Super-parallel Python port of the C-Reduce License: BSD-3-Clause @@ -25,8 +25,8 @@ Source: %{name}-%{version}.tar.xz BuildRequires: astyle %if %{?suse_version} > 1500 -BuildRequires: clang10-devel -BuildRequires: llvm10-devel +BuildRequires: clang11-devel +BuildRequires: llvm11-devel %else BuildRequires: clang9-devel BuildRequires: llvm9-devel @@ -43,8 +43,8 @@ BuildRequires: unifdef Requires: astyle %if %{?suse_version} > 1500 -Requires: clang10 -Requires: llvm10 +Requires: clang11 +Requires: llvm11 %else Requires: clang9 Requires: llvm9 ++++++ cvise-1.8.0+git.20201029.d32ab1b.tar.xz -> cvise-1.8.0+git.20201106.e7821e8.tar.xz ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/cvise-1.8.0+git.20201029.d32ab1b/cvise/utils/testing.py new/cvise-1.8.0+git.20201106.e7821e8/cvise/utils/testing.py --- old/cvise-1.8.0+git.20201029.d32ab1b/cvise/utils/testing.py 2020-10-29 15:42:39.000000000 +0100 +++ new/cvise-1.8.0+git.20201106.e7821e8/cvise/utils/testing.py 2020-11-06 13:13:21.000000000 +0100 @@ -9,8 +9,8 @@ import os.path import pebble import platform +import psutil import shutil -import signal import subprocess import sys import tempfile @@ -303,14 +303,28 @@ active_pids.add(event.pid) for pid in active_pids: try: - os.kill(pid, signal.SIGTERM) - except ProcessLookupError: + process = psutil.Process(pid) + children = process.children(recursive=True) + children.append(process) + for child in children: + try: + child.terminate() + except psutil.NoSuchProcess: + pass + except psutil.NoSuchProcess: pass def release_future(self, future): self.futures.remove(future) self.release_folder(future) + def save_extra_dir(self, test_case_path): + extra_dir = self.get_extra_dir("cvise_extra_", self.MAX_EXTRA_DIRS) + if extra_dir != None: + os.mkdir(extra_dir) + shutil.move(test_case_path, extra_dir) + logging.info("Created extra directory {} for you to look at later".format(extra_dir)) + def process_done_futures(self): quit_loop = False new_futures = set() @@ -324,7 +338,8 @@ if future.exception(): if type(future.exception()) is TimeoutError: self.timeout_count += 1 - logging.debug("Test timed out!") + logging.warning("Test timed out.") + self.save_extra_dir(self.temporary_folders[future]) if self.timeout_count >= self.MAX_TIMEOUTS: logging.warning("Maximum number of timeout were reached: %d" % self.MAX_TIMEOUTS) quit_loop = True @@ -351,11 +366,7 @@ assert test_env.exitcode if (self.also_interesting is not None and test_env.exitcode == self.also_interesting): - extra_dir = self.get_extra_dir("cvise_extra_", self.MAX_EXTRA_DIRS) - if extra_dir != None: - os.mkdir(extra_dir) - shutil.move(test_env.test_case_path, extra_dir) - logging.info("Created extra directory {} for you to look at later".format(extra_dir)) + self.save_extra_dir(test_env.test_case_path) elif test_env.result == PassResult.STOP: quit_loop = True elif test_env.result == PassResult.ERROR: diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/cvise-1.8.0+git.20201029.d32ab1b/cvise.py new/cvise-1.8.0+git.20201106.e7821e8/cvise.py --- old/cvise-1.8.0+git.20201029.d32ab1b/cvise.py 2020-10-29 15:42:39.000000000 +0100 +++ new/cvise-1.8.0+git.20201106.e7821e8/cvise.py 2020-11-06 13:13:21.000000000 +0100 @@ -31,8 +31,10 @@ class DeltaTimeFormatter(logging.Formatter): def format(self, record): - duration = datetime.datetime.utcfromtimestamp(record.relativeCreated / 1000) - record.delta = duration.strftime("%H:%M:%S") + record.delta = str(datetime.timedelta(seconds=int(record.relativeCreated / 1000))) + # pad with one more zero + if record.delta[1] == ':': + record.delta = '0' + record.delta return super().format(record) script_path = os.path.dirname(os.path.realpath(__file__)) @@ -162,7 +164,7 @@ parser.add_argument("--remove-pass", help="Remove all instances of the specified passes from the schedule (comma-separated)") parser.add_argument("--no-timing", action="store_true", default=False, help="Do not print timestamps about reduction progress") parser.add_argument("--timestamp", action="store_true", default=False, help="Print timestamps instead of relative time from a reduction start") - parser.add_argument("--timeout", type=int, nargs="?", const=300, help="Interestingness test timeout in seconds") + parser.add_argument("--timeout", type=int, nargs="?", default=300, help="Interestingness test timeout in seconds") parser.add_argument("--no-cache", action="store_true", default=False, help="Don't cache behavior of passes") parser.add_argument("--skip-key-off", action="store_true", default=False, help="Disable skipping the rest of the current pass when \"s\" is pressed") parser.add_argument("--max-improvement", metavar="BYTES", type=int, help="Largest improvement in file size from a single transformation that C-Vise should accept (useful only to slow C-Vise down)")