From bb3cd6d4e3270adf85d8e2c72fd761ee3bf61dd4 Mon Sep 17 00:00:00 2001 From: Giulio Eulisse <10544+ktf@users.noreply.github.com> Date: Thu, 3 Oct 2024 11:42:10 +0200 Subject: [PATCH] Revert "Add --only-deps option for caching (#870)" This reverts commit e45487b48ba53e7267923c2a44868d272aa6da25. --- alibuild_helpers/args.py | 2 -- alibuild_helpers/build.py | 6 ------ tests/test_build.py | 1 - 3 files changed, 9 deletions(-) diff --git a/alibuild_helpers/args.py b/alibuild_helpers/args.py index da45b0bc..8f48bc42 100644 --- a/alibuild_helpers/args.py +++ b/alibuild_helpers/args.py @@ -107,8 +107,6 @@ def doParseArgs(): "in multiple packages. The comment will only be stored if " "PACKAGE is compiled or downloaded during this run; if it " "already exists, this does not happen.")) - build_parser.add_argument("--only-deps", dest="onlyDeps", default=False, action="store_true", - help="Only build dependencies, not the main package (e.g. for caching)") build_docker = build_parser.add_argument_group(title="Build inside a container", description="""\ Builds can be done inside a Docker container, to make it easier to get a diff --git a/alibuild_helpers/build.py b/alibuild_helpers/build.py index ebcf086d..fccc30bb 100644 --- a/alibuild_helpers/build.py +++ b/alibuild_helpers/build.py @@ -693,12 +693,6 @@ def doBuild(args, parser): deps=",".join(buildOrder[:-1]), ), args.architecture) - # If we are building only the dependencies, the last package in - # the build order can be considered done. - if args.onlyDeps and len(buildOrder) > 1: - mainPackage = buildOrder.pop() - warning("Not rebuilding %s because --only-deps option provided.", mainPackage) - while buildOrder: p = buildOrder[0] spec = specs[p] diff --git a/tests/test_build.py b/tests/test_build.py index a0f4ed72..246f00fb 100644 --- a/tests/test_build.py +++ b/tests/test_build.py @@ -276,7 +276,6 @@ def test_coverDoBuild(self, mock_debug, mock_listdir, mock_warning, mock_sys, mo environment={}, autoCleanup=False, noDevel=[], - onlyDeps=False, fetchRepos=False, forceTracked=False, plugin="legacy"