Hi Mike Yes, I agree with you. If we have a way to reduce the number of INF, we should. Feel free to submit patch.
BTW: Do you think we have chance to combine OpensslLibOpt.inf with OpensslLib.inf, with PCD Feature Flag: "Opt"? Thank you Yao Jiewen > -----Original Message----- > From: Kinney, Michael D <michael.d.kin...@intel.com> > Sent: Saturday, September 24, 2022 4:58 AM > To: devel@edk2.groups.io; Yao, Jiewen <jiewen....@intel.com>; Kinney, > Michael D <michael.d.kin...@intel.com>; Wang, Jian J > <jian.j.w...@intel.com>; Lu, Xiaoyu1 <xiaoyu1...@intel.com>; Jiang, > Guomin <guomin.ji...@intel.com> > Subject: CryptoPkg OpensslLib INF files > > Hi Jiewen, > > I see we now have 6 INF files for the OpensslLib > > * OpensslLib.inf > * OpensslLibCrypto.inf > * OpensslLibIa32.inf > * OpensslLibIa32Gcc.inf > * OpensslLibX64.inf > * OpensslLibX64Gcc.inf > > If I look at the difference between OpensslLib and OpensslLibCrypto, the > OpensslLibCrypto includes the "ssl" source files. > > This looks like a similar problem as the "ec" sources. But the "ec" sources > were addressed with a PCD FeatureFlag expression so we did not have to > add another INF. > > Could the same technique be applied to the "ssl" sources so we can get > back to just OpensslLib.inf with an SSL PCD and an EC PCD to conditionally > build the extra source files? > > For the other 4 INF files, these contain the assembly optimized algorithms > for IA32/X64. I think these 4 INFs can be combined into a single INF. > Perhaps OpensslLibOpt.inf? > > Mike -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94260): https://edk2.groups.io/g/devel/message/94260 Mute This Topic: https://groups.io/mt/93879787/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
