Re: [edk2-devel][PATCH V1 1/1] BaseTools: Generate compile information in build report

["Michael D Kinney" <michael.d.kinney@intel.com>]

Hi Guillermo,

Can you please look at Ards PR and make sure his fix is included in your PR.

Also, please work with Christine and Bob to see what is going on with the
Code Coverage check.  We do want it to be easy for all community members to
submit change requests.  We may need support from the edk2-bastools maintainers
to help with CI issues and help with changes to address.

Thanks,

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ard Biesheuvel
> Sent: Monday, February 27, 2023 10:58 AM
> To: devel@edk2.groups.io; Palomino Sosa, Guillermo A <guillermo.a.palomino.sosa@intel.com>
> Cc: Ni, Ray <ray.ni@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Chen, Christine <yuwei.chen@intel.com>; Feng,
> Bob C <bob.c.feng@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Oram, Isaac W <isaac.w.oram@intel.com>; Sean Brogan
> <sean.brogan@microsoft.com>
> Subject: Re: [edk2-devel][PATCH V1 1/1] BaseTools: Generate compile information in build report
> 
> On Mon, 27 Feb 2023 at 18:40, Guillermo Antonio Palomino Sosa
> <guillermo.a.palomino.sosa@intel.com> wrote:
> >
> > Hi. I have submitted a pull request to edk2-basetools repository: https://github.com/tianocore/edk2-basetools/pull/88
> > This is the feature request for it: https://github.com/tianocore/edk2-basetools/issues/87
> > I'm also attaching the patch here: (0001-BaseTools-Generate-compile-information-in-build-repo.patch)
> >
> > On a side note, seems like tip of edk2-basetools is broken due this commit that makes direct import of Common package:
> > https://github.com/tianocore/edk2-basetools/commit/8e6018d3ea4c1aae7185f589d129cea14a5d89fd
> > edk2-basetools\edk2basetools\GenFds\SubTypeGuidSection.py:
> > import Common.LongFilePathOs as os
> >
> >
> 
> I sent a fix and a PR for this about a month ago:
> 
> https://github.com/tianocore/edk2-basetools/pull/86
> 
> but CodeCov seems to take issue with it, for reasons I don't
> understand (and I can't be bothered to spend more time on yet another
> CI automation pass that adds to my workload rather than reduce it)
> 
> 
> 
>