From 8f324eceefe0b5ab0b05ad1157128ded3b97e6d1 Mon Sep 17 00:00:00 2001 From: Ruben Vorderman Date: Mon, 19 Apr 2021 14:12:02 +0200 Subject: [PATCH] Also check for boolean True, not just string 'True' when evaluating conditional dependencies. When people know how actual yaml works they might have galaxy: watch_tools: true as their config. Which signifies an actual Boolean value. This should be considered by the script as well. --- lib/galaxy/dependencies/__init__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/galaxy/dependencies/__init__.py b/lib/galaxy/dependencies/__init__.py index 2587a4bc5de..5fc4fe47d37 100644 --- a/lib/galaxy/dependencies/__init__.py +++ b/lib/galaxy/dependencies/__init__.py @@ -180,7 +180,7 @@ class ConditionalDependencies(object): return 'irods' in self.object_stores def check_watchdog(self): - install_set = {'auto', 'True', 'true', 'polling'} + install_set = {'auto', 'True', 'true', 'polling', True} return (self.config['watch_tools'] in install_set or self.config['watch_tool_data_dir'] in install_set)