[edk2-devel] [edk2-libc Patch 1/1] AppPkg/Applications/Python: Remove py2.7.2 support from edk2-libc

Michael D Kinney michael.d.kinney at intel.com
Sat Oct 23 17:09:45 UTC 2021


Can you please send V2 of the patch series with the Readme changes in its own patch?

Thanks,

Mike

> -----Original Message-----
> From: Jayaprakash, N <n.jayaprakash at intel.com>
> Sent: Friday, October 22, 2021 4:23 AM
> To: Kinney, Michael D <michael.d.kinney at intel.com>; devel at edk2.groups.io
> Cc: Rebecca Cran <rebecca at nuviainc.com>
> Subject: RE: [edk2-devel] [edk2-libc Patch 1/1] AppPkg/Applications/Python: Remove py2.7.2 support from edk2-libc
> 
> Hi Mike,
> 
> Could you look into this and let me know if there is anything else need to be done.
> 
> Regards,
> JP
> -----Original Message-----
> From: Jayaprakash, N
> Sent: 20 October 2021 23:15
> To: Kinney, Michael D <michael.d.kinney at intel.com>; devel at edk2.groups.io
> Cc: Rebecca Cran <rebecca at nuviainc.com>
> Subject: RE: [edk2-devel] [edk2-libc Patch 1/1] AppPkg/Applications/Python: Remove py2.7.2 support from edk2-libc
> 
> Hi Mike,
> 
> Thanks for the review comments.
> 
> The PythonReadMe.txt available @ https://github.com/tianocore/edk2-
> libc/blob/master/AppPkg/Applications/Python/PythonReadMe.txt
> is the readme file for Py2.7.2 and we don't need to retain this file. So I have deleted this file as part of the patch
> sent for review.
> 
> Py 2.7.10 and Py 3.6.8 have their respective readme files as Py2710ReadMe.txt @ https://github.com/jpshivakavi/edk2-
> libc/tree/master/AppPkg/Applications/Python/Python-2.7.10
> Py368ReadMe.txt @ https://github.com/jpshivakavi/edk2-libc/tree/master/AppPkg/Applications/Python/Python-3.6.8
> 
> 
> Besides this, I have taken care of all the other documentation changes required as given below
> 
> Updated the readme.md file from this location and removed the reference to Py2.7.2 license
> https://github.com/tianocore/edk2-libc/blob/master/Readme.md
> 
> 	AppPkg/Applications/Python/Python-2.7.2/Tools/pybench
> 	AppPkg/Applications/Python/Python-2.7.2
> 
> Updated the readme.txt from the below location to remove references to 2.7.2 and replace it with 3.6.8 references.
> https://github.com/tianocore/edk2-libc/blob/master/AppPkg/ReadMe.txt
> Also updated the version of this readme file along with the date
> 		   Version 1.03
>                                  18 Oct. 2021
> 
> 
> Besides documentation changes following changes have been done to delete py 2.7.2 support from edk2-libc
> Updated the AppPkg.dsc file to remove the Python 2.7.2 inf references.
> https://github.com/jpshivakavi/edk2-libc/blob/remove_py272_support/AppPkg/AppPkg.dsc
> 
> 
> Removed all files and folders corresponding to Py2.7.2 support from
> https://github.com/jpshivakavi/edk2-libc/tree/master/AppPkg/Applications/Python
> Efi\
> Ia32\
> PyMod-2.7.2\
> Python-2.7.2\
> X64\
> PythonCore.inf    // Inf file for py 2.7.2
> PythonReadme.txt // Readme file for Py 2.7.2
> 
> 
> Let me know if there is anything else needed.
> 
> Regards,
> JP
> 
> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney at intel.com>
> Sent: 20 October 2021 21:35
> To: devel at edk2.groups.io; Jayaprakash, N <n.jayaprakash at intel.com>; Kinney, Michael D <michael.d.kinney at intel.com>
> Cc: Rebecca Cran <rebecca at nuviainc.com>
> Subject: RE: [edk2-devel] [edk2-libc Patch 1/1] AppPkg/Applications/Python: Remove py2.7.2 support from edk2-libc
> 
> Hi JP,
> 
> Can you also update the documentation to remove references to Python 2.x or update for Python 3.x?
> 
> For example, the following file has Python 2.x references.
> 
> https://github.com/tianocore/edk2-libc/blob/master/AppPkg/Applications/Python/PythonReadMe.txt
> 
> Mike
> 
> > -----Original Message-----
> > From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of
> > Jayaprakash, N
> > Sent: Tuesday, October 19, 2021 8:43 PM
> > To: devel at edk2.groups.io
> > Cc: Rebecca Cran <rebecca at nuviainc.com>; Kinney, Michael D
> > <michael.d.kinney at intel.com>; Jayaprakash, N <n.jayaprakash at intel.com>
> > Subject: [edk2-devel] [edk2-libc Patch 1/1]
> > AppPkg/Applications/Python: Remove py2.7.2 support from edk2-libc
> >
> >
> >
> > 
> >



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#82602): https://edk2.groups.io/g/devel/message/82602
Mute This Topic: https://groups.io/mt/86457669/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-






More information about the edk2-devel-archive mailing list