summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* doc/Makefile: Update comment about sphinx entrypointchandan/update-doc-makefile-noteChandan Singh2019-01-021-4/+4
| | | | | | https://github.com/sphinx-doc/sphinx/issues/4375 has been closed as `wontfix`. Update the comment in doc/Makefile that mentions it, based on the discussion on the issue.
* Merge branch 'chandan/element-filename-requirements' into 'master'Tristan Van Berkom2018-12-317-18/+127
|\ | | | | | | | | Add warnings about invalid characters in filename See merge request BuildStream/buildstream!1028
| * doc: Add note about valid element namesChandan Singh2018-12-311-0/+24
| |
| * tests/buildcheckout.py: Add tests for invalid chars in filenameChandan Singh2018-12-314-0/+27
| |
| * _loader/loader.py: Add warnings about invalid characters in filenameChandan Singh2018-12-312-0/+43
| | | | | | | | | | See https://mail.gnome.org/archives/buildstream-list/2018-December/msg00061.html for some related discussion.
| * _loader/loader.py: Refactor warnings about element names in one methodChandan Singh2018-12-311-18/+33
|/ | | | | | | Currently we some duplication in the way we check for invalid filenames. To make it more robust and allow room for adding more warnings, refactor it into a separate method `_warn_invalid_elements()` that handles just this.
* Merge branch 'tristan/junction-tests' into 'master'Tristan Van Berkom2018-12-312-40/+47
|\ | | | | | | | | Junction test addition See merge request BuildStream/buildstream!1029
| * tests/loader/junctions.py: Test the error we expect when a cross junction ↵Tristan Van Berkom2018-12-312-1/+23
| | | | | | | | element is not found
| * tests/loader/junctions.py: Use Result checking APIs instead of manuallyTristan Van Berkom2018-12-311-39/+24
|/ | | | Better to use the same API everywhere for thecking CLI results.
* Merge branch 'tristan/fix-netrc-crasher' into 'master'Tristan Van Berkom2018-12-263-3/+28
|\ | | | | | | | | Fix netrc crasher See merge request BuildStream/buildstream!1025
| * tests/sources/tar.py: Test that we don't crash when HOME is unsetTristan Van Berkom2018-12-261-0/+16
| | | | | | | | | | This happens when the netrc module is searching for a ~/.netrc file and it doesnt find any HOME set.
| * tests/testutils/runcli.py: Fixed broken environment handlingTristan Van Berkom2018-12-261-2/+5
| | | | | | | | | | Treat None values in the passed dictionary as keys to be removed from the environment, this was already happening at restoration time.
| * _downloadablefilesource.py: Avoid crashes when HOME is unsetTristan Van Berkom2018-12-261-1/+7
|/ | | | | | The python netrc module will raise OSError in the case that HOME is not set, this was discovered while running tests under tox.
* Merge branch 'tristan/test-fixes' into 'master'Tristan Van Berkom2018-12-2610-21/+11
|\ | | | | | | | | | | | | Misc fixes to the tests and newer linter errors Closes #636 See merge request BuildStream/buildstream!1011
| * dev-requirements.txt: Remove the restriction on pylint.tristan/test-fixesTristan Van Berkom2018-12-261-1/+1
| | | | | | | | | | | | After having fixed the more recent linting errors, the tests should pass with recent versions of pylint. Tested with pylint 2.2.2.
| * dev-requirements.txt: Require new version of pytest-datafilesTristan Van Berkom2018-12-261-1/+1
| | | | | | | | | | | | | | | | Since Thomas Nixon fixed the datafiles plugin to be compatible with pytest >= 3.8.0, we can now depend on it and avoid the warnings and errors which the older datafiles plugin causes. This fixes issue #636
| * buildstream/storage/_casbaseddirectory.py: Fixing new (unnecessary-pass) ↵Tristan Van Berkom2018-12-261-5/+0
| | | | | | | | linter errors
| * plugins/elements/import.py: Fixing new ↵Tristan Van Berkom2018-12-261-1/+1
| | | | | | | | (duplicate-string-formatting-argument) linter error
| * _frontend/widget.py: Fixing new (wrong-import-order) linter errorTristan Van Berkom2018-12-261-1/+1
| |
| * _frontend/complete.py: Fixing new (simplifiable-if-expression) linter errorTristan Van Berkom2018-12-261-1/+1
| |
| * _frontend/cli.py: Fixing new (unnecessary-pass) linter errorsTristan Van Berkom2018-12-261-3/+0
| |
| * _artifactcache/casserver.py: Fixed new (wrong-import-order) linter errorTristan Van Berkom2018-12-261-1/+1
| |
| * source.py: Fixing new (unnecessary-pass) linter errorTristan Van Berkom2018-12-261-1/+0
| |
| * element.py: Fixing new (unnecessary-pass) and (simplifiable-if-expression) ↵Tristan Van Berkom2018-12-261-3/+2
| | | | | | | | linter errors
| * _signals.py: Fixing new (simplifiable-if-expression) linter errorsTristan Van Berkom2018-12-261-2/+2
| |
| * dev-requirements.txt: Require at least pytest 3.9Tristan Van Berkom2018-12-261-1/+1
|/ | | | | Recently we have been adding some tests which use the new tmp_path fixture from pytest, this is new in 3.9.
* Merge branch 'tpollard/fixbuildtreereq' into 'master'Tom Pollard2018-12-211-1/+1
|\ | | | | | | | | _stream.py: fix _buildtree_pull_required logic See merge request BuildStream/buildstream!1021
| * _stream.py: fix _buildtree_pull_required logictpollard/fixbuildtreereqTom Pollard2018-12-201-1/+1
|/
* Merge branch 'chandan/fix-warning' into 'master'Chandan Singh2018-12-202-4/+4
|\ | | | | | | | | Fix Deprecation warnings from regex module See merge request BuildStream/buildstream!1010
| * buildstream/utils.py: Fix regex Deprecation WarningChandan Singh2018-12-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | Specify flags at the start of the expression as per the recommendation of the standard library. Without this patch, we currently get the following warning: ``` tests/examples/junctions.py::test_open_cross_junction_workspace /builds/BuildStream/buildstream/dist/buildstream/buildstream/utils.py:213: DeprecationWarning: Flags not at the start of the expression '\\/[^/]*\\Z(?ms)' regexer = re.compile(expression) ```
| * tests/frontend/logging.py: Fix regex Deprecation WarningChandan Singh2018-12-201-2/+2
|/ | | | | | | | | | | | | | | | | Use raw strings for regex searches, which is the preferred way to do regular expressions in Python. Without this patch, currently we get the following warnings: ``` tests/frontend/logging.py:44 /builds/BuildStream/buildstream/dist/buildstream/tests/frontend/logging.py:44: DeprecationWarning: invalid escape sequence \[ m = re.search("\[\d\d:\d\d:\d\d\]\[\]\[\] SUCCESS Checking sources", result.stderr) tests/frontend/logging.py:80 /builds/BuildStream/buildstream/dist/buildstream/tests/frontend/logging.py:80: DeprecationWarning: invalid escape sequence \d m = re.search("\d\d:\d\d:\d\d,\d\d:\d\d:\d\d.\d{6},\d\d:\d\d:\d\d,,,SUCCESS,Checking sources", result.stderr) ```
* Merge branch 'aevri/rm-autoinit' into 'master'Angelos Evripiotis2018-12-205-27/+20
|\ | | | | | | | | | | | | BREAK: remove auto-init behaviour Closes #826 See merge request BuildStream/buildstream!1015
| * BREAK: remove auto-init behaviourAngelos Evripiotis2018-12-204-24/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the event that the project could not be found, stop BuildStream from asking if the user would like to create a new project. Exit with error instead, and give a hint to the user in case they're new. As proposed on the mailing list here: https://mail.gnome.org/archives/buildstream-list/2018-December/msg00082.html The new interaction looks like this: $ bst show nonsuch.bst No project found. You can create a new project like so: bst init Error loading project: None of ['project.conf', '.bstproject.yaml'] found in '/src/temp/blah' or any of its parent directories Fixes #826
| * _project::_find_project_dir: fix error messageAngelos Evripiotis2018-12-201-3/+4
|/ | | | | | | | | | | | Now that we are also looking for WORKSPACE_PROJECT_FILE, we should add it to the error message. While we're there, also mention the directory we are looking in - this could help scripters be clear about what went wrong. The new error message looks like this: Error loading project: None of ['project.conf', '.bstproject.yaml'] found in '/src/temp/blah' or any of its parent directories
* Merge branch 'chandan/abstract-git-source' into 'master'Chandan Singh2018-12-203-644/+678
|\ | | | | | | | | | | | | Expose base class for Git source plugins Closes #739 See merge request BuildStream/buildstream!1019
| * Expose base class for Git source pluginsChandan Singh2018-12-203-644/+678
|/ | | | | | | | | | | | | | | | | | | Add a `_GitSourceBase` class to act as the base class for deriving source plugins that work with Git. The first user of this base class is the `git` source plugin that's in core at the moment. The contents of this base class is almost identical to the existing `GitSource` class. The two notable differences are: - the private methods now have leading underscores, as some were missing it originally - the name of the class Note that we are exposing a private member here as we expect it to move to a separate package soon. See the following discussion for more details: https://gitlab.com/BuildStream/buildstream/issues/739#note_124819869 Fixes #739.
* Merge branch 'juerg/fetch' into 'master'Jürg Billeter2018-12-206-4/+113
|\ | | | | | | | | Do not call fetch() for cached sources See merge request BuildStream/buildstream!992
| * tests/sources: Test that fetch() is not called for cached sourcesJürg Billeter2018-12-204-0/+98
| |
| * element.py: Do not call fetch() for cached sourcesJürg Billeter2018-12-201-1/+2
| |
| * Move fetch logic from FetchQueue to ElementJürg Billeter2018-12-202-4/+14
|/ | | | | The queue shouldn't need to know about individual sources. This is in line with _track() and _get_consistency().
* Merge branch 'jjardon/license_badge' into 'master'716-add-example-with-build-directory-outside-of-source-directoryJavier Jardón2018-12-191-0/+3
|\ | | | | | | | | README.rst: Add license badge See merge request BuildStream/buildstream!1014
| * README.rst: Add license badgeJavier Jardón2018-12-191-0/+3
|/
* Merge branch 'willsalmon/shellBuildTrees' into 'master'Will Salmon2018-12-196-16/+174
|\ | | | | | | | | | | | | Shell --build has optional buildtrees Closes #740 See merge request BuildStream/buildstream!986
| * Basic options for shell --build to use buildtreesWilliam Salmon2018-12-196-16/+174
|/ | | | Fixes issue #740
* Merge branch 'raoul/627-RE-instance-config' into 'master'Jim MacArthur2018-12-183-25/+53
|\ | | | | | | | | | | | | Remote-execution instance configuration support Closes #627 See merge request BuildStream/buildstream!952
| * Documentation: Include instance-name in remote execution documentationraoul/627-RE-instance-configJim MacArthur2018-12-181-1/+10
| |
| * _cascache.py: Add instance names to GRPC callsJim MacArthur2018-12-181-18/+31
| | | | | | | | | | | | | | | | | | Adds the 'instance_name' parameter, which may be None, to most GRPC calls in the CASCache object. ByteStream requests already have instance_name supplied in the resource name, so do not need the parameter. Closes #627.
| * sandboxremote: Add server/storage config with defaultsJim MacArthur2018-12-181-9/+7
| | | | | | | | Adds instance name support for the remote execution storage service.
| * Add remote execution instance optionRaoul Hidalgo Charman2018-12-181-2/+10
|/ | | | | | | This is used when sending execution requests, to specify which instance of the execution server to use. Partial fix for #627.
* Merge branch 'chandan/source-subgroup' into 'master'Chandan Singh2018-12-1428-259/+325
|\ | | | | | | | | | | | | Introduce new "source" command group Closes #814 See merge request BuildStream/buildstream!1003