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.jayaprak...@intel.com> > Sent: Friday, October 22, 2021 4:23 AM > To: Kinney, Michael D <michael.d.kin...@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebe...@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.kin...@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebe...@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.kin...@intel.com> > Sent: 20 October 2021 21:35 > To: devel@edk2.groups.io; Jayaprakash, N <n.jayaprak...@intel.com>; Kinney, > Michael D <michael.d.kin...@intel.com> > Cc: Rebecca Cran <rebe...@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@edk2.groups.io <devel@edk2.groups.io> On Behalf Of > > Jayaprakash, N > > Sent: Tuesday, October 19, 2021 8:43 PM > > To: devel@edk2.groups.io > > Cc: Rebecca Cran <rebe...@nuviainc.com>; Kinney, Michael D > > <michael.d.kin...@intel.com>; Jayaprakash, N <n.jayaprak...@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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
