Skip to content

Commit

Permalink
Revert "Add --only-deps option for caching (#870)"
Browse files Browse the repository at this point in the history
This reverts commit e45487b.
  • Loading branch information
ktf authored Oct 3, 2024
1 parent e45487b commit bb3cd6d
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 9 deletions.
2 changes: 0 additions & 2 deletions alibuild_helpers/args.py
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
6 changes: 0 additions & 6 deletions alibuild_helpers/build.py
Original file line number Diff line number Diff line change
Expand Up @@ -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]
Expand Down
1 change: 0 additions & 1 deletion tests/test_build.py
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down

0 comments on commit bb3cd6d

Please sign in to comment.