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

[Michael D Kinney]

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 at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Ard Biesheuvel
> Sent: Monday, February 27, 2023 10:58 AM
> To: devel at edk2.groups.io; Palomino Sosa, Guillermo A <guillermo.a.palomino.sosa at intel.com>
> Cc: Ni, Ray <ray.ni at intel.com>; Kinney, Michael D <michael.d.kinney at intel.com>; Chen, Christine <yuwei.chen at intel.com>; Feng,
> Bob C <bob.c.feng at intel.com>; Gao, Liming <gaoliming at byosoft.com.cn>; Oram, Isaac W <isaac.w.oram at intel.com>; Sean Brogan
> <sean.brogan at 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 at 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)
> 
> 
> 
> 



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