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