GNOME.org
 

[Notes] [Git][BuildStream/buildstream][valentindavid/448-autocompletion-broken-defaults] 9 commits: Fix crash when --debug is passed

Title: GitLab

Valentin David pushed to branch valentindavid/448-autocompletion-broken-defaults at BuildStream / buildstream

Commits:

12 changed files:

Changes:

  • buildstream/_artifactcache/cascache.py
    ... ... @@ -845,6 +845,9 @@ class _CASRemote():
    845 845
    846 846
    847 847 
     def _grouper(iterable, n):
    848 
    -    # pylint: disable=stop-iteration-return
    849 848 
         while True:
    850 
    -        yield itertools.chain([next(iterable)], itertools.islice(iterable, n - 1))
    849 
    +        try:
    850 
    +            current = next(iterable)
    851 
    +        except StopIteration:
    852 
    +            return
    853 
    +        yield itertools.chain([current], itertools.islice(iterable, n - 1))

  • buildstream/_frontend/cli.py
    ... ... @@ -827,4 +827,5 @@ def source_bundle(app, element, force, directory,
    827 827 
             app.stream.source_bundle(element, directory,
    828 828 
                                      track_first=track_,
    829 829 
                                      force=force,
    830 
    -                                 compression=compression)
    830 
    +                                 compression=compression,
    831 
    +                                 except_targets=except_)

  • buildstream/_frontend/complete.py
    ... ... @@ -68,9 +68,10 @@ def complete_path(path_type, incomplete, base_directory='.'):
    68 68 
             # If there was nothing on the left of the last separator,
    69 69 
             # we are completing files in the filesystem root
    70 70 
             base_path = os.path.join(base_directory, base_path)
    71 
    -
    72 
    -    elif os.path.isdir(incomplete):
    73 
    -        base_path = incomplete
    71 
    +    else:
    72 
    +        incomplete_base_path = os.path.join(base_directory, incomplete)
    73 
    +        if os.path.isdir(incomplete_base_path):
    74 
    +            base_path = incomplete_base_path
    74 75
    75 76 
         try:
    76 77 
             if base_path:

  • buildstream/_stream.py
    ... ... @@ -478,7 +478,7 @@ class Stream():
    478 478
    479 479 
             # Check for workspace config
    480 480 
             workspace = workspaces.get_workspace(target._get_full_name())
    481 
    -        if workspace:
    481 
    +        if workspace and not force:
    482 482 
                 raise StreamError("Workspace '{}' is already defined at: {}"
    483 483 
                                   .format(target.name, workspace.path))
    484 484
    ... ... @@ -497,6 +497,10 @@ class Stream():
    497 497 
                                   "fetch the latest version of the " +
    498 498 
                                   "source.")
    499 499
    500 
    +        if workspace:
    501 
    +            workspaces.delete_workspace(target._get_full_name())
    502 
    +            workspaces.save_config()
    503 
    +            shutil.rmtree(directory)
    500 504 
             try:
    501 505 
                 os.makedirs(directory, exist_ok=True)
    502 506 
             except OSError as e:
    ... ... @@ -654,7 +658,8 @@ class Stream():
    654 658 
         def source_bundle(self, target, directory, *,
    655 659 
                           track_first=False,
    656 660 
                           force=False,
    657 
    -                      compression="gz"):
    661 
    +                      compression="gz",
    662 
    +                      except_targets=()):
    658 663
    659 664 
             if track_first:
    660 665 
                 track_targets = (target,)
    ... ... @@ -663,6 +668,7 @@ class Stream():
    663 668
    664 669 
             elements, track_elements = self._load((target,), track_targets,
    665 670 
                                                   selection=PipelineSelection.ALL,
    671 
    +                                              except_targets=except_targets,
    666 672 
                                                   track_selection=PipelineSelection.ALL,
    667 673 
                                                   fetch_subprojects=True)
    668 674

  • buildstream/element.py
    ... ... @@ -194,6 +194,9 @@ class Element(Plugin):
    194 194
    195 195 
         def __init__(self, context, project, artifacts, meta, plugin_conf):
    196 196
    197 
    +        self.__cache_key_dict = None            # Dict for cache key calculation
    198 
    +        self.__cache_key = None                 # Our cached cache key
    199 
    +
    197 200 
             super().__init__(meta.name, context, project, meta.provenance, "element")
    198 201
    199 202 
             self.__is_junction = meta.kind == "junction"
    ... ... @@ -212,8 +215,6 @@ class Element(Plugin):
    212 215 
             self.__runtime_dependencies = []        # Direct runtime dependency Elements
    213 216 
             self.__build_dependencies = []          # Direct build dependency Elements
    214 217 
             self.__sources = []                     # List of Sources
    215 
    -        self.__cache_key_dict = None            # Dict for cache key calculation
    216 
    -        self.__cache_key = None                 # Our cached cache key
    217 218 
             self.__weak_cache_key = None            # Our cached weak cache key
    218 219 
             self.__strict_cache_key = None          # Our cached cache key for strict builds
    219 220 
             self.__artifacts = artifacts            # Artifact cache

  • tests/completions/completions.py
    ... ... @@ -212,6 +212,10 @@ def test_option_directory(datafiles, cli, cmd, word_idx, expected, subdir):
    212 212 
         # Also try multi arguments together
    213 213 
         ('no-element-path', 'bst --directory ../ checkout t ', 4, ['target.bst '], 'files'),
    214 214 
         ('no-element-path', 'bst --directory ../ checkout target.bst ', 5, ['bin-files/', 'dev-files/'], 'files'),
    215 
    +
    216 
    +    # When element-path have sub-folders
    217 
    +    ('sub-folders', 'bst show base', 2, ['base/wanted.bst '], None),
    218 
    +    ('sub-folders', 'bst show base/', 2, ['base/wanted.bst '], None),
    215 219 
     ])
    216 220 
     def test_argument_element(datafiles, cli, project, cmd, word_idx, expected, subdir):
    217 221 
         cwd = os.path.join(str(datafiles), project)

  • tests/completions/sub-folders/base/unwanted.bst
    1 
    +kind: autotools
    2 
    +description: |
    3 
    +
    4 
    +    Not auto-completed element

  • tests/completions/sub-folders/elements/base.bst
    1 
    +kind: stack
    2 
    +description: Base stack
    3 
    +
    4 
    +depends:
    5 
    +- base/wanted.bst

  • tests/completions/sub-folders/elements/base/wanted.bst
    1 
    +kind: autotools
    2 
    +description: |
    3 
    +
    4 
    +    Auto-completed element

  • tests/completions/sub-folders/elements/hello.bst
    1 
    +kind: autotools
    2 
    +description: |
    3 
    +
    4 
    +    Hello world

  • tests/completions/sub-folders/project.conf
    1 
    +# Project config for frontend build test
    2 
    +name: test
    3 
    +
    4 
    +element-path: elements

  • tests/frontend/workspace.py
    ... ... @@ -123,6 +123,58 @@ def test_open_force(cli, tmpdir, datafiles, kind):
    123 123 
         result.assert_success()
    124 124
    125 125
    126 
    +@pytest.mark.datafiles(DATA_DIR)
    127 
    +@pytest.mark.parametrize("kind", repo_kinds)
    128 
    +def test_open_force_open(cli, tmpdir, datafiles, kind):
    129 
    +    element_name, project, workspace = open_workspace(cli, tmpdir, datafiles, kind, False)
    130 
    +
    131 
    +    # Assert the workspace dir exists
    132 
    +    assert os.path.exists(workspace)
    133 
    +
    134 
    +    # Now open the workspace again with --force, this should happily succeed
    135 
    +    result = cli.run(project=project, args=[
    136 
    +        'workspace', 'open', '--force', element_name, workspace
    137 
    +    ])
    138 
    +    result.assert_success()
    139 
    +
    140 
    +
    141 
    +@pytest.mark.datafiles(DATA_DIR)
    142 
    +@pytest.mark.parametrize("kind", repo_kinds)
    143 
    +def test_open_force_different_workspace(cli, tmpdir, datafiles, kind):
    144 
    +    element_name, project, workspace = open_workspace(cli, tmpdir, datafiles, kind, False, "-alpha")
    145 
    +
    146 
    +    # Assert the workspace dir exists
    147 
    +    assert os.path.exists(workspace)
    148 
    +
    149 
    +    hello_path = os.path.join(workspace, 'usr', 'bin', 'hello')
    150 
    +    hello1_path = os.path.join(workspace, 'usr', 'bin', 'hello1')
    151 
    +
    152 
    +    tmpdir = os.path.join(str(tmpdir), "-beta")
    153 
    +    shutil.move(hello_path, hello1_path)
    154 
    +    element_name2, project2, workspace2 = open_workspace(cli, tmpdir, datafiles, kind, False, "-beta")
    155 
    +
    156 
    +    # Assert the workspace dir exists
    157 
    +    assert os.path.exists(workspace2)
    158 
    +
    159 
    +    # Assert that workspace 1 contains the modified file
    160 
    +    assert os.path.exists(hello1_path)
    161 
    +
    162 
    +    # Assert that workspace 2 contains the unmodified file
    163 
    +    assert os.path.exists(os.path.join(workspace2, 'usr', 'bin', 'hello'))
    164 
    +
    165 
    +    # Now open the workspace again with --force, this should happily succeed
    166 
    +    result = cli.run(project=project, args=[
    167 
    +        'workspace', 'open', '--force', element_name2, workspace
    168 
    +    ])
    169 
    +
    170 
    +    # Assert that the file in workspace 1 has been replaced
    171 
    +    # With the file from workspace 2
    172 
    +    assert os.path.exists(hello_path)
    173 
    +    assert not os.path.exists(hello1_path)
    174 
    +
    175 
    +    result.assert_success()
    176 
    +
    177 
    +
    126 178 
     @pytest.mark.datafiles(DATA_DIR)
    127 179 
     @pytest.mark.parametrize("kind", repo_kinds)
    128 180 
     def test_close(cli, tmpdir, datafiles, kind):

    • —
    View it on GitLab.
    You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can adjust your notification settings.

    [Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]