From 8ab342f21c4c4f88ee8b87ef7d9e4b3ef6b30291 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jelmer=20Vernoo=C4=B3?= Date: Fri, 26 Mar 2021 17:23:28 +0000 Subject: [PATCH] ADd TODO. --- ognibuild/debian/fix_build.py | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/ognibuild/debian/fix_build.py b/ognibuild/debian/fix_build.py index 66ed2fb..5e12726 100644 --- a/ognibuild/debian/fix_build.py +++ b/ognibuild/debian/fix_build.py @@ -203,11 +203,7 @@ class PackageDependencyFixer(BuildFixer): def add_dependency(context, phase, requirement: AptRequirement): if phase[0] == "autopkgtest": - return add_test_dependency( - context, - phase[1], - requirement, - ) + return add_test_dependency(context, phase[1], requirement) elif phase[0] == "build": return add_build_dependency(context, requirement) else: @@ -249,6 +245,9 @@ def add_test_dependency(context, testname, requirement): tests_control_path = context.abspath("debian/tests/control") + # TODO(jelmer): If requirement is for one of our binary packages + # but "@" is already present then don't do anything. + try: with Deb822Editor(path=tests_control_path) as updater: command_counter = 1