Re: [edk2-libc Patch V2 0/9] Update for edk2/master and VS2017/VS2019

["Rebecca Cran" <rebecca@nuviainc.com>]

For the entire series:


Reviewed-by: Rebecca Cran <rebecca@nuviainc.com>


-- 
Rebecca Cran


On 8/18/21 3:28 PM, Michael D Kinney wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3549
>
> New in V2
> =========
> * Add LibNetUtil to StdLib/BsdSocketLib in alphebetic order
> * Add comments explaining why SocketDxe is removed from the build.
>    New TianoCore Bugzilla with more details on this issue:
>    https://bugzilla.tianocore.org/show_bug.cgi?id=3572
>
> Add BsdSocketLib and EfiSocketLib to [LibraryClasses] section
> of Python-2.7.10.inf.  Even if sockets are disabled in Python
> config, with the /WHOLEARCHIVE linker option, there are still
> references to the socket related functions.  Add these lib
> classes so the link can pass for both sockets enabled and sockets
> disabled builds.
>
> Cc: Rebecca Cran <rebecca@nuviainc.com>
> Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
>
> Michael D Kinney (9):
>    StdLib: Add /Wv:11 to VS2017/VS2019 CC_FLAGS
>    StdLib: Add MdeLibs.dsc.inc to StdLib.dsc
>    StdLib: Remove SocketDxe from build
>    StdLib/LibC/gdtoa: Remove source file with unused ldtoa() function.
>    StdLib/BsdSocketLib: Add missing LibNetUtil dependency to INF
>    StdLib/BsdSocketLib: Add /wd4706 to CC_FLAGS
>    AppPkg: Add MdeLibs.dsc.inc to AppPkg.dsc
>    AppPkg/Applications/Python: Enable HAVE_COPYSIGN define
>    AppPkg/Applications/Python/Python-2.7.10: Update INF lib classes
>
>   AppPkg/AppPkg.dsc                                        | 2 ++
>   AppPkg/Applications/Python/Ia32/pyconfig.h               | 2 +-
>   AppPkg/Applications/Python/Python-2.7.10/Ia32/pyconfig.h | 2 +-
>   AppPkg/Applications/Python/Python-2.7.10/Python2710.inf  | 7 ++-----
>   AppPkg/Applications/Python/Python-2.7.10/X64/pyconfig.h  | 2 +-
>   AppPkg/Applications/Python/X64/pyconfig.h                | 3 +--
>   StdLib/BsdSocketLib/BsdSocketLib.inf                     | 4 ++++
>   StdLib/LibC/gdtoa/gdtoa.inf                              | 1 -
>   StdLib/StdLib.dsc                                        | 2 ++
>   StdLib/StdLib.inc                                        | 8 +++++++-
>   10 files changed, 21 insertions(+), 12 deletions(-)
>