Skip to content

Commit 612ccdf

Browse files
committed
Fixed last remaining subprocess calls with safe print.
1 parent f5bb07b commit 612ccdf

1 file changed

Lines changed: 4 additions & 6 deletions

File tree

omnipkg/core.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -314,7 +314,7 @@ def _has_suitable_python311(self) -> bool:
314314
python_311_path = registry.get('interpreters', {}).get('3.11')
315315
if python_311_path and Path(python_311_path).exists():
316316
try:
317-
result = subprocess.run([python_311_path, '-c', "import sys; safe_print(f'{sys.version_info.major}.{sys.version_info.minor}')"], capture_output=True, text=True, timeout=5)
317+
result = subprocess.run([python_311_path, '-c', "import sys; print(f'{sys.version_info.major}.{sys.version_info.minor}')"], capture_output=True, text=True, timeout=5)
318318
if result.returncode == 0 and result.stdout.strip() == '3.11':
319319
return True
320320
except:
@@ -335,7 +335,7 @@ def _has_suitable_python311(self) -> bool:
335335
exe_path = bin_dir / (f'{possible_name}.exe' if platform.system() == 'Windows' else possible_name)
336336
if exe_path.exists():
337337
try:
338-
result = subprocess.run([str(exe_path), '-c', "import sys; safe_print(f'{sys.version_info.major}.{sys.version_info.minor}')"], capture_output=True, text=True, timeout=5)
338+
result = subprocess.run([str(exe_path), '-c', "import sys; print(f'{sys.version_info.major}.{sys.version_info.minor}')"], capture_output=True, text=True, timeout=5)
339339
if result.returncode == 0 and result.stdout.strip() == '3.11':
340340
return True
341341
except:
@@ -1187,10 +1187,8 @@ def _get_sensible_defaults(self, python_exe_override: str = None) -> Dict:
11871187
(CORRECTED) Generates sensible default configuration paths based STRICTLY on the
11881188
currently active virtual environment to ensure safety and prevent discovery conflicts.
11891189
"""
1190-
safe_print(_('💡 Grounding configuration in the current active environment...'))
11911190
active_python_exe = sys.executable
1192-
safe_print(_(' ✅ Using: {} (Your active interpreter)').format(active_python_exe))
1193-
1191+
11941192
# --- THIS IS THE CRITICAL FIX ---
11951193
# First, get the paths for the currently running, active interpreter. This is ground truth.
11961194
calculated_paths = self._get_paths_for_interpreter(active_python_exe)
@@ -5428,7 +5426,7 @@ def ignore_patterns(dir, files):
54285426
safe_print(_(' - ❌ Copy completed but Python executable not found in destination'))
54295427
shutil.rmtree(dest, ignore_errors=True)
54305428
return self._fallback_to_download(version)
5431-
test_cmd = [str(copied_python), '-c', 'import sys; safe_print(sys.version)']
5429+
test_cmd = [str(copied_python), '-c', 'import sys; print(sys.version)']
54325430
test_result = subprocess.run(test_cmd, capture_output=True, timeout=10)
54335431
if test_result.returncode != 0:
54345432
safe_print(_(' - ❌ Copied Python executable failed basic test'))

0 commit comments

Comments
 (0)