diff options
Diffstat (limited to 'app/tasks/minetestcheck/tree.py')
| -rw-r--r-- | app/tasks/minetestcheck/tree.py | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/app/tasks/minetestcheck/tree.py b/app/tasks/minetestcheck/tree.py index 07e67f6..57a769f 100644 --- a/app/tasks/minetestcheck/tree.py +++ b/app/tasks/minetestcheck/tree.py @@ -128,17 +128,20 @@ class PackageTreeNode: result["optional_depends"] = [] - # Check dependencies - for dep in result["depends"]: - if not basenamePattern.match(dep): - raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, names must only contain a-z0-9_.") \ - .format(dep, self.relative)) + def checkDependencies(deps): + for dep in result["depends"]: + if not basenamePattern.match(dep): + if " " in dep: + raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, did you forget a comma?") \ + .format(dep, self.relative)) + else: + raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, names must only contain a-z0-9_.") \ + .format(dep, self.relative)) - for dep in result["optional_depends"]: - if not basenamePattern.match(dep): - raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, names must only contain a-z0-9_.") \ - .format(dep, self.relative)) + # Check dependencies + checkDependencies(result["depends"]) + checkDependencies(result["optional_depends"]) # Fix games using "name" as "title" if self.type == ContentType.GAME: |