From 41edb7660e2facff1afae57b97b0fde35225a492 Mon Sep 17 00:00:00 2001 From: Will Greenberg Date: Mon, 24 Oct 2022 19:51:35 -0700 Subject: [PATCH] mypy fixup --- certbot-apache/certbot_apache/_internal/override_centos.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/certbot-apache/certbot_apache/_internal/override_centos.py b/certbot-apache/certbot_apache/_internal/override_centos.py index acb29284d..506908837 100644 --- a/certbot-apache/certbot_apache/_internal/override_centos.py +++ b/certbot-apache/certbot_apache/_internal/override_centos.py @@ -152,8 +152,10 @@ class CentOSConfigurator(BaseCentOSConfigurator): def _override_cmds(self) -> None: """ As of RHEL 9, apachectl can't be passed flags like "-t -D", so instead - use options.bin (which is "httpd") + use options.bin (i.e. httpd) """ + if not self.options.bin: + raise ValueError("OS option apache_bin must be set for CentOS") self.options.get_modules_cmd[0] = self.options.bin self.options.get_includes_cmd[0] = self.options.bin self.options.get_defines_cmd[0] = self.options.bin