Jonathan Maw pushed to branch 463-make-dependency-type-default-to-build-2 at BuildStream / buildstream
Commits:
-
be7b57ea
by Jonathan Maw at 2018-08-10T14:20:59Z
-
80b174fe
by Jonathan Maw at 2018-08-10T14:20:59Z
-
b733e10a
by Jonathan Maw at 2018-08-10T14:20:59Z
-
670fb008
by Jonathan Maw at 2018-08-10T14:20:59Z
12 changed files:
- buildstream/_loader/loadelement.py
- buildstream/_loader/types.py
- buildstream/_versions.py
- doc/source/format_declaring.rst
- tests/loader/dependencies.py
- + tests/loader/dependencies/elements/builddep-list.bst
- tests/loader/dependencies/elements/firstdep.bst
- + tests/loader/dependencies/elements/list-combine.bst
- + tests/loader/dependencies/elements/list-overlap.bst
- + tests/loader/dependencies/elements/runtimedep-list.bst
- + tests/loader/dependencies/elements/seconddep.bst
- + tests/loader/dependencies/elements/thirddep.bst
Changes:
... | ... | @@ -71,6 +71,7 @@ class LoadElement(): |
71 | 71 |
'kind', 'depends', 'sources', 'sandbox',
|
72 | 72 |
'variables', 'environment', 'environment-nocache',
|
73 | 73 |
'config', 'public', 'description',
|
74 |
+ 'build-depends', 'runtime-depends',
|
|
74 | 75 |
])
|
75 | 76 |
|
76 | 77 |
# Extract the Dependencies
|
... | ... | @@ -127,28 +128,44 @@ class LoadElement(): |
127 | 128 |
# Returns:
|
128 | 129 |
# (list): a list of Dependency objects
|
129 | 130 |
#
|
130 |
-def _extract_depends_from_node(node):
|
|
131 |
- depends = _yaml.node_get(node, list, Symbol.DEPENDS, default_value=[])
|
|
131 |
+def _extract_depends_from_node(node, *, key=None):
|
|
132 |
+ if key is None:
|
|
133 |
+ build_depends = _extract_depends_from_node(node, key=Symbol.BUILD_DEPENDS)
|
|
134 |
+ runtime_depends = _extract_depends_from_node(node, key=Symbol.RUNTIME_DEPENDS)
|
|
135 |
+ depends = _extract_depends_from_node(node, key=Symbol.DEPENDS)
|
|
136 |
+ return build_depends + runtime_depends + depends
|
|
137 |
+ elif key == Symbol.BUILD_DEPENDS:
|
|
138 |
+ default_dep_type = Symbol.BUILD
|
|
139 |
+ elif key == Symbol.RUNTIME_DEPENDS:
|
|
140 |
+ default_dep_type = Symbol.RUNTIME
|
|
141 |
+ elif key == Symbol.DEPENDS:
|
|
142 |
+ default_dep_type = None
|
|
143 |
+ |
|
144 |
+ depends = _yaml.node_get(node, list, key, default_value=[])
|
|
132 | 145 |
output_deps = []
|
133 | 146 |
|
134 | 147 |
for dep in depends:
|
135 |
- dep_provenance = _yaml.node_get_provenance(node, key=Symbol.DEPENDS, indices=[depends.index(dep)])
|
|
148 |
+ dep_provenance = _yaml.node_get_provenance(node, key=key, indices=[depends.index(dep)])
|
|
136 | 149 |
|
137 | 150 |
if isinstance(dep, str):
|
138 |
- dependency = Dependency(dep, provenance=dep_provenance)
|
|
151 |
+ dependency = Dependency(dep, provenance=dep_provenance, dep_type=default_dep_type)
|
|
139 | 152 |
|
140 | 153 |
elif isinstance(dep, Mapping):
|
141 |
- _yaml.node_validate(dep, ['filename', 'type', 'junction'])
|
|
142 |
- |
|
143 |
- # Make type optional, for this we set it to None
|
|
144 |
- dep_type = _yaml.node_get(dep, str, Symbol.TYPE, default_value=None)
|
|
145 |
- if dep_type is None or dep_type == Symbol.ALL:
|
|
146 |
- dep_type = None
|
|
147 |
- elif dep_type not in [Symbol.BUILD, Symbol.RUNTIME]:
|
|
148 |
- provenance = _yaml.node_get_provenance(dep, key=Symbol.TYPE)
|
|
149 |
- raise LoadError(LoadErrorReason.INVALID_DATA,
|
|
150 |
- "{}: Dependency type '{}' is not 'build', 'runtime' or 'all'"
|
|
151 |
- .format(provenance, dep_type))
|
|
154 |
+ if default_dep_type:
|
|
155 |
+ _yaml.node_validate(dep, ['filename', 'junction'])
|
|
156 |
+ dep_type = default_dep_type
|
|
157 |
+ else:
|
|
158 |
+ _yaml.node_validate(dep, ['filename', 'type', 'junction'])
|
|
159 |
+ |
|
160 |
+ # Make type optional, for this we set it to None
|
|
161 |
+ dep_type = _yaml.node_get(dep, str, Symbol.TYPE, default_value=None)
|
|
162 |
+ if dep_type is None or dep_type == Symbol.ALL:
|
|
163 |
+ dep_type = None
|
|
164 |
+ elif dep_type not in [Symbol.BUILD, Symbol.RUNTIME]:
|
|
165 |
+ provenance = _yaml.node_get_provenance(dep, key=Symbol.TYPE)
|
|
166 |
+ raise LoadError(LoadErrorReason.INVALID_DATA,
|
|
167 |
+ "{}: Dependency type '{}' is not 'build', 'runtime' or 'all'"
|
|
168 |
+ .format(provenance, dep_type))
|
|
152 | 169 |
|
153 | 170 |
filename = _yaml.node_get(dep, str, Symbol.FILENAME)
|
154 | 171 |
junction = _yaml.node_get(dep, str, Symbol.JUNCTION, default_value=None)
|
... | ... | @@ -159,13 +176,13 @@ def _extract_depends_from_node(node): |
159 | 176 |
|
160 | 177 |
else:
|
161 | 178 |
index = depends.index(dep)
|
162 |
- p = _yaml.node_get_provenance(node, key=Symbol.DEPENDS, indices=[index])
|
|
179 |
+ p = _yaml.node_get_provenance(node, key=key, indices=[index])
|
|
163 | 180 |
raise LoadError(LoadErrorReason.INVALID_DATA,
|
164 | 181 |
"{}: Dependency is not specified as a string or a dictionary".format(p))
|
165 | 182 |
|
166 | 183 |
output_deps.append(dependency)
|
167 | 184 |
|
168 |
- # Now delete "depends", we dont want it anymore
|
|
169 |
- del node[Symbol.DEPENDS]
|
|
185 |
+ # Now delete the field, we dont want it anymore
|
|
186 |
+ del node[key]
|
|
170 | 187 |
|
171 | 188 |
return output_deps
|
... | ... | @@ -26,6 +26,8 @@ class Symbol(): |
26 | 26 |
FILENAME = "filename"
|
27 | 27 |
KIND = "kind"
|
28 | 28 |
DEPENDS = "depends"
|
29 |
+ BUILD_DEPENDS = "build-depends"
|
|
30 |
+ RUNTIME_DEPENDS = "runtime-depends"
|
|
29 | 31 |
SOURCES = "sources"
|
30 | 32 |
CONFIG = "config"
|
31 | 33 |
VARIABLES = "variables"
|
... | ... | @@ -23,7 +23,7 @@ |
23 | 23 |
# This version is bumped whenever enhancements are made
|
24 | 24 |
# to the `project.conf` format or the core element format.
|
25 | 25 |
#
|
26 |
-BST_FORMAT_VERSION = 13
|
|
26 |
+BST_FORMAT_VERSION = 14
|
|
27 | 27 |
|
28 | 28 |
|
29 | 29 |
# The base BuildStream artifact version
|
... | ... | @@ -98,6 +98,68 @@ relative filename to the elements they depend on here. |
98 | 98 |
See :ref:`format_dependencies` for more information on the dependency model.
|
99 | 99 |
|
100 | 100 |
|
101 |
+.. _format_build_depends:
|
|
102 |
+ |
|
103 |
+Build-Depends
|
|
104 |
+~~~~~~~~~~~~~
|
|
105 |
+ |
|
106 |
+.. code:: yaml
|
|
107 |
+ |
|
108 |
+ # Specify some build-dependencies
|
|
109 |
+ build-depends:
|
|
110 |
+ - element1.bst
|
|
111 |
+ - element2.bst
|
|
112 |
+ |
|
113 |
+Build dependencies between elements can be specified with the ``build-depends`` attribute.
|
|
114 |
+The above code snippet is equivalent to:
|
|
115 |
+ |
|
116 |
+.. code:: yaml
|
|
117 |
+ |
|
118 |
+ # Specify some build-dependencies
|
|
119 |
+ depends:
|
|
120 |
+ - filename: element1.bst
|
|
121 |
+ type: build
|
|
122 |
+ - filename: element2.bst
|
|
123 |
+ type: build
|
|
124 |
+ |
|
125 |
+See :ref:`format_dependencies` for more information on the dependency model.
|
|
126 |
+ |
|
127 |
+.. note::
|
|
128 |
+ |
|
129 |
+ The ``build-depends`` configuration is available since :ref:`format version 14 <project_format_version>`
|
|
130 |
+ |
|
131 |
+ |
|
132 |
+.. _format_runtime_depends:
|
|
133 |
+ |
|
134 |
+Runtime-Depends
|
|
135 |
+~~~~~~~~~~~~~~~
|
|
136 |
+ |
|
137 |
+.. code:: yaml
|
|
138 |
+ |
|
139 |
+ # Specify some runtime-dependencies
|
|
140 |
+ runtime-depends:
|
|
141 |
+ - element1.bst
|
|
142 |
+ - element2.bst
|
|
143 |
+ |
|
144 |
+Runtime dependencies between elements can be specified with the ``runtime-depends`` attribute.
|
|
145 |
+The above code snippet is equivalent to:
|
|
146 |
+ |
|
147 |
+.. code:: yaml
|
|
148 |
+ |
|
149 |
+ # Specify some runtime-dependencies
|
|
150 |
+ depends:
|
|
151 |
+ - filename: element1.bst
|
|
152 |
+ type: runtime
|
|
153 |
+ - filename: element2.bst
|
|
154 |
+ type: runtime
|
|
155 |
+ |
|
156 |
+See :ref:`format_dependencies` for more information on the dependency model.
|
|
157 |
+ |
|
158 |
+.. note::
|
|
159 |
+ |
|
160 |
+ The ``runtime-depends`` configuration is available since :ref:`format version 14 <project_format_version>`
|
|
161 |
+ |
|
162 |
+ |
|
101 | 163 |
.. _format_sources:
|
102 | 164 |
|
103 | 165 |
Sources
|
... | ... | @@ -276,8 +338,8 @@ attributes are suitable. |
276 | 338 |
|
277 | 339 |
.. note::
|
278 | 340 |
|
279 |
- Note the order in which element dependencies are declared in the ``depends``
|
|
280 |
- list is not meaningful.
|
|
341 |
+ Note the order in which element dependencies are declared in the ``depends``,
|
|
342 |
+ ``build-depends`` and ``runtime-depends`` lists are not meaningful.
|
|
281 | 343 |
|
282 | 344 |
Dependency dictionary:
|
283 | 345 |
|
... | ... | @@ -299,6 +361,8 @@ Attributes: |
299 | 361 |
* ``type``
|
300 | 362 |
|
301 | 363 |
This attribute is used to express the :ref:`dependency type <format_dependencies_types>`.
|
364 |
+ This field is not permitted in :ref:`Build-Depends <format_build_depends>` or
|
|
365 |
+ :ref:`Runtime-Depends <format_runtime_depends>`.
|
|
302 | 366 |
|
303 | 367 |
* ``junction``
|
304 | 368 |
|
... | ... | @@ -3,6 +3,7 @@ import pytest |
3 | 3 |
|
4 | 4 |
from buildstream._exceptions import LoadError, LoadErrorReason
|
5 | 5 |
from buildstream._loader import Loader, MetaElement
|
6 |
+from tests.testutils import cli
|
|
6 | 7 |
from . import make_loader
|
7 | 8 |
|
8 | 9 |
DATA_DIR = os.path.join(
|
... | ... | @@ -27,7 +28,7 @@ def test_two_files(datafiles): |
27 | 28 |
assert(len(element.dependencies) == 1)
|
28 | 29 |
firstdep = element.dependencies[0]
|
29 | 30 |
assert(isinstance(firstdep, MetaElement))
|
30 |
- assert(firstdep.kind == 'thefirstdep')
|
|
31 |
+ assert(firstdep.kind == 'manual')
|
|
31 | 32 |
|
32 | 33 |
|
33 | 34 |
@pytest.mark.datafiles(DATA_DIR)
|
... | ... | @@ -47,7 +48,7 @@ def test_shared_dependency(datafiles): |
47 | 48 |
#
|
48 | 49 |
firstdep = element.dependencies[0]
|
49 | 50 |
assert(isinstance(firstdep, MetaElement))
|
50 |
- assert(firstdep.kind == 'thefirstdep')
|
|
51 |
+ assert(firstdep.kind == 'manual')
|
|
51 | 52 |
assert(len(firstdep.dependencies) == 0)
|
52 | 53 |
|
53 | 54 |
# The second specified dependency is 'shareddep'
|
... | ... | @@ -86,7 +87,7 @@ def test_dependency_dict(datafiles): |
86 | 87 |
assert(len(element.dependencies) == 1)
|
87 | 88 |
firstdep = element.dependencies[0]
|
88 | 89 |
assert(isinstance(firstdep, MetaElement))
|
89 |
- assert(firstdep.kind == 'thefirstdep')
|
|
90 |
+ assert(firstdep.kind == 'manual')
|
|
90 | 91 |
|
91 | 92 |
|
92 | 93 |
@pytest.mark.datafiles(DATA_DIR)
|
... | ... | @@ -186,3 +187,49 @@ def test_all_dependency(datafiles): |
186 | 187 |
assert(isinstance(firstdep, MetaElement))
|
187 | 188 |
firstbuilddep = element.build_dependencies[0]
|
188 | 189 |
assert(firstdep == firstbuilddep)
|
190 |
+ |
|
191 |
+ |
|
192 |
+@pytest.mark.datafiles(DATA_DIR)
|
|
193 |
+def test_list_build_dependency(cli, datafiles):
|
|
194 |
+ project = str(datafiles)
|
|
195 |
+ |
|
196 |
+ # Check that the pipeline includes the build dependency
|
|
197 |
+ deps = cli.get_pipeline(project, ['elements/builddep-list.bst'], scope="build")
|
|
198 |
+ assert "elements/firstdep.bst" in deps
|
|
199 |
+ |
|
200 |
+ |
|
201 |
+@pytest.mark.datafiles(DATA_DIR)
|
|
202 |
+def test_list_runtime_dependency(cli, datafiles):
|
|
203 |
+ project = str(datafiles)
|
|
204 |
+ |
|
205 |
+ # Check that the pipeline includes the runtime dependency
|
|
206 |
+ deps = cli.get_pipeline(project, ['elements/runtimedep-list.bst'], scope="run")
|
|
207 |
+ assert "elements/firstdep.bst" in deps
|
|
208 |
+ |
|
209 |
+ |
|
210 |
+@pytest.mark.datafiles(DATA_DIR)
|
|
211 |
+def test_list_dependencies_combined(cli, datafiles):
|
|
212 |
+ project = str(datafiles)
|
|
213 |
+ |
|
214 |
+ # Check that runtime deps get combined
|
|
215 |
+ rundeps = cli.get_pipeline(project, ['elements/list-combine.bst'], scope="run")
|
|
216 |
+ assert "elements/firstdep.bst" not in rundeps
|
|
217 |
+ assert "elements/seconddep.bst" in rundeps
|
|
218 |
+ assert "elements/thirddep.bst" in rundeps
|
|
219 |
+ |
|
220 |
+ # Check that build deps get combined
|
|
221 |
+ builddeps = cli.get_pipeline(project, ['elements/list-combine.bst'], scope="build")
|
|
222 |
+ assert "elements/firstdep.bst" in builddeps
|
|
223 |
+ assert "elements/seconddep.bst" not in builddeps
|
|
224 |
+ assert "elements/thirddep.bst" in builddeps
|
|
225 |
+ |
|
226 |
+ |
|
227 |
+@pytest.mark.datafiles(DATA_DIR)
|
|
228 |
+def test_list_overlap(cli, datafiles):
|
|
229 |
+ project = str(datafiles)
|
|
230 |
+ |
|
231 |
+ # Check that dependencies get merged
|
|
232 |
+ rundeps = cli.get_pipeline(project, ['elements/list-overlap.bst'], scope="run")
|
|
233 |
+ assert "elements/firstdep.bst" in rundeps
|
|
234 |
+ builddeps = cli.get_pipeline(project, ['elements/list-overlap.bst'], scope="build")
|
|
235 |
+ assert "elements/firstdep.bst" in builddeps
|
1 |
+kind: stack
|
|
2 |
+description: This element has a build-only dependency specified via build-depends
|
|
3 |
+build-depends:
|
|
4 |
+ - elements/firstdep.bst
|
1 |
-kind: thefirstdep
|
|
1 |
+kind: manual
|
|
2 | 2 |
description: This is the first dependency
|
1 |
+kind: stack
|
|
2 |
+description: This element depends on three elements in different ways
|
|
3 |
+build-depends:
|
|
4 |
+- elements/firstdep.bst
|
|
5 |
+runtime-depends:
|
|
6 |
+- elements/seconddep.bst
|
|
7 |
+depends:
|
|
8 |
+- elements/thirddep.bst
|
1 |
+kind: stack
|
|
2 |
+description: This element depends on two elements in different ways
|
|
3 |
+build-depends:
|
|
4 |
+- elements/firstdep.bst
|
|
5 |
+depends:
|
|
6 |
+- filename: elements/firstdep.bst
|
|
7 |
+ type: runtime
|
1 |
+kind: stack
|
|
2 |
+description: This element has a runtime-only dependency
|
|
3 |
+runtime-depends:
|
|
4 |
+ - elements/firstdep.bst
|
1 |
+kind: manual
|
|
2 |
+description: This is the second dependency
|
1 |
+kind: manual
|
|
2 |
+description: This is the third dependency
|