Re: [edk2-staging/EdkRepo] [PATCH V3 2/3] EdkRepo: Add edkrepo/common/workspace_maitenance/manifest_repos_maitenance

["Nate DeSimone" <nathaniel.l.desimone@intel.com>]

Hi Ashley,

There is a misspelling, "maitenance" should be "maintenance". Also, you need to update the filename at the top of manifest_repos_maitenance_humble.py Other than that, looks good.

Thanks,
Nate

On 4/22/20, 4:29 PM, "Desimone, Ashley E" <ashley.e.desimone@intel.com> wrote:

    Add a directory to edkrepo/common to store workspace maitenance
    functionatlity.
    Add edkrepo/common/workspace_matenance/humble directory to store
    related strings and included manifest_repos_maitenance_humble.py
    Add edkrepo/common/workspace_maitenance/manifest_repos_maitenance.py
    to support the maitenance of multiple manifest repositories.
    Added pull_single_manifest_repo() to manifest_repos_maitenance.py
    Updated setup.py to include the new directories.

    Signed-off-by: Ashley E Desimone <ashley.e.desimone@intel.com>
    Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
    Cc: Puja Pandya <puja.pandya@intel.com>
    Cc: Erik Bjorge <erik.c.bjorge@intel.com>
    Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
    Cc: Prince Agyeman <prince.agyeman@intel.com>
    ---
     .../humble/manifest_repos_maitenance_humble.py     | 23 +++++++++
     .../manifest_repos_maitenance.py                   | 59 ++++++++++++++++++++++
     setup.py                                           |  5 +-
     3 files changed, 85 insertions(+), 2 deletions(-)
     create mode 100644 edkrepo/common/workspace_maitenance/humble/manifest_repos_maitenance_humble.py
     create mode 100644 edkrepo/common/workspace_maitenance/manifest_repos_maitenance.py

    diff --git a/edkrepo/common/workspace_maitenance/humble/manifest_repos_maitenance_humble.py b/edkrepo/common/workspace_maitenance/humble/manifest_repos_maitenance_humble.py
    new file mode 100644
    index 0000000..ee7b4bb
    --- /dev/null
    +++ b/edkrepo/common/workspace_maitenance/humble/manifest_repos_maitenance_humble.py
    @@ -0,0 +1,23 @@
    +#!/usr/bin/env python3
    +#
    +## @file
    +# manifest_repos_mgmt_humble.py
    +#
    +# Copyright (c) 2017- 2020, Intel Corporation. All rights reserved.<BR>
    +# SPDX-License-Identifier: BSD-2-Clause-Patent
    +#
    +
    +''' Contains user facing strings for manifest_repos_mgmt.py '''
    +
    +from colorama import Fore
    +from colorama import Style
    +
    +CLONE_SINGLE_MAN_REPO = 'Cloning global manifest repository to: {} from: {}'
    +SYNC_SINGLE_MAN_REPO = 'Syncing the global manifest repository: {}'
    +SINGLE_MAN_REPO_DIRTY = ('Uncommited changes present in the global manifest '
    +                         'repository: {} Resolve these changes and attempt your'
    +                         ' operation again.')
    +SINGLE_MAN_REPO_NOT_CFG_BRANCH = ('The current active branch, {}, is not the '
    +                                  'specified branch for global manifst repository: {}')
    +SINGLE_MAN_REPO_CHECKOUT_CFG_BRANCH = 'Checking out the specified branch: {} prior to syncing'
    +SINGLE_MAN_REPO_MOVED = '{}{}WARNING:{}{} The global manifest repository has moved. Backing up previous global manifest repository to: {{}}{}\n'.format(Style.BRIGHT, Fore.RED, Style.RESET_ALL, Fore.RED, Style.RESET_ALL)
    \ No newline at end of file
    diff --git a/edkrepo/common/workspace_maitenance/manifest_repos_maitenance.py b/edkrepo/common/workspace_maitenance/manifest_repos_maitenance.py
    new file mode 100644
    index 0000000..c70a413
    --- /dev/null
    +++ b/edkrepo/common/workspace_maitenance/manifest_repos_maitenance.py
    @@ -0,0 +1,59 @@
    +#!/usr/bin/env python3
    +#
    +## @file
    +# manifest_repos_mgmt.py
    +#
    +# Copyright (c) 2017- 2020, Intel Corporation. All rights reserved.<BR>
    +# SPDX-License-Identifier: BSD-2-Clause-Patent
    +#
    +
    +import os
    +import traceback
    +import shutil
    +
    +import git
    +from git import Repo
    +
    +import edkrepo.config.config_factory as cfg
    +from edkrepo.common.edkrepo_exception import EdkrepoUncommitedChangesException
    +from edkrepo.common.progress_handler import GitProgressHandler
    +import edkrepo.common.workspace_maitenance.humble.manifest_repos_maitenance_humble as humble
    +from edkrepo.common.workspace_maitenance.workspace_maitenance import generate_name_for_obsolete_backup
    +
    +
    +def pull_single_manifest_repo(url, branch, local_path, reset_hard=False):
    +    '''
    +    Clones or syncs a single global manifest repository as defined in either
    +    the edkrepo.cfg or the edkrepo_user.cfg
    +    '''
    +    # If a relative path is used join to the edkrepo global data directory path
    +    if not os.path.isabs(local_path):
    +        local_path = os.path.join(cfg.get_edkrepo_global_data_directory(), local_path)
    +    # Clone the repository if it does not exist locally
    +    if not os.path.exists(local_path):
    +        print(humble.CLONE_SINGLE_MAN_REPO.format(local_path, url))
    +        repo = Repo.clone_from(url, local_path, progress=GitProgressHandler(), branch=branch)
    +    # Sync the repository if it exists locally
    +    else:
    +        repo = Repo(local_path)
    +        if url in repo.remotes['origin'].urls:
    +            if repo.is_dirty(untracked_files=True) and not reset_hard:
    +                raise EdkrepoUncommitedChangesException(humble.SINGLE_MAN_REPO_DIRTY.format(local_path))
    +            elif repo.is_dirty(untracked_files=True) and reset_hard:
    +                repo.git.reset('--hard')
    +            print(humble.SYNC_SINGLE_MAN_REPO.format(local_path))
    +            if repo.active_branch.name != branch:
    +                print(humble.SINGLE_MAN_REPO_NOT_CFG_BRANCH.format(repo.active_branch.name, local_path))
    +                print(humble.SINGLE_MAN_REPO_CHECKOUT_CFG_BRANCH.format(branch))
    +                repo.git.checkout(branch)
    +            repo.remotes.origin.pull()
    +        # If the URL specified for this manifest repo has moved back up the existing
    +        # local copy and clone the new repository
    +        else:
    +            new_path = generate_name_for_obsolete_backup(local_path)
    +            new_path = os.path.join(os.path.dirname(local_path), new_path)
    +            print(humble.SINGLE_MAN_REPO_MOVED.format(new_path))
    +            shutil.move(local_path, new_path)
    +            print (humble.CLONE_SINGLE_MAN_REPO.format(local_path, url))
    +            repo = Repo.clone_from(url, local_path, progress=GitProgressHandler(), branch=branch)
    +
    diff --git a/setup.py b/setup.py
    index 06a382b..f8bb0fc 100755
    --- a/setup.py
    +++ b/setup.py
    @@ -12,8 +12,9 @@ setup(name='edkrepo',
           version='2.0.0',
           description='The edkrepo tools',
           packages=['edkrepo', 'edkrepo.commands', 'edkrepo.commands.arguments', 'edkrepo.commands.humble',
    -                'edkrepo.git_automation', 'edkrepo.common', 'edkrepo.config', 'edkrepo.config.humble',
    -                'edkrepo_manifest_parser', 'project_utils'],
    +                'edkrepo.git_automation', 'edkrepo.common', 'edkrepo.common.workspace_maitenance',
    +                'edkrepo.common.workspace_maitenance.humble', 'edkrepo.config', 'edkrepo.config.humble',
    +                'edkrepo_manifest_parser', 'project_utils'],
           package_data={
              },
           include_package_data=True,
    -- 
    2.16.2.windows.1