Rob Browning <
r...@defaultvalue.org> writes:
> Greg Troxel <
g...@lexort.com> writes:
>
>> My last build that was ok:
>>
>> ==> CHECK.0.33-866-g2fc0ebb8.python3.13.UTC <==
>> ========== 165 passed, 12 skipped, 1 deselected in 1691.87s (0:28:11) ==========
>>
>> and just now a lot failed. Maybe this is a new test dependency not documented?
>>
>>
>> /usr/pkg/lib/python3.13/subprocess.py:577: CalledProcessError
>> ----------------------------- Captured stderr call -----------------------------
>> Could not find platform independent libraries <prefix>
>> Fatal Python error: Failed to import encodings module
>> Python runtime state: core initialized
>> ModuleNotFoundError: No module named 'encodings'
>
> Hmm, I don't see anywhere we import that directly. Wonder if
> something's odd or changed with respect to the local system.
I have just upgraded a bunch of things, but bisecting is looking like
it's not my system. I'm just not 100% sure yet.
> The other possibility might be that the changes I made when we raised
> the minimum to 3.8 aren't OK on NetBSD.
>
> If you want to test that, you could try before/after
>
> cf087f523e7148df6d6771ca7c5b65d4e98db054
> Raise minimum python from 3.7 to 3.8
One commit later it's good
$ tail -1 CHECK.0.33-864-gbbc619a9.python3.13.UTC
========== 165 passed, 12 skipped, 1 deselected in 1618.29s (0:26:58) ==========
(just because that test is sitting there already).
I had been bisecting. Just finished:
$ git bisect good
1b4976f9fc100f12b8b1a34d1a7e96e5fcdd4116 is the first 'bad' commit
commit 1b4976f9fc100f12b8b1a34d1a7e96e5fcdd4116 (upstream/main, upstream/HEAD, github-upstream/master, github-upstream/main, github-upstream/HEAD, main)
Author: Rob Browning <
r...@defaultvalue.org>
Date: Tue Jul 7 16:28:45 2026 -0500
RemoteRepo._config_get_fallback: remove unnecessary return (pylint)
Signed-off-by: Rob Browning <
r...@defaultvalue.org>
Tested-by: Rob Browning <
r...@defaultvalue.org>
lib/bup/repo/remote.py | 1 -
1 file changed, 1 deletion(-)
with status by hash
==> CHECK.0.33-855-g3d6869ea.python3.13.UTC <==
========== 165 passed, 12 skipped, 1 deselected in 1561.12s (0:26:01) ==========
==> CHECK.0.33-864-gbbc619a9.python3.13.UTC <==
========== 165 passed, 12 skipped, 1 deselected in 1618.29s (0:26:58) ==========
==> CHECK.0.33-866-g2fc0ebb8.python3.13.UTC <==
========== 165 passed, 12 skipped, 1 deselected in 1691.87s (0:28:11) ==========
==> CHECK.0.33-869-geef904c0.python3.13.UTC <==
========== 165 passed, 12 skipped, 1 deselected in 2088.47s (0:34:48) ==========
==> CHECK.0.33-871-gc507fab9.python3.13.UTC <==
===== 1 failed, 163 passed, 14 skipped, 1 deselected in 1967.92s (0:32:47) =====
==> CHECK.0.33-872-g1b4976f9.python3.13.UTC <==
===== 46 failed, 127 passed, 5 skipped, 1 deselected in 1401.71s (0:23:21) =====
I called 871 good, since it seemed a minor and likely separate issue.
This points to removing the unnecessary return. I am having a hard
time following.
I have reverted 1b4976f9fc100f12b8b1a34d1a7e96e5fcdd4116 locally and am
re-running tests. Yes, I realized that's like checking out
1b4976f9fc100f12b8b1a34d1a7e96e5fcdd4116^.