Re: [edk2-libc Patch 1/1] AppPkg/Applications/Python: Remove py2.7.2 support from edk2-libc
Hi Mike,toggle quoted messageShow quoted text
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
Updated the readme.txt from the below location to remove references to 2.7.2 and replace it with 3.6.8 references.
Also updated the version of this readme file along with the date
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.
Removed all files and folders corresponding to Py2.7.2 support from
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.
From: Kinney, Michael D <email@example.com>
Sent: 20 October 2021 21:35
To: firstname.lastname@example.org; Jayaprakash, N <email@example.com>; Kinney, Michael D <firstname.lastname@example.org>
Cc: Rebecca Cran <email@example.com>
Subject: RE: [edk2-devel] [edk2-libc Patch 1/1] AppPkg/Applications/Python: Remove py2.7.2 support from edk2-libc
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.