Chandan Singh authored Sep 26, 2018 and Benjamin Schubert committed Dec 03, 2018

Currently, `bst checkout --deps none` command always produces empty
output. Fix this issue and add regression test for the same.

- element_enums.py: Add Scope.NONE.

- element.py: Ensure Scope.NONE works correctly in addition to
  Scope.RUN/Scope.ALL in Element.dependencies() and Element.search().

- tests/frontend/buildcheckout.py: Fix tests for `--deps none`.

Fixes #670.

Chandan Singh authored Sep 21, 2018 and Benjamin Schubert committed Dec 03, 2018

As discussed in
https://mail.gnome.org/archives/buildstream-list/2018-September/msg00064.html,
add `--deps build` option to `bst checkout`. This will allow users to
checkout the all build dependencies of a given element using a single
command.

- _frontend/cli.py: Add `--deps build` option for `bst checkout`.

- element.py: Support `deps='build'` in Element._prepare_sandbox().

- tests/frontend/buildcheckout.py: Ensure `--deps build` works as
  expected.

Before we would have a intricate logics with multiple arguments
that might get ignored.

This simplifies the design and introduces a bool `shell` instead
of having two different variables concerned about scope

Add `--deps build` option to `bst checkout`

Closes #670

See merge request !819

Javier Jardón authored Dec 02, 2018 and Tristan Van Berkom committed Dec 03, 2018

See #783

git: warn about checkout submodules  by default

See merge request !983

This change makes `ArtifactCache.setup_remotes()` use a boolean parameter, set
by `Stream.push()` or `Stream.pull()`, to configure the overriden `remote_url`
correctly, instead of for `push` by default.

This fixes #789.

This change simplifies the responsibilities of `setup_remotes()` by extracting
"getting the default remotes" logic to its own function and moving "--remote"
related logic to _load(). The latter, especially, helps to remove the need of
passing the "remote_url" string and "can_push?" boolean to `setup_remotes()`.