WIP: Add explicit `--fetch` flag to bst commands

Description

This introduces a new flag --fetch that was mentioned in the mailing list.

Proposed changes

To put simply, this changes the following commands' behaviour so they'd not try to implicitly fetch junctions and elements (offline-first?).

• bst {pull,push,track,checkout} element(s) command

  • Before: Fetches the junction(s) if they are used and not cached yet.
  • After: Don't do junction fetch implicitly. In case one is needed, error out by saying need to fetch junction. --fetch is supplied to fetch_subprojects.

• bst source-bundle element command

  • Before: (Possibly track and) fetch implicitly.
  • After: Now we have 4 combinations in total (--fetch and --track)
    • Only --track: Attempt track elements, error out if there is a new ref (since no --fetch supplied, continue if there is nothing new.
    • Only --fetch: Fetch current refs for elements (including related junctions) - they might be cached already.
    • --fetch and --track: Previous behaviour of the command with --track, track for new refs if possible and fetch them.
    • No flags: error out if the current refs are not cached.

• bst {workspace-open,workspace-reset} element.bst .. We are interested in the cases where no_checkout is NOT used.

  • Before: Error out if junction(s) are required but not cached yet. Fetch for element is always done. Track before fetch can be requested using --track.
  • After: Now we have 4 combinations in total (--fetch and --track)
    • Only --track: Attempt track elements, error out if there is a new ref (since no --fetch supplied, continue if there is nothing new.
    • Only --fetch: Fetch current refs for elements (including related junctions) - they might be cached already.
    • --fetch and --track: Previous behaviour of the command with --track, track for new refs if possible and fetch them.
    • No flags: error out if the current refs are not cached.

You can notice that the original behaviour of workspace commands are almost equivalent to source-bundle, except the latter implicitly fetches junctions where the former complains and terminates.

This merge request, when approved, will close: N/A at the moment.

---

buildstream/_frontend/cli.py

@@ -395,8 +395,10 @@ def fetch(app, elements, deps, track_, except_, track_cross_junctions):
               help="Allow crossing junction boundaries")
 @click.argument('elements', nargs=-1,
                 type=click.Path(readable=False))
+@click.option('--fetch', 'fetch_', default=False, is_flag=True,
+              help="Enable auto-fetching of junction(s) automatically if elements has dependencies")
 @click.pass_obj
-def track(app, elements, deps, except_, cross_junctions):
+def track(app, elements, deps, except_, cross_junctions, fetch_):
     """Consults the specified tracking branches for new versions available
     to build and updates the project with any newly available references.
 
@@ -417,7 +419,8 @@ def track(app, elements, deps, except_, cross_junctions):
         app.stream.track(elements,
                          selection=deps,
                          except_targets=except_,
-                         cross_junctions=cross_junctions)
+                         cross_junctions=cross_junctions,
+                         fetch_subprojects=fetch_)
 
 
 ##################################################################