Phillip Smyth authored Nov 07, 2018 and Jürg Billeter committed Nov 12, 2018

_project.py: Added validate_nodes() helper function to prevent duplicate lists

element-path was being used before node validation resulting in uncaught errors

Element path not validated before use

See merge request !937

Valentin David authored Nov 12, 2018 and Valentin David committed Nov 14, 2018

Fixes #755

Run tests on aarch64

Closes #369 and #755

See merge request !948

It's inconvenient to have to create a Linux platform to parse the bwrap version
and we want to get the version in a consistent manner.

If `bwrap` fails to set up the sandbox and start the payload command
it won't write an exit-code in --json-status-fd,
so we can report if it was a sandboxing failure if we don't get exit-code status
and a payload command failure if we do and it's non-zero.

Closes #286

Distinguish between bubblewrap sandboxing failure and command failure

Closes #286

See merge request !868

Angelos Evripiotis authored Nov 08, 2018 and Jürg Billeter committed Nov 15, 2018

Decrease uncertainty around whether unit tests are welcome in the
project or not.

contributing: add guidance on unit tests

See merge request !943

Tom Pollard authored Nov 05, 2018 and Jürg Billeter committed Nov 17, 2018

_context.py: Add pull_buildtrees global user context, the default
of which is set to False via the addition of pull-buildtrees to
userconfig.yaml cache group.

_frontend/app.py & cli.py: Add --pull-buildtrees as a bst main
option, which when passed will override the default or user defined
context for pull_buildtrees.

tests/completions/completions.py: Update for the added flag.

Tom Pollard authored Nov 05, 2018 and Jürg Billeter committed Nov 17, 2018

The addition of cached build trees being included in element
artifacts has led to mostly redundant download overheads when
pulling from a remote artifact server. As such the default behaviour
of pull shouldn't fetch the build tree object if available.

element.py: extend relevant pull logic for specified subdir
consideration and ensure push logic does not lead to partial
artifact pushes. Change assumption that the buildtree can be
extracted if artifact is cached. __cached_buildtree() and
__pull_directories() helpers added.

_artifactcache/: artifactcache.py & cascache.py inclusion of
helper functions for subdir artifact checking & fetching, fetch
logic extended to only pull required artifact directories.
extract & checkout updated to handle full/partial operation.

tests/: addition of integration test pullbuildtrees.py,
buildtrees.py adapted cli options, testutils/artifactshare.py
has_artifact changed to return artifact digest on true condition.

Don't pull artifact buildtrees by default

See merge request !786

In some virtual environments, we might not have the binary with the
exact python version, which would lead the current pip plugin to
fail to find a working pip version.

This adds "python" at the start of the list of valid python versions

plugins/sources/pip.py: also look for python version named "python"

Closes #758

See merge request !942

Benjamin Schubert authored Nov 12, 2018 and richardmaw-codethink committed Nov 19, 2018

Previously the result of the first call in test_build_track was not
checked to be successful, which would lead to potential erros later on
that were hard to debug.

Let's check it everytime

Benjamin Schubert authored Nov 07, 2018 and richardmaw-codethink committed Nov 19, 2018

According to the documentation
(https://www.unix.com/man-page/POSIX/3posix/rename/), when the directory
already is there, either EEXIST or ENOTEMPTY could be thrown.

Previously only ENOTEMPTY was checked.

Done:
  - Separated the move into its own function
  - Check for both errors
  - Create unit tests for it, covering most test cases

Benjamin Schubert authored Nov 09, 2018 and richardmaw-codethink committed Nov 19, 2018

Moving atomically a file/directory can be tricky since different
errors might be raised for the same underlying problem.

Having a utility function to reduce this discrepancies will help
in ensuring we have correct behavior

Benjamin Schubert authored Nov 09, 2018 and richardmaw-codethink committed Nov 19, 2018

This uses move_atomic insteand of the manual os.rename and manual error
checking and throws a SourceError for consistency with other modules.

Fix os.rename in git source element to correctly handle error codes

See merge request !938

Chandan Singh authored Oct 23, 2018 and Chandan Singh committed Nov 19, 2018

Currently, `Pipeline.assert_consistent()` prints an error message like
the one below when certain elements have inconsistent sources:

```
...
      Element: hello.bst is inconsistent
        Source tar source at hello.bst [line 16 column 2] is missing ref
...
```

Drop the word "Source" from the beginning of the message as
"Source <source-kind> source ..." is kind of awkward to read.

Chandan Singh authored Sep 22, 2018 and Chandan Singh committed Nov 19, 2018

As discussed in
https://mail.gnome.org/archives/buildstream-list/2018-September/msg00064.html,
add `bst source-checkout` command. This will allow users to checkout
sources for a given target.

* _frontend/cli.py: Add source-checkout command
* _pipeline.py: Add assert_sources_cached() method
* _stream.py: Add source_checkout method, abstract out
  __check_location_writable() method that used to part of checkout()

Add `bst source-checkout` command

See merge request !820

Abderrahim Kitouni authored Oct 28, 2018 and Valentin David committed Nov 19, 2018

cmake sometimes misinterprets relative paths as relative to the current directory
if this is not specified. See freedesktop-sdk/freedesktop-sdk#431

adjust tests/format/variables.py accordingly.

plugins/elements/cmake.yaml: always specify variable types

Closes #761

See merge request !947

Sandbox errors (like missing host tools) are dependent on the host
system and rarely on what is actually done.

It is therefore better to not cache them as they are subject to
change between two runs.

Also add test to ensure sandbox failure are not cached

Reverse the way we were handling caching by only caching
when we know it's a problem with the build or the build is a
success in order not to cache transient errors.

Don't cache sandbox failures

Closes #727

See merge request !895

This doesn't change the test behavior and enable for better
composability of tests

This will allow for further de-duplication of work.

The unix-logs is always empty, there is therefore no reasons of keeping
it there

This makes all tests use the same template, which makes reasoning
about them simpler

This allows us to make sure that tests behave nicely while missing brwap
and ostree.

This removes the need of having it synchronized in multiple places