Sorry a little late to review this. It looks like these are container based
builds. If that is the case we should handle this by updating the container.
The whole point of the container is to get the requirements all included in the
container. @Chris Fernald<mailto:chfer...@microsoft.com> can you share details.
Thanks
Sean
________________________________
From: devel@edk2.groups.io <devel@edk2.groups.io> on behalf of Michael D Kinney
<michael.d.kin...@intel.com>
Sent: Saturday, April 22, 2023 8:15:56 PM
To: Guo, Gua <gua....@intel.com>; devel@edk2.groups.io <devel@edk2.groups.io>
Cc: Sean Brogan <sean.bro...@microsoft.com>; Michael Kubacki
<mikub...@linux.microsoft.com>; Oliver Steffen <ostef...@redhat.com>; Kinney,
Michael D <michael.d.kin...@intel.com>
Subject: Re: [edk2-devel] [PATCH v2 2/2] .azurepipelines: Install code coverage
tool
Reviewed-by: Michael D Kinney <michael.d.kin...@intel.com>
> -----Original Message-----
> From: Guo, Gua <gua....@intel.com>
> Sent: Saturday, April 22, 2023 6:22 PM
> To: devel@edk2.groups.io
> Cc: Guo, Gua <gua....@intel.com>; Kinney, Michael D
> <michael.d.kin...@intel.com>; Sean Brogan <sean.bro...@microsoft.com>;
> Michael Kubacki <mikub...@linux.microsoft.com>; Oliver Steffen
> <ostef...@redhat.com>
> Subject: [PATCH v2 2/2] .azurepipelines: Install code coverage tool
>
> From: Gua Guo <gua....@intel.com>
>
> Azure should install code coverage tool (lcov), it didn't
> exist on Fedora and Ubuntu by default.
>
> Cc: Michael D Kinney <michael.d.kin...@intel.com>
> Cc: Sean Brogan <sean.bro...@microsoft.com>
> Cc: Michael Kubacki <mikub...@linux.microsoft.com>
> Cc: Oliver Steffen <ostef...@redhat.com>
> Signed-off-by: Gua Guo <gua....@intel.com>
> ---
> .azurepipelines/Ubuntu-GCC5.yml | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/.azurepipelines/Ubuntu-GCC5.yml b/.azurepipelines/Ubuntu-GCC5.yml
> index b9a3b851cf..d884256148 100644
> --- a/.azurepipelines/Ubuntu-GCC5.yml
> +++ b/.azurepipelines/Ubuntu-GCC5.yml
> @@ -24,3 +24,7 @@ jobs:
> container: ${{ variables.default_linux_image }}
>
> arch_list: "IA32,X64,ARM,AARCH64,RISCV64,LOONGARCH64"
>
> usePythonVersion: '' # use Python from the container image
>
> + extra_install_step:
>
> + - bash: sudo dnf install -y lcov
>
> + displayName: Install Code Coverage Tools
>
> + condition: and(gt(variables.pkg_count, 0), succeeded())
>
> --
> 2.39.2.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#103447): https://edk2.groups.io/g/devel/message/103447
Mute This Topic: https://groups.io/mt/98443857/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
