It will change it. For example if we want to make SM2 conditional also, we can 
change config list to:
my %conditional_feature = ("PcdOpensslEcEnabled"=>["EC", "ECDH", "ECDSA", 
"TLS1_3"], "PcdOpensslSM2Enabled=>["SM2"]);
my %conditional_feature_dir = ("/ec/"=>"PcdOpensslEcEnabled", 
"/sm2/"=>"PcdOpensslSM2Enabled");
It will add the relevant define to opensslconf.h and INF files without manual 
addition.
Better ideas are also welcome.


-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Gerd Hoffmann
Sent: Wednesday, May 11, 2022 3:18 PM
To: Li, Yi1 <yi1...@intel.com>
Cc: devel@edk2.groups.io; Yao, Jiewen <jiewen....@intel.com>; Wang, Jian J 
<jian.j.w...@intel.com>; Lu, Xiaoyu1 <xiaoyu1...@intel.com>; Jiang, Guomin 
<guomin.ji...@intel.com>
Subject: Re: [edk2-devel] [PATCH V2 2/5] CryptoPkg: Separate auto-generated 
openssl config and edk2 openssl config

On Tue, May 10, 2022 at 11:43:54AM +0000, Li, Yi1 wrote:
> Just for convenience.
> There are too many places need to configure, so I think putting all 
> the configuration options close together, all in process_file.pl, will 
> be more clear to user ;)

Well, I find it confusing to let process_file.pl write a file which doesn't 
change ...

take care,
  Gerd








-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89676): https://edk2.groups.io/g/devel/message/89676
Mute This Topic: https://groups.io/mt/91006616/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to