Hi., Is this Patch Uploaded in GitHub?
Regards, Ashraf Ali S Intel Technology India Pvt. Ltd. -----Original Message----- From: Ni, Ray <ray...@intel.com> Sent: Tuesday, July 27, 2021 10:42 AM To: S, Ashraf Ali <ashraf.al...@intel.com>; devel@edk2.groups.io Cc: Kumar, Rahul1 <rahul1.ku...@intel.com>; De, Debkumar <debkumar...@intel.com>; Han, Harry <harry....@intel.com>; West, Catharine <catharine.w...@intel.com>; V, Sangeetha <sangeeth...@intel.com> Subject: RE: [PATCH v2 2/2] UefiCpuPkg: ResetVector Tool additional debug prints Reviewed-by: Ray Ni <ray...@intel.com> > -----Original Message----- > From: S, Ashraf Ali <ashraf.al...@intel.com> > Sent: Friday, July 23, 2021 4:40 PM > To: devel@edk2.groups.io > Cc: S, Ashraf Ali <ashraf.al...@intel.com>; Ni, Ray > <ray...@intel.com>; Kumar, Rahul1 <rahul1.ku...@intel.com>; De, > Debkumar <debkumar...@intel.com>; Han, Harry <harry....@intel.com>; > West, Catharine <catharine.w...@intel.com>; V, Sangeetha > <sangeeth...@intel.com> > Subject: [PATCH v2 2/2] UefiCpuPkg: ResetVector Tool additional debug > prints > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3506 > > Before executing the nasm command, added print statement to know what > commands are executing. > before printing the output file need check the status of command which > is executed. if the status is 0 then only print the output file name. > > Cc: Ray Ni <ray...@intel.com> > Cc: Rahul Kumar <rahul1.ku...@intel.com> > Cc: Debkumar De <debkumar...@intel.com> > Cc: Harry Han <harry....@intel.com> > Cc: Catharine West <catharine.w...@intel.com> > Cc: Sangeetha V <sangeeth...@intel.com> > Signed-off-by: Ashraf Ali S <ashraf.al...@intel.com> > --- > UefiCpuPkg/ResetVector/Vtf0/Build.py | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/UefiCpuPkg/ResetVector/Vtf0/Build.py > b/UefiCpuPkg/ResetVector/Vtf0/Build.py > index 55f4edd87b..b791d32762 100644 > --- a/UefiCpuPkg/ResetVector/Vtf0/Build.py > +++ b/UefiCpuPkg/ResetVector/Vtf0/Build.py > @@ -32,9 +32,12 @@ for arch in ('ia32', 'x64'): > '-o', output, > 'Vtf0.nasmb', > ) > + print(f"Command : {' '.join(commandLine)}") > ret = RunCommand(commandLine) > + if ret != 0: > + print(f"something went wrong while executing {commandLine[-1]}") > + sys.exit() > print('\tASM\t' + output) > - if ret != 0: sys.exit(ret) > > commandLine = ( > 'python', > -- > 2.30.2.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#79314): https://edk2.groups.io/g/devel/message/79314 Mute This Topic: https://groups.io/mt/84397007/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
