This is an automated email from the ASF dual-hosted git repository. tvb pushed a commit to branch gokcen/add_no_fetch_flag in repository https://gitbox.apache.org/repos/asf/buildstream.git
commit e23c138225eb657a29ef6ee3d560a75aa12a7797 Author: Gökçen Nurlu <[email protected]> AuthorDate: Thu Oct 25 18:30:36 2018 +0100 Add no-fetch flag for 'bst pull' command --- buildstream/_frontend/cli.py | 6 ++++-- buildstream/_stream.py | 6 ++++-- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/buildstream/_frontend/cli.py b/buildstream/_frontend/cli.py index b1b4e03..eac304f 100644 --- a/buildstream/_frontend/cli.py +++ b/buildstream/_frontend/cli.py @@ -433,8 +433,10 @@ def track(app, elements, deps, except_, cross_junctions): help="The URL of the remote cache (defaults to the first configured cache)") @click.argument('elements', nargs=-1, type=click.Path(readable=False)) [email protected]('--no-fetch', 'no_fetch', default=False, is_flag=True, + help="Disable auto-fetching of junction(s)") @click.pass_obj -def pull(app, elements, deps, remote): +def pull(app, elements, deps, remote, no_fetch): """Pull a built artifact from the configured remote artifact cache. By default the artifact will be pulled one of the configured caches @@ -448,7 +450,7 @@ def pull(app, elements, deps, remote): all: All dependencies """ with app.initialized(session_name="Pull"): - app.stream.pull(elements, selection=deps, remote=remote) + app.stream.pull(elements, selection=deps, remote=remote, no_fetch=no_fetch) ################################################################## diff --git a/buildstream/_stream.py b/buildstream/_stream.py index 76f1d67..992b37a 100644 --- a/buildstream/_stream.py +++ b/buildstream/_stream.py @@ -289,13 +289,15 @@ class Stream(): # targets (list of str): Targets to pull # selection (PipelineSelection): The selection mode for the specified targets # remote (str): The URL of a specific remote server to pull from, or None + # no_fetch (bool): Flag to disable auto-fetch of junctions needed by `targets` # # If `remote` specified as None, then regular configuration will be used # to determine where to pull artifacts from. # def pull(self, targets, *, selection=PipelineSelection.NONE, - remote=None): + remote=None, + no_fetch): use_config = True if remote: @@ -305,7 +307,7 @@ class Stream(): selection=selection, use_artifact_config=use_config, artifact_remote_url=remote, - fetch_subprojects=True) + fetch_subprojects=not no_fetch) if not self._artifacts.has_fetch_remotes(): raise StreamError("No artifact caches available for pulling artifacts")
