rick/LEDMatrix
1
0
Fork 0
mirror of https://github.com/ChuckBuilds/LEDMatrix.git synced 2026-05-25 13:43:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix uninstall-no-record-file detection condition

Browse Source 
The previous check used a string replacement that left 'error:' in the
remaining text, causing the condition to always evaluate false. Simplify
to a direct substring check: if 'uninstall-no-record-file' appears in pip
stderr the affected package is installed at the system level and we write
the marker, suppressing the repeated warning on every restart.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Chuck
2026-05-23 16:33:24 -04:00
parent 3d4de89fd5
commit 53806da8c5
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/plugin_system/plugin_loader.py
View File

@@ -176,7 +176,7 @@ class PluginLoader:
# system level (e.g. installed via dnf/apt without a pip RECORD file). # system level (e.g. installed via dnf/apt without a pip RECORD file).
# pip can't replace it, but it IS installed — write the marker so we # pip can't replace it, but it IS installed — write the marker so we
# don't retry on every restart. # don't retry on every restart.
if "uninstall-no-record-file" in stderr and "error" not in stderr.lower().replace("uninstall-no-record-file", ""): if "uninstall-no-record-file" in stderr:
self.logger.warning( self.logger.warning(
"Dependencies for %s include system-managed packages (no pip RECORD). " "Dependencies for %s include system-managed packages (no pip RECORD). "
"Assuming they are satisfied: %s", "Assuming they are satisfied: %s",