Why not move to edk2 repo? Thanks, Hot
-----Original Message----- From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Leif Lindholm Sent: Friday, October 02, 2020 7:26 To: Desimone, Nathaniel L <nathaniel.l.desim...@intel.com> Cc: r...@edk2.groups.io; devel@edk2.groups.io; Laszlo Ersek <ler...@redhat.com>; Ard Biesheuvel <ard.biesheu...@arm.com>; Kirkendall, Garrett <garrett.kirkend...@amd.com>; Lendacky, Thomas <thomas.lenda...@amd.com>; thomas.abra...@arm.com; frank....@dell.com; Jim Dailey <jim.dai...@dell.com>; Abner Chang <abner.ch...@hpe.com>; Wang, Sunny (HPS SW) <sunnyw...@hpe.com>; Daniel Schaefer <daniel.schae...@hpe.com>; Liming Gao <gaolim...@byosoft.com.cn>; Tim Lewis <tim.le...@insyde.com>; Kevin@Insyde <kevin.da...@insyde.com>; Felix Polyudov <fel...@ami.com>; Kinney, Michael D <michael.d.kin...@intel.com>; Oram, Isaac W <isaac.w.o...@intel.com>; Ni, Ray <ray...@intel.com> Subject: Re: [edk2-devel] [edk2-rfc] [RFC] Request to move MinPlatformPkg out of the Intel folder On Thu, Oct 01, 2020 at 23:20:01 +0000, Desimone, Nathaniel L wrote: > > One thing I think is needed for this to move out of Platform/Intel > > would be a ReadMe.rst describing the intended real-world use. > > Some of this lives in Platform/Intel/Readme.md today. > > Could we start with a set moving this text into a separate file > > under MinPlatformPkg and rework that into something that stands on its own? > > Yeah agreed, given that the feedback on this RFC has been pretty > positive thus far, I'm planning on making a patch series for this. > I'll make these changes part of said patch series. Sounds good. > > If I was to bikeshed, I'd probably suggest > > Platform/TianoCore/MinPlatformPkg. Oops, guess I did. > > Seems a little superfluous to me... isn't edk2-platforms part of > TianoCore by definition? 😊 It is. It's just the pattern for everything else in edk2-platforms (apart from OptionRomPkg, which was just transplanted from edk2) exists in the form of <section>/$VENDOR/. Platform/MinPlatformPkg would be the only bare package in Platform/. > > I think we should see this as a separate step, regardless. > > > > I do think we should be able to do better on sharing some of this > > platform image tooling cross-architecture (even moreso now Risc-V is > > involved), but I would prefer for us to spend the effort of > > identifying the intersections upfront before we start uploading > > tools to places where they look generic and may confuse people. > > > > Maybe we should raise this part as a BZ? > > Yup, sounds like good next steps to me. I've filed the BZ: > https://bugzilla.tianocore.org/show_bug.cgi?id=2992 Top man. Best Regards, Leif -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#65829): https://edk2.groups.io/g/devel/message/65829 Mute This Topic: https://groups.io/mt/77252394/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
