Re: [edk2-devel] [Patch] EmulatorPkg: Add CI build for SECURE_BOOT_ENABLE

2020-09-24 Thread Sean
I think you should add NOOPT target as well. NOOPT catches more compiler issues than debug/release given the change in optimization flags and it is used less by developers. Once you run a build you should also add the badge to the readme.rst (https://github.com/tianocore/edk2/blob/master/Read

[edk2-devel] [Patch] EmulatorPkg: Add CI build for SECURE_BOOT_ENABLE

2020-09-24 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2979 Add EmulatorPkg CI builds for SECURE_BOOT_ENABLE=TRUE for IA32/X64 and DEBUG/RELEASE. Label this these as FULL builds, so if additional build options are added in the future, they can be added to these FULL builds. Cc: Jordan Justen Cc: Andrew