tianocore / containers

Repository to maintain and manage edk2 containers

Other

23 stars 27 forks source link

Install lcov in all linux based image #70

Closed gguo11837463 closed 1 year ago

gguo11837463 commented 1 year ago

Description

For code coverage need to be enabled in azure pipeline, It requires for lcov install on all linux based machine.

Containers Affected

Fedora-35-test Fedora-35-dev Fedora-37-test Fedora-37-dev Ubuntu-20-test Ubuntu-20-dev Ubuntu-22-test Ubuntu-22-dev

Description

Please include a summary of the change and which issue is fixed or feature is added.

Issue #(issue)

Containers Affected

Please list the container images affected by this change. Including any containers based on the targeted dockerfile.

gguo11837463 commented 1 year ago

@cfernald if fedora 35 had installed lcov by default, I am not sure why had a lcov failure previously. Anyway let us check in first to check whether is useful to unlock my change. https://github.com/tianocore/edk2/blob/master/.devcontainer/devcontainer.json

Could you help merge it or I need to get other people approval?