On 2015-06-30 02:01:24, Star Zeng wrote: > AuthVariableLib and TpmMeasurementLib library classes are now linked with > MdeModulePkg/Universal/Variable/RuntimeDxe/VariableRuntimeDxe.inf > to optionally support secure variables. > > For Nt32Pkg, > link AuthVariableLib and DxeTpmMeasurementLib in SecurityPkg > when SECURE_BOOT_ENABLE = TRUE, > and link AuthVariableLibNull and TpmMeasurementLibNull in MdeModulePkg > when SECURE_BOOT_ENABLE = FALSE. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Star Zeng <star.z...@intel.com> > Reviewed-by: Ruiyu Ni <ruiyu...@intel.com> > Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com>
I didn't actually give my r-b for patch 8 or 9. (I wanted to see how the commit message was re-worded first.) But, if you fix the subject line of these patches to be more like patch 10, then you can keep my r-b on them. Oh ... I see you already committed the patches. -Jordan > --- > Nt32Pkg/Nt32Pkg.dsc | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Nt32Pkg/Nt32Pkg.dsc b/Nt32Pkg/Nt32Pkg.dsc > index b0d3fbf..2344d9a 100644 > --- a/Nt32Pkg/Nt32Pkg.dsc > +++ b/Nt32Pkg/Nt32Pkg.dsc > @@ -136,6 +136,10 @@ [LibraryClasses] > IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf > > TpmMeasurementLib|SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.inf > + AuthVariableLib|SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf > +!else > + > TpmMeasurementLib|MdeModulePkg/Library/TpmMeasurementLibNull/TpmMeasurementLibNull.inf > + > AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLibNull.inf > !endif > > [LibraryClasses.common.USER_DEFINED] > -- > 1.9.5.msysgit.0 > > > ------------------------------------------------------------------------------ > Don't Limit Your Business. Reach for the Cloud. > GigeNET's Cloud Solutions provide you with the tools and support that > you need to offload your IT needs and focus on growing your business. > Configured For All Businesses. Start Your Cloud Today. > https://www.gigenetcloud.com/ > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/edk2-devel ------------------------------------------------------------------------------ Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
