public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Ayush Singh" <ayushdevel1325@gmail.com>
To: edk2-devel-groups-io <devel@edk2.groups.io>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>
Cc: "Desimone, Nathaniel L" <nathaniel.l.desimone@intel.com>
Subject: Re: [edk2-devel] [PATCH v1 0/1] BaseTools: Fix Python Formatting
Date: Sat, 15 Oct 2022 18:39:36 +0530	[thread overview]
Message-ID: <CA+Yfj7vs1bt2YCOK=_XAf1Zx1YE7iU4B7zq_iNvz2EUZXaDYfg@mail.gmail.com> (raw)
In-Reply-To: <CO1PR11MB49292B3FC3314B16B42E510DD2229@CO1PR11MB4929.namprd11.prod.outlook.com>

[-- Attachment #1: Type: text/plain, Size: 29181 bytes --]

Thanks Michael. I totally missed the Maintainers.txt. I will try improving
the Getting Started section of edk2 wiki.


I have opened an issue (
https://github.com/tianocore/edk2-basetools/issues/57) and PR (
https://github.com/tianocore/edk2-basetools/pull/58) to discuss the same.

Yours Sincerely,
Ayush Singh

On Wed, 12 Oct, 2022, 10:42 Michael D Kinney, <michael.d.kinney@intel.com>
wrote:

> Hi Ayush,
>
>
>
> All BaseTools python changes are initially done in the edk2-basetools
> repo to generate pip modules and then ported to edk2 repo BaseTools dir
>
>
>
> https://github.com/tianocore/edk2-basetools/
>
>
>
> Maintained by EDK II Tools Maintainers
>
>
>
> https://github.com/orgs/tianocore/teams/edk-ii-tool-maintainers
>
>
>
> edk2 Maintainers.txt lists the following Maintainers and Reviewers for the
> BaseTools directory in edk2 repo
>
> https://github.com/tianocore/edk2/blob/master/Maintainers.txt
>
>
>
> BaseTools
>
> F: BaseTools/
>
> W: https://github.com/tianocore/tianocore.github.io/wiki/BaseTools
>
> M: Bob Feng <bob.c.feng@intel.com> [BobCF]
>
> M: Liming Gao <gaoliming@byosoft.com.cn> [lgao4]
>
> R: Yuwei Chen <yuwei.chen@intel.com> [YuweiChen1110]
>
>
>
> There is also a Tools and CI meeting in Monday listed in the groups.io
> calendar where these types of topics are discussed.
>
>
>
> https://github.com/tianocore/edk2/discussions/2614
>
>
>
> Best regards,
>
>
>
> Mike
>
>
>
> *From:* Ayush Singh <ayushdevel1325@gmail.com>
> *Sent:* Tuesday, October 11, 2022 9:57 PM
> *To:* edk2-devel-groups-io <devel@edk2.groups.io>
> *Cc:* Kinney, Michael D <michael.d.kinney@intel.com>; Desimone, Nathaniel
> L <nathaniel.l.desimone@intel.com>
> *Subject:* Re: [PATCH v1 0/1] BaseTools: Fix Python Formatting
>
>
>
> I wanted to ask if BaseTools has a maintainer or someone I should directly
> Cc. I also think that it would be great to have a list of people for
> different parts of edk2 (like Linux Kernel has) for contributions to each
> package.
>
>
>
> On Tue, 11 Oct, 2022, 01:36 Ayush Singh, <ayushdevel1325@gmail.com> wrote:
>
> Fix formatting of Python files in BaseTools to conform to PEP8 using
> autopep8. This does not fix all the warnings/errors from flake8, but I
> wanted to get this patch checked out first to see if ignoring those
> warnings is deliberate or not.
>
> The complete code can be found:
> https://github.com/Ayush1325/edk2/tree/formatting
>
> Ayush Singh (1):
>   Format BaseTools python files using autopep8
>
>  BaseTools/Bin/CYGWIN_NT-5.1-i686/armcc_wrapper.py
>                |   74 +-
>  BaseTools/Edk2ToolsBuild.py
>                |    4 +-
>  BaseTools/Plugin/BuildToolsReport/BuildToolsReportGenerator.py
>               |   15 +-
>  BaseTools/Plugin/LinuxGcc5ToolChain/LinuxGcc5ToolChain.py
>                |    6 +-
>  BaseTools/Plugin/WindowsResourceCompiler/WinRcPath.py
>                |    8 +-
>  BaseTools/Scripts/BinToPcd.py
>                |  185 +-
>  BaseTools/Scripts/ConvertFceToStructurePcd.py
>                | 1312 ++--
>  BaseTools/Scripts/ConvertMasmToNasm.py
>               |    7 +-
>  BaseTools/Scripts/ConvertUni.py
>                |   14 +-
>  BaseTools/Scripts/DetectNotUsedItem.py
>               |   23 +-
>  BaseTools/Scripts/FormatDosFiles.py
>                |   25 +-
>  BaseTools/Scripts/GetMaintainer.py
>               |   19 +-
>  BaseTools/Scripts/GetUtcDateTime.py
>                |   18 +-
>  BaseTools/Scripts/MemoryProfileSymbolGen.py
>                |  162 +-
>  BaseTools/Scripts/PackageDocumentTools/__init__.py
>               |    2 +-
>  BaseTools/Scripts/PackageDocumentTools/packagedoc_cli.py
>               |  138 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/__init__.py
>                |    2 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/basemodel/__init__.py
>        |    2 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/basemodel/doxygen.py
>         |   79 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/basemodel/efibinary.py
>       |   96 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/basemodel/ini.py
>             |   92 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/basemodel/inidocview.py
>      |    3 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/basemodel/message.py
>         |   12 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/__init__.py
>             |    2 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/__init__.py
>       |    2 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/baseobject.py
>     |  165 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/dec.py
>            |   41 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/doxygengen.py
>     |  374 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/doxygengen_spec.py
> |  372 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/dsc.py
>            |   25 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/inf.py
>            |   59 +-
>  BaseTools/Scripts/PackageDocumentTools/plugins/__init__.py
>               |    2 +-
>  BaseTools/Scripts/PatchCheck.py
>                |   90 +-
>  BaseTools/Scripts/RunMakefile.py
>               |  258 +-
>  BaseTools/Scripts/SetupGit.py
>                |   23 +-
>  BaseTools/Scripts/SmiHandlerProfileSymbolGen.py
>                |  165 +-
>  BaseTools/Scripts/UpdateBuildVersions.py
>               |   64 +-
>  BaseTools/Scripts/efi_debugging.py
>               |    4 +-
>  BaseTools/Scripts/efi_gdb.py
>               |    1 +
>  BaseTools/Source/C/Makefiles/NmakeSubdirs.py
>               |   43 +-
>  BaseTools/Source/C/PyEfiCompressor/setup.py
>                |   16 +-
>  BaseTools/Source/Python/AmlToC/AmlToC.py
>               |   29 +-
>  BaseTools/Source/Python/AutoGen/AutoGen.py
>               |   58 +-
>  BaseTools/Source/Python/AutoGen/AutoGenWorker.py
>               |  145 +-
>  BaseTools/Source/Python/AutoGen/BuildEngine.py
>               |  158 +-
>  BaseTools/Source/Python/AutoGen/DataPipe.py
>                |  152 +-
>  BaseTools/Source/Python/AutoGen/GenC.py
>                |  944 ++-
>  BaseTools/Source/Python/AutoGen/GenDepex.py
>                |  211 +-
>  BaseTools/Source/Python/AutoGen/GenMake.py
>               |  738 +-
>  BaseTools/Source/Python/AutoGen/GenPcdDb.py
>                |  533 +-
>  BaseTools/Source/Python/AutoGen/GenVar.py
>                |  182 +-
>  BaseTools/Source/Python/AutoGen/IdfClassObject.py
>                |   97 +-
>  BaseTools/Source/Python/AutoGen/IncludesAutoGen.py
>               |  109 +-
>  BaseTools/Source/Python/AutoGen/InfSectionParser.py
>                |   47 +-
>  BaseTools/Source/Python/AutoGen/ModuleAutoGen.py
>               |  945 ++-
>  BaseTools/Source/Python/AutoGen/ModuleAutoGenHelper.py
>               |  255 +-
>  BaseTools/Source/Python/AutoGen/PlatformAutoGen.py
>               |  554 +-
>  BaseTools/Source/Python/AutoGen/StrGather.py
>               |  221 +-
>  BaseTools/Source/Python/AutoGen/UniClassObject.py
>                |  261 +-
>  BaseTools/Source/Python/AutoGen/ValidCheckingInfoObject.py
>               |   23 +-
>  BaseTools/Source/Python/AutoGen/WorkspaceAutoGen.py
>                |  405 +-
>  BaseTools/Source/Python/AutoGen/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/BPDG/BPDG.py
>               |   37 +-
>  BaseTools/Source/Python/BPDG/GenVpd.py
>               |  348 +-
>  BaseTools/Source/Python/BPDG/StringTable.py
>                |   47 +-
>  BaseTools/Source/Python/BPDG/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/Capsule/GenerateCapsule.py
>               | 1329 ++--
>  BaseTools/Source/Python/Capsule/GenerateWindowsDriver.py
>               |  119 +-
>  BaseTools/Source/Python/Capsule/WindowsCapsuleSupportHelper.py
>               |   83 +-
>  BaseTools/Source/Python/Common/BuildToolError.py
>               |  109 +-
>  BaseTools/Source/Python/Common/BuildVersion.py
>               |    2 +-
>  BaseTools/Source/Python/Common/DataType.py
>               |  187 +-
>  BaseTools/Source/Python/Common/Edk2/Capsule/FmpPayloadHeader.py
>                |   84 +-
>  BaseTools/Source/Python/Common/Edk2/Capsule/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Common/Edk2/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Common/EdkLogger.py
>                |  112 +-
>  BaseTools/Source/Python/Common/Expression.py
>               |  243 +-
>  BaseTools/Source/Python/Common/GlobalData.py
>               |   24 +-
>  BaseTools/Source/Python/Common/LongFilePathOs.py
>               |   31 +-
>  BaseTools/Source/Python/Common/LongFilePathOsPath.py
>               |   10 +-
>  BaseTools/Source/Python/Common/LongFilePathSupport.py
>                |   11 +-
>  BaseTools/Source/Python/Common/Misc.py
>               |  479 +-
>  BaseTools/Source/Python/Common/MultipleWorkspace.py
>                |   32 +-
>  BaseTools/Source/Python/Common/Parsing.py
>                |  332 +-
>  BaseTools/Source/Python/Common/RangeExpression.py
>                |   89 +-
>  BaseTools/Source/Python/Common/StringUtils.py
>                |  194 +-
>  BaseTools/Source/Python/Common/TargetTxtClassObject.py
>               |   73 +-
>  BaseTools/Source/Python/Common/ToolDefClassObject.py
>               |   89 +-
>  BaseTools/Source/Python/Common/Uefi/Capsule/CapsuleDependency.py
>               |  394 +-
>  BaseTools/Source/Python/Common/Uefi/Capsule/FmpAuthHeader.py
>               |  117 +-
>  BaseTools/Source/Python/Common/Uefi/Capsule/FmpCapsuleHeader.py
>                |  286 +-
>  BaseTools/Source/Python/Common/Uefi/Capsule/UefiCapsuleHeader.py
>               |  110 +-
>  BaseTools/Source/Python/Common/Uefi/Capsule/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Common/Uefi/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Common/VariableAttributes.py
>               |   14 +-
>  BaseTools/Source/Python/Common/VpdInfoFile.py
>                |   96 +-
>  BaseTools/Source/Python/Common/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/Common/caching.py
>                |   29 +-
>  BaseTools/Source/Python/CommonDataClass/CommonClass.py
>               |   29 +-
>  BaseTools/Source/Python/CommonDataClass/DataClass.py
>               |   88 +-
>  BaseTools/Source/Python/CommonDataClass/Exceptions.py
>                |   12 +-
>  BaseTools/Source/Python/CommonDataClass/FdfClass.py
>                |  129 +-
>  BaseTools/Source/Python/CommonDataClass/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Ecc/CParser3/CLexer.py
>               | 1908 ++---
>  BaseTools/Source/Python/Ecc/CParser3/CParser.py
>                | 7876 +++++++++-----------
>  BaseTools/Source/Python/Ecc/CParser4/CLexer.py
>               |  140 +-
>  BaseTools/Source/Python/Ecc/CParser4/CListener.py
>                |  359 +-
>  BaseTools/Source/Python/Ecc/CParser4/CParser.py
>                | 2451 +++---
>  BaseTools/Source/Python/Ecc/Check.py
>               |  404 +-
>  BaseTools/Source/Python/Ecc/CodeFragment.py
>                |   68 +-
>  BaseTools/Source/Python/Ecc/CodeFragmentCollector.py
>               |  151 +-
>  BaseTools/Source/Python/Ecc/Configuration.py
>               |  245 +-
>  BaseTools/Source/Python/Ecc/Database.py
>                |  111 +-
>  BaseTools/Source/Python/Ecc/EccGlobalData.py
>               |    2 +-
>  BaseTools/Source/Python/Ecc/EccMain.py
>               |  144 +-
>  BaseTools/Source/Python/Ecc/EccToolError.py
>                |  177 +-
>  BaseTools/Source/Python/Ecc/Exception.py
>               |   16 +-
>  BaseTools/Source/Python/Ecc/FileProfile.py
>               |   10 +-
>  BaseTools/Source/Python/Ecc/MetaDataParser.py
>                |   62 +-
>  BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaDataTable.py
>               |   43 +-
>  BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py
>                |  827 +-
>  BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileTable.py
>               |  181 +-
>  BaseTools/Source/Python/Ecc/MetaFileWorkspace/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Ecc/ParserWarning.py
>               |    8 +-
>  BaseTools/Source/Python/Ecc/Xml/XmlRoutines.py
>               |   39 +-
>  BaseTools/Source/Python/Ecc/Xml/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Ecc/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Ecc/c.py
>               |  512 +-
>  BaseTools/Source/Python/Eot/CParser3/CLexer.py
>               | 1908 ++---
>  BaseTools/Source/Python/Eot/CParser3/CParser.py
>                | 7876 +++++++++-----------
>  BaseTools/Source/Python/Eot/CParser4/CLexer.py
>               |  139 +-
>  BaseTools/Source/Python/Eot/CParser4/CListener.py
>                |  358 +-
>  BaseTools/Source/Python/Eot/CParser4/CParser.py
>                | 2451 +++---
>  BaseTools/Source/Python/Eot/CodeFragment.py
>                |   78 +-
>  BaseTools/Source/Python/Eot/CodeFragmentCollector.py
>               |  119 +-
>  BaseTools/Source/Python/Eot/Database.py
>                |   77 +-
>  BaseTools/Source/Python/Eot/EotGlobalData.py
>               |    5 +-
>  BaseTools/Source/Python/Eot/EotMain.py
>               |  544 +-
>  BaseTools/Source/Python/Eot/EotToolError.py
>                |    7 +-
>  BaseTools/Source/Python/Eot/FileProfile.py
>               |   10 +-
>  BaseTools/Source/Python/Eot/Identification.py
>                |   11 +-
>  BaseTools/Source/Python/Eot/InfParserLite.py
>               |   52 +-
>  BaseTools/Source/Python/Eot/Parser.py
>                |  244 +-
>  BaseTools/Source/Python/Eot/ParserWarning.py
>               |    6 +-
>  BaseTools/Source/Python/Eot/Report.py
>                |   63 +-
>  BaseTools/Source/Python/Eot/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Eot/c.py
>               |  100 +-
>  BaseTools/Source/Python/FMMT/FMMT.py
>               |   57 +-
>  BaseTools/Source/Python/FMMT/__init__.py
>               |    4 +-
>  BaseTools/Source/Python/FMMT/core/BinaryFactoryProduct.py
>                |  151 +-
>  BaseTools/Source/Python/FMMT/core/BiosTree.py
>                |   47 +-
>  BaseTools/Source/Python/FMMT/core/BiosTreeNode.py
>                |   77 +-
>  BaseTools/Source/Python/FMMT/core/FMMTOperation.py
>               |   40 +-
>  BaseTools/Source/Python/FMMT/core/FMMTParser.py
>                |   30 +-
>  BaseTools/Source/Python/FMMT/core/FvHandler.py
>               |  201 +-
>  BaseTools/Source/Python/FMMT/core/GuidTools.py
>               |   47 +-
>  BaseTools/Source/Python/FMMT/utils/FmmtLogger.py
>               |   10 +-
>  BaseTools/Source/Python/FMMT/utils/FvLayoutPrint.py
>                |   29 +-
>  BaseTools/Source/Python/FirmwareStorageFormat/Common.py
>                |   20 +-
>  BaseTools/Source/Python/FirmwareStorageFormat/FfsFileHeader.py
>               |    5 +-
>  BaseTools/Source/Python/FirmwareStorageFormat/FvHeader.py
>                |   31 +-
>  BaseTools/Source/Python/FirmwareStorageFormat/SectionHeader.py
>               |    9 +-
>  BaseTools/Source/Python/FirmwareStorageFormat/__init__.py
>                |    4 +-
>  BaseTools/Source/Python/GenFds/AprioriSection.py
>               |   48 +-
>  BaseTools/Source/Python/GenFds/Capsule.py
>                |  103 +-
>  BaseTools/Source/Python/GenFds/CapsuleData.py
>                |  111 +-
>  BaseTools/Source/Python/GenFds/CompressSection.py
>                |   42 +-
>  BaseTools/Source/Python/GenFds/DataSection.py
>                |   63 +-
>  BaseTools/Source/Python/GenFds/DepexSection.py
>               |   40 +-
>  BaseTools/Source/Python/GenFds/EfiSection.py
>               |  167 +-
>  BaseTools/Source/Python/GenFds/Fd.py
>               |   80 +-
>  BaseTools/Source/Python/GenFds/FdfParser.py
>                | 1680 +++--
>  BaseTools/Source/Python/GenFds/Ffs.py
>                |   58 +-
>  BaseTools/Source/Python/GenFds/FfsFileStatement.py
>               |   67 +-
>  BaseTools/Source/Python/GenFds/FfsInfStatement.py
>                |  572 +-
>  BaseTools/Source/Python/GenFds/Fv.py
>               |  240 +-
>  BaseTools/Source/Python/GenFds/FvImageSection.py
>               |   77 +-
>  BaseTools/Source/Python/GenFds/GenFds.py
>               |  353 +-
>  BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py
>               |  355 +-
>  BaseTools/Source/Python/GenFds/GuidSection.py
>                |   90 +-
>  BaseTools/Source/Python/GenFds/OptRomFileStatement.py
>                |   16 +-
>  BaseTools/Source/Python/GenFds/OptRomInfStatement.py
>               |   56 +-
>  BaseTools/Source/Python/GenFds/OptionRom.py
>                |   46 +-
>  BaseTools/Source/Python/GenFds/Region.py
>               |  113 +-
>  BaseTools/Source/Python/GenFds/Rule.py
>               |    8 +-
>  BaseTools/Source/Python/GenFds/RuleComplexFile.py
>                |   12 +-
>  BaseTools/Source/Python/GenFds/RuleSimpleFile.py
>               |   10 +-
>  BaseTools/Source/Python/GenFds/Section.py
>                |  121 +-
>  BaseTools/Source/Python/GenFds/UiSection.py
>                |   23 +-
>  BaseTools/Source/Python/GenFds/VerSection.py
>               |   15 +-
>  BaseTools/Source/Python/GenFds/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/GenPatchPcdTable/GenPatchPcdTable.py
>               |   61 +-
>  BaseTools/Source/Python/GenPatchPcdTable/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/PatchPcdValue/PatchPcdValue.py
>               |   49 +-
>  BaseTools/Source/Python/PatchPcdValue/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Pkcs7Sign/Pkcs7Sign.py
>               |  408 +-
>  BaseTools/Source/Python/Rsa2048Sha256Sign/Rsa2048Sha256GenerateKeys.py
>               |  259 +-
>  BaseTools/Source/Python/Rsa2048Sha256Sign/Rsa2048Sha256Sign.py
>               |  350 +-
>  BaseTools/Source/Python/Split/Split.py
>               |   17 +-
>  BaseTools/Source/Python/Table/Table.py
>               |   25 +-
>  BaseTools/Source/Python/Table/TableDataModel.py
>                |   17 +-
>  BaseTools/Source/Python/Table/TableDec.py
>                |   15 +-
>  BaseTools/Source/Python/Table/TableDsc.py
>                |   15 +-
>  BaseTools/Source/Python/Table/TableEotReport.py
>                |   19 +-
>  BaseTools/Source/Python/Table/TableFdf.py
>                |   15 +-
>  BaseTools/Source/Python/Table/TableFile.py
>               |   26 +-
>  BaseTools/Source/Python/Table/TableFunction.py
>               |   15 +-
>  BaseTools/Source/Python/Table/TableIdentifier.py
>               |   15 +-
>  BaseTools/Source/Python/Table/TableInf.py
>                |   15 +-
>  BaseTools/Source/Python/Table/TablePcd.py
>                |   15 +-
>  BaseTools/Source/Python/Table/TableQuery.py
>                |   12 +-
>  BaseTools/Source/Python/Table/TableReport.py
>               |   37 +-
>  BaseTools/Source/Python/Table/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/TargetTool/TargetTool.py
>               |  107 +-
>  BaseTools/Source/Python/TargetTool/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/Trim/Trim.py
>               |  224 +-
>  BaseTools/Source/Python/UPT/BuildVersion.py
>                |    2 +-
>  BaseTools/Source/Python/UPT/Core/DependencyRules.py
>                |   82 +-
>  BaseTools/Source/Python/UPT/Core/DistributionPackageClass.py
>               |   78 +-
>  BaseTools/Source/Python/UPT/Core/FileHook.py
>               |   50 +-
>  BaseTools/Source/Python/UPT/Core/IpiDb.py
>                |  230 +-
>  BaseTools/Source/Python/UPT/Core/PackageFile.py
>                |   61 +-
>  BaseTools/Source/Python/UPT/Core/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/UPT/GenMetaFile/GenDecFile.py
>                |  149 +-
>  BaseTools/Source/Python/UPT/GenMetaFile/GenInfFile.py
>                |  181 +-
>  BaseTools/Source/Python/UPT/GenMetaFile/GenMetaFileMisc.py
>               |   40 +-
>  BaseTools/Source/Python/UPT/GenMetaFile/GenXmlFile.py
>                |    2 +-
>  BaseTools/Source/Python/UPT/GenMetaFile/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/UPT/InstallPkg.py
>                |  295 +-
>  BaseTools/Source/Python/UPT/InventoryWs.py
>               |   45 +-
>  BaseTools/Source/Python/UPT/Library/CommentGenerating.py
>               |   88 +-
>  BaseTools/Source/Python/UPT/Library/CommentParsing.py
>                |  177 +-
>  BaseTools/Source/Python/UPT/Library/DataType.py
>                |  379 +-
>  BaseTools/Source/Python/UPT/Library/ExpressionValidate.py
>                |  147 +-
>  BaseTools/Source/Python/UPT/Library/GlobalData.py
>                |    2 +-
>  BaseTools/Source/Python/UPT/Library/Misc.py
>                |  227 +-
>  BaseTools/Source/Python/UPT/Library/ParserValidate.py
>                |  130 +-
>  BaseTools/Source/Python/UPT/Library/Parsing.py
>               |  363 +-
>  BaseTools/Source/Python/UPT/Library/StringUtils.py
>               |  203 +-
>  BaseTools/Source/Python/UPT/Library/UniClassObject.py
>                |  447 +-
>  BaseTools/Source/Python/UPT/Library/Xml/XmlRoutines.py
>               |   37 +-
>  BaseTools/Source/Python/UPT/Library/Xml/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/UPT/Library/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/UPT/Logger/Log.py
>                |   97 +-
>  BaseTools/Source/Python/UPT/Logger/StringTable.py
>                |  933 +--
>  BaseTools/Source/Python/UPT/Logger/ToolError.py
>                |  117 +-
>  BaseTools/Source/Python/UPT/Logger/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/UPT/MkPkg.py
>               |   73 +-
>  BaseTools/Source/Python/UPT/Object/POM/CommonObject.py
>               |   93 +-
>  BaseTools/Source/Python/UPT/Object/POM/ModuleObject.py
>               |   35 +-
>  BaseTools/Source/Python/UPT/Object/POM/PackageObject.py
>                |   13 +-
>  BaseTools/Source/Python/UPT/Object/POM/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/UPT/Object/Parser/DecObject.py
>               |  186 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfBinaryObject.py
>               |  138 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfBuildOptionObject.py
>                |   15 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfCommonObject.py
>               |   46 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfDefineCommonObject.py
>               |   36 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfDefineObject.py
>               |  334 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfDepexObject.py
>                |   21 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfGuidObject.py
>               |   51 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfHeaderObject.py
>               |   34 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfLibraryClassesObject.py
>               |   35 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfMisc.py
>               |   23 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfPackagesObject.py
>               |   35 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfPcdObject.py
>                |  116 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfPpiObject.py
>                |   46 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfProtocolObject.py
>               |   42 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfSoucesObject.py
>               |   66 +-
>  BaseTools/Source/Python/UPT/Object/Parser/InfUserExtensionObject.py
>                |   24 +-
>  BaseTools/Source/Python/UPT/Object/Parser/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/UPT/Object/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/UPT/Parser/DecParser.py
>                |  280 +-
>  BaseTools/Source/Python/UPT/Parser/DecParserMisc.py
>                |   56 +-
>  BaseTools/Source/Python/UPT/Parser/InfAsBuiltProcess.py
>                |   47 +-
>  BaseTools/Source/Python/UPT/Parser/InfBinarySectionParser.py
>               |   44 +-
>  BaseTools/Source/Python/UPT/Parser/InfBuildOptionSectionParser.py
>                |   53 +-
>  BaseTools/Source/Python/UPT/Parser/InfDefineSectionParser.py
>               |   45 +-
>  BaseTools/Source/Python/UPT/Parser/InfDepexSectionParser.py
>                |   15 +-
>  BaseTools/Source/Python/UPT/Parser/InfGuidPpiProtocolSectionParser.py
>                |   73 +-
>  BaseTools/Source/Python/UPT/Parser/InfLibrarySectionParser.py
>                |   38 +-
>  BaseTools/Source/Python/UPT/Parser/InfPackageSectionParser.py
>                |   34 +-
>  BaseTools/Source/Python/UPT/Parser/InfParser.py
>                |  164 +-
>  BaseTools/Source/Python/UPT/Parser/InfParserMisc.py
>                |  122 +-
>  BaseTools/Source/Python/UPT/Parser/InfPcdSectionParser.py
>                |   45 +-
>  BaseTools/Source/Python/UPT/Parser/InfSectionParser.py
>               |   94 +-
>  BaseTools/Source/Python/UPT/Parser/InfSourceSectionParser.py
>               |   39 +-
>  BaseTools/Source/Python/UPT/Parser/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/UPT/PomAdapter/DecPomAlignment.py
>                |  236 +-
>  BaseTools/Source/Python/UPT/PomAdapter/InfPomAlignment.py
>                |  183 +-
>  BaseTools/Source/Python/UPT/PomAdapter/InfPomAlignmentMisc.py
>                |   35 +-
>  BaseTools/Source/Python/UPT/PomAdapter/__init__.py
>               |    2 +-
>  BaseTools/Source/Python/UPT/ReplacePkg.py
>                |   60 +-
>  BaseTools/Source/Python/UPT/RmPkg.py
>               |   66 +-
>  BaseTools/Source/Python/UPT/TestInstall.py
>               |   27 +-
>  BaseTools/Source/Python/UPT/UPT.py
>               |  150 +-
>  BaseTools/Source/Python/UPT/UnitTest/CommentGeneratingUnitTest.py
>                |   86 +-
>  BaseTools/Source/Python/UPT/UnitTest/CommentParsingUnitTest.py
>               |   91 +-
>  BaseTools/Source/Python/UPT/UnitTest/DecParserTest.py
>                |   23 +-
>  BaseTools/Source/Python/UPT/UnitTest/DecParserUnitTest.py
>                |  118 +-
>  BaseTools/Source/Python/UPT/UnitTest/InfBinarySectionTest.py
>               |  131 +-
>  BaseTools/Source/Python/UPT/Xml/CommonXml.py
>               |  265 +-
>  BaseTools/Source/Python/UPT/Xml/GuidProtocolPpiXml.py
>                |  139 +-
>  BaseTools/Source/Python/UPT/Xml/IniToXml.py
>                |  152 +-
>  BaseTools/Source/Python/UPT/Xml/ModuleSurfaceAreaXml.py
>                |  143 +-
>  BaseTools/Source/Python/UPT/Xml/PackageSurfaceAreaXml.py
>               |   87 +-
>  BaseTools/Source/Python/UPT/Xml/PcdXml.py
>                |  141 +-
>  BaseTools/Source/Python/UPT/Xml/XmlParser.py
>               |  363 +-
>  BaseTools/Source/Python/UPT/Xml/XmlParserMisc.py
>               |   22 +-
>  BaseTools/Source/Python/UPT/Xml/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/Workspace/BuildClassObject.py
>                |  303 +-
>  BaseTools/Source/Python/Workspace/DecBuildData.py
>                |  184 +-
>  BaseTools/Source/Python/Workspace/DscBuildData.py
>                | 2007 +++--
>  BaseTools/Source/Python/Workspace/InfBuildData.py
>                |  426 +-
>  BaseTools/Source/Python/Workspace/MetaDataTable.py
>               |   98 +-
>  BaseTools/Source/Python/Workspace/MetaFileCommentParser.py
>               |   21 +-
>  BaseTools/Source/Python/Workspace/MetaFileParser.py
>                |  885 ++-
>  BaseTools/Source/Python/Workspace/MetaFileTable.py
>               |  217 +-
>  BaseTools/Source/Python/Workspace/WorkspaceCommon.py
>               |   60 +-
>  BaseTools/Source/Python/Workspace/WorkspaceDatabase.py
>               |   67 +-
>  BaseTools/Source/Python/Workspace/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/build/BuildReport.py
>               |  766 +-
>  BaseTools/Source/Python/build/__init__.py
>                |    2 +-
>  BaseTools/Source/Python/build/build.py
>               | 1145 +--
>  BaseTools/Source/Python/build/buildoptions.py
>                |  113 +-
>  BaseTools/Source/Python/sitecustomize.py
>               |   11 +-
>  BaseTools/Source/Python/tests/Split/test_split.py
>                |   37 +-
>  BaseTools/Tests/CToolsTests.py
>               |    6 +-
>  BaseTools/Tests/CheckPythonSyntax.py
>               |   15 +-
>  BaseTools/Tests/CheckUnicodeSourceFiles.py
>               |    4 +-
>  BaseTools/Tests/PythonTest.py
>                |    2 +-
>  BaseTools/Tests/PythonToolsTests.py
>                |    4 +-
>  BaseTools/Tests/RunTests.py
>                |    7 +-
>  BaseTools/Tests/TestRegularExpression.py
>               |    7 +-
>  BaseTools/Tests/TestTools.py
>               |   54 +-
>  BaseTools/Tests/TianoCompress.py
>               |   15 +-
>  335 files changed, 35765 insertions(+), 32705 deletions(-)
>
> --
> 2.37.3
>
> 
>

[-- Attachment #2: Type: text/html, Size: 49676 bytes --]

      reply	other threads:[~2022-10-15 13:09 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-10 20:05 [PATCH v1 0/1] BaseTools: Fix Python Formatting Ayush Singh
2022-10-10 20:05 ` [PATCH v1 1/1] Format BaseTools python files using autopep8 Ayush Singh
2022-10-12  4:56 ` [PATCH v1 0/1] BaseTools: Fix Python Formatting Ayush Singh
2022-10-12  5:12   ` Michael D Kinney
2022-10-15 13:09     ` Ayush Singh [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CA+Yfj7vs1bt2YCOK=_XAf1Zx1YE7iU4B7zq_iNvz2EUZXaDYfg@mail.gmail.com' \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox