Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42239 )
Change subject: vendorcode/intel: Add edk2-stable202005 support ......................................................................
Patch Set 3: Code-Review+1
(2 comments)
Patch Set 2: Code-Review-2
Hold on, there should be a discussion to configure this as a git submodule.
So we just had the leadership meeting and talked about this, and looks like having edk2 as a submodule is going to give quite a bit of trouble... Even taking an approach like we do for payloads would be troublesome for Jenkins builds, which run concurrently.
In short, even though I would like to use the edk2 repo directly, doing so would result in several technical problems. So, I believe it's just not worth the hassle.
https://review.coreboot.org/c/coreboot/+/42239/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/42239/3//COMMIT_MSG@14 PS3, Line 14: includ includ*e*
https://review.coreboot.org/c/coreboot/+/42239/3//COMMIT_MSG@17 PS3, Line 17: safe guarding safeguarding