Skip to content

Commit 0dcddd7

Browse files
author
Tjaz Erzen
committed
Remove verifying system requirements at runtime
1 parent ce04ab6 commit 0dcddd7

3 files changed

Lines changed: 0 additions & 30 deletions

File tree

config/system_config.yaml

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,5 @@
11
client_version: "0.17.0"
22

3-
system_requirements:
4-
timeout:
5-
command: timeout
6-
error_message: |
7-
Error: Required system command 'timeout' is not available.
8-
This command is needed to enforce time limits on test execution.
9-
10-
To install the timeout command:
11-
- On Linux: Install coreutils package
12-
- Debian/Ubuntu: sudo apt-get install coreutils
13-
- CentOS/RHEL: sudo yum install coreutils
14-
- Fedora: sudo dnf install coreutils
15-
- On macOS:
16-
- Using Homebrew: brew install coreutils
17-
- Using MacPorts: port install coreutils
18-
- On Windows: Use Windows Subsystem for Linux (WSL), Cygwin, or Git Bash
19-
203
error_messages:
214
template_not_found:
225
message: |

plain2code.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -185,9 +185,6 @@ def _check_connection(codeplainAPI):
185185

186186

187187
def render(args, run_state: RunState, codeplain_api, event_bus: EventBus): # noqa: C901
188-
# Check system requirements before proceeding
189-
system_config.verify_requirements()
190-
191188
template_dirs = file_utils.get_template_directories(args.filename, args.template_dir, DEFAULT_TEMPLATE_DIRS)
192189

193190
console.info(f"Rendering {args.filename} to target code.")

system_config.py

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import importlib.resources
2-
import shutil
32
import sys
43

54
import yaml
@@ -14,8 +13,6 @@ def __init__(self):
1413
self.config = self._load_config()
1514
if "client_version" not in self.config:
1615
raise KeyError("Missing 'client_version' in system_config.yaml")
17-
if "system_requirements" not in self.config:
18-
raise KeyError("Missing 'system_requirements' section in system_config.yaml")
1916
if "error_messages" not in self.config:
2017
raise KeyError("Missing 'error_messages' section in system_config.yaml")
2118

@@ -34,13 +31,6 @@ def _load_config(self):
3431
console.error(f"Failed to load system configuration: {e}")
3532
sys.exit(69)
3633

37-
def verify_requirements(self):
38-
"""Verify all system requirements are met."""
39-
for req_data in self.requirements.values():
40-
if not shutil.which(req_data["command"]):
41-
console.error(req_data["error_message"])
42-
sys.exit(69)
43-
4434
def get_error_message(self, message_key, **kwargs):
4535
"""Get a formatted error message by its key."""
4636
if message_key not in self.error_messages:

0 commit comments

Comments
 (0)