回复: [edk2-devel] [Patch 1/1] UnitTestFrameworkPkg: Use TianoCore mirror of cmocka repository

[gaoliming]

Mike:
  I agree this change. Reviewed-by: Liming Gao <gaoliming at byosoft.com.cn>

Thanks
Liming
> -----邮件原件-----
> 发件人: devel at edk2.groups.io <devel at edk2.groups.io> 代表 Michael D
> Kinney
> 发送时间: 2021年4月7日 4:12
> 收件人: devel at edk2.groups.io
> 抄送: Sean Brogan <sean.brogan at microsoft.com>; Bret Barkelew
> <Bret.Barkelew at microsoft.com>; Liming Gao <gaoliming at byosoft.com.cn>;
> Andrew Fish <afish at apple.com>; Laszlo Ersek <lersek at redhat.com>; Leif
> Lindholm <leif at nuviainc.com>
> 主题: [edk2-devel] [Patch 1/1] UnitTestFrameworkPkg: Use TianoCore mirror
> of cmocka repository
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3301
> 
> The cmocka repository https://git.cryptomilk.org/projects/cmocka.git
> has gone down a few times in past year.  When it is down, it blocks
> EDK II CI.  A mirror of this repository has been created in the
> TianoCore organization at https://github.com/tianocore/edk2-cmocka.git
> and uses a GitHub Action to auto-sync changes from
> https://git.cryptomilk.org/projects/cmocka.git.
> 
> * Update .gitmodules to use https://github.com/tianocore/edk2-cmocka.git
>   instead of https://git.cryptomilk.org/projects/cmocka.git.
> 
> * Update README.rst to reference the COPYING file in
>   https://github.com/tianocore/edk2-cmocka.git with the cmocka license.
> 
> * Update Azure Pipelines YML files to remove a temporary workaround that
>   used an alternate GitHub mirror of cmocka.  With the workaround
> removed,
>   EDK II CI always uses the TianoCore mirror of cmocka.
> 
> Cc: Sean Brogan <sean.brogan at microsoft.com>
> Cc: Bret Barkelew <Bret.Barkelew at microsoft.com>
> Cc: Liming Gao <gaoliming at byosoft.com.cn>
> Cc: Andrew Fish <afish at apple.com>
> Cc: Laszlo Ersek <lersek at redhat.com>
> Cc: Leif Lindholm <leif at nuviainc.com>
> Signed-off-by: Michael D Kinney <michael.d.kinney at intel.com>
> ---
>  .azurepipelines/templates/platform-build-run-steps.yml | 3 ---
>  .azurepipelines/templates/pr-gate-steps.yml            | 3 ---
>  .gitmodules                                            | 2 +-
>  ReadMe.rst                                             | 2 +-
>  4 files changed, 2 insertions(+), 8 deletions(-)
> 
> diff --git a/.azurepipelines/templates/platform-build-run-steps.yml
> b/.azurepipelines/templates/platform-build-run-steps.yml
> index b712e2fafafb..97e7faa26682 100644
> --- a/.azurepipelines/templates/platform-build-run-steps.yml
> +++ b/.azurepipelines/templates/platform-build-run-steps.yml
> @@ -51,9 +51,6 @@ steps:
>  # Set default
>  - bash: echo "##vso[task.setvariable variable=pkg_count]${{ 1 }}"
> 
> -# Use altername cmocka repo
> -- bash: git config --global
> url.https://github.com/neverware-mirrors/cmocka.git.insteadOf
> https://git.cryptomilk.org/projects/cmocka.git
> -
>  # Fetch the target branch so that pr_eval can diff them.
>  # Seems like azure pipelines/github changed checkout process in nov 2020.
>  - script: git fetch origin $(System.PullRequest.targetBranch)
> diff --git a/.azurepipelines/templates/pr-gate-steps.yml
> b/.azurepipelines/templates/pr-gate-steps.yml
> index 28edb453bddc..70c19a462194 100644
> --- a/.azurepipelines/templates/pr-gate-steps.yml
> +++ b/.azurepipelines/templates/pr-gate-steps.yml
> @@ -31,9 +31,6 @@ steps:
>      echo "##vso[task.setvariable
> variable=pkgs_to_build]${{ parameters.build_pkgs }}"
>      echo "##vso[task.setvariable variable=pkg_count]${{ 1 }}"
> 
> -# Use altername cmocka repo
> -- bash: git config --global
> url.https://github.com/neverware-mirrors/cmocka.git.insteadOf
> https://git.cryptomilk.org/projects/cmocka.git
> -
>  # Fetch the target branch so that pr_eval can diff them.
>  # Seems like azure pipelines/github changed checkout process in nov 2020.
>  - script: git fetch origin $(System.PullRequest.targetBranch)
> diff --git a/.gitmodules b/.gitmodules
> index 0f06c09a29a1..b845c9ee3ff0 100644
> --- a/.gitmodules
> +++ b/.gitmodules
> @@ -6,7 +6,7 @@
>  	url = https://github.com/ucb-bar/berkeley-softfloat-3.git
>  [submodule "UnitTestFrameworkPkg/Library/CmockaLib/cmocka"]
>  	path = UnitTestFrameworkPkg/Library/CmockaLib/cmocka
> -	url = https://git.cryptomilk.org/projects/cmocka.git
> +	url = https://github.com/tianocore/edk2-cmocka.git
>  [submodule "MdeModulePkg/Universal/RegularExpressionDxe/oniguruma"]
>  	path = MdeModulePkg/Universal/RegularExpressionDxe/oniguruma
>  	url = https://github.com/kkos/oniguruma
> diff --git a/ReadMe.rst b/ReadMe.rst
> index b754378646db..8f5db11281bf 100644
> --- a/ReadMe.rst
> +++ b/ReadMe.rst
> @@ -92,7 +92,7 @@ that are covered by additional licenses.
>  -  `CryptoPkg/Library/OpensslLib/openssl
> <https://github.com/openssl/openssl/blob/e2e09d9fba1187f8d6aafaa34d417
> 2f56f1ffb72/LICENSE>`__
>  -  `MdeModulePkg/Library/BrotliCustomDecompressLib/brotli
> <https://github.com/google/brotli/blob/666c3280cc11dc433c303d79a83d4ff
> bdd12cc8d/LICENSE>`__
>  -  `MdeModulePkg/Universal/RegularExpressionDxe/oniguruma
> <https://github.com/kkos/oniguruma/blob/abfc8ff81df4067f309032467785e
> 06975678f0d/COPYING>`__
> --  `UnitTestFrameworkPkg/Library/CmockaLib/cmocka
> <https://git.cryptomilk.org/projects/cmocka.git/tree/COPYING?h=cmocka-1.
> 1.5&id=f5e2cd77c88d9f792562888d2b70c5a396bfbf7a>`__
> +-  `UnitTestFrameworkPkg/Library/CmockaLib/cmocka
> <https://github.com/tianocore/edk2-cmocka/blob/f5e2cd77c88d9f79256288
> 8d2b70c5a396bfbf7a/COPYING>`__
>  -  `RedfishPkg/Library/JsonLib/jansson
> <https://github.com/akheron/jansson/blob/2882ead5bb90cf12a01b07b2c23
> 61e24960fae02/LICENSE>`__
> 
>  The EDK II Project is composed of packages. The maintainers for each
> package
> --
> 2.31.1.windows.1
> 
> 
> 
> 
> 





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#73751): https://edk2.groups.io/g/devel/message/73751
Mute This Topic: https://groups.io/mt/81906226/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-