diff options
author | James Ennis <james.ennis@codethink.co.uk> | 2019-09-11 16:07:24 +0100 |
---|---|---|
committer | Daniel Silverstone <daniel.silverstone@codethink.co.uk> | 2019-09-13 07:33:48 +0000 |
commit | 5a778ca352f2e1e5d4837e826656107db26cff34 (patch) | |
tree | cb318f0b4a41dc8f6b8ea24f92eed7d65012ef14 | |
parent | c30993ebc677bc665425c8f6810628e1302a6f9e (diff) | |
download | buildstream-5a778ca352f2e1e5d4837e826656107db26cff34.tar.gz |
cli.py: Allow checkout to handle --deps all
source checkout supports --deps all, so we should be
consistent.
Additionally, a user may want to have build deps in a
potential chroot so you could rebuild any component.
-rw-r--r-- | src/buildstream/_frontend/cli.py | 4 | ||||
-rw-r--r-- | tests/frontend/buildcheckout.py | 6 |
2 files changed, 5 insertions, 5 deletions
diff --git a/src/buildstream/_frontend/cli.py b/src/buildstream/_frontend/cli.py index d657ec76d..f9c3e9546 100644 --- a/src/buildstream/_frontend/cli.py +++ b/src/buildstream/_frontend/cli.py @@ -1052,7 +1052,7 @@ def artifact_show(app, deps, artifacts): @click.option('--force', '-f', is_flag=True, help="Allow files to be overwritten") @click.option('--deps', '-d', default='run', show_default=True, - type=click.Choice(['run', 'build', 'none']), + type=click.Choice(['run', 'build', 'none', 'all']), help='The dependencies to checkout') @click.option('--integrate/--no-integrate', default=None, is_flag=True, help="Whether to run integration commands") @@ -1121,7 +1121,7 @@ def artifact_checkout(app, force, deps, integrate, hardlinks, tar, compression, if not target: raise AppError('Missing argument "ELEMENT".') - scope = {'run': Scope.RUN, 'build': Scope.BUILD, 'none': Scope.NONE} + scope = {'run': Scope.RUN, 'build': Scope.BUILD, 'none': Scope.NONE, 'all': Scope.ALL} app.stream.checkout(target, location=location, force=force, diff --git a/tests/frontend/buildcheckout.py b/tests/frontend/buildcheckout.py index c9a42239a..7044739a1 100644 --- a/tests/frontend/buildcheckout.py +++ b/tests/frontend/buildcheckout.py @@ -158,7 +158,7 @@ def test_build_invalid_filename_chars_dep(datafiles, cli): @pytest.mark.datafiles(DATA_DIR) -@pytest.mark.parametrize("deps", [("run"), ("none"), ("build")]) +@pytest.mark.parametrize("deps", [("run"), ("none"), ("build"), ("all")]) def test_build_checkout_deps(datafiles, cli, deps): project = str(datafiles) checkout = os.path.join(cli.directory, 'checkout') @@ -187,14 +187,14 @@ def test_build_checkout_deps(datafiles, cli, deps): # Verify output of this element's build dependencies filename = os.path.join(checkout, 'usr', 'include', 'pony.h') - if deps == "build": + if deps in ["build", "all"]: assert os.path.exists(filename) else: assert not os.path.exists(filename) # Verify output of this element's runtime dependencies filename = os.path.join(checkout, 'usr', 'bin', 'hello') - if deps == "run": + if deps in ["run", "all"]: assert os.path.exists(filename) else: assert not os.path.exists(filename) |