Re: [edk2-devel] [PATCH] BaseTools/GenBiosId: Add a new tool GenBiosId

2019-06-14 Thread Michael D Kinney
Thank you for converting to Python. Please use python argparse module to parse arguments and make sure the tool supports the standard EDK II tool command line arguments. BinToPcd is a good example: https://github.com/tianocore/edk2/blob/master/BaseTools/Scripts/BinToPcd.py Mike > -Original

Re: [edk2-devel] [PATCH] BaseTools/GenBiosId: Add a new tool GenBiosId

2019-05-30 Thread Zhang, Shenglei
> -Original Message- > From: Kinney, Michael D > Sent: Wednesday, May 29, 2019 12:49 AM > To: devel@edk2.groups.io; Zhang, Shenglei ; > Kinney, Michael D > Subject: RE: [edk2-devel] [PATCH] BaseTools/GenBiosId: Add a new tool > GenBiosId > > Could this tool

Re: [edk2-devel] [PATCH] BaseTools/GenBiosId: Add a new tool GenBiosId

2019-05-28 Thread Michael D Kinney
Could this tools be ported to Python using the python struct module to access C structures? The implementation may be much simpler if python was used for command line parsing. And this capability could also be provided as a python module to support direct access from pre/post build steps impleme