Absolutely. I should have thought to add that. "Python ScriptCheck.py t"
Assuming there is no commit or file called "t". Basically just anything that is not a commit identifier nor a filename. I found it when I tried to select a commit and misspelled it. -Jaben > -----Original Message----- > From: Gao, Liming > Sent: Friday, August 03, 2018 2:44 AM > To: Carsey, Jaben <jaben.car...@intel.com>; edk2-devel@lists.01.org > Subject: RE: [edk2] [PATCH v1 1/1] PatchCheck - add error message for > invalid parameter > Importance: High > > Jaben: > Could you give me one failure case? Then, I can further understand the > patch. > > Thanks > Liming > >-----Original Message----- > >From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > >Jaben Carsey > >Sent: Friday, August 03, 2018 7:21 AM > >To: edk2-devel@lists.01.org > >Cc: Gao, Liming <liming....@intel.com> > >Subject: [edk2] [PATCH v1 1/1] PatchCheck - add error message for invalid > >parameter > > > >Currently if an invalid parameter is passed, it gives a stack trace. > >This changes it to an error message. > > > >Cc: Liming Gao <liming....@intel.com> > >Cc: Yonghong Zhu <yonghong....@intel.com> > >Contributed-under: TianoCore Contribution Agreement 1.1 > >Signed-off-by: Jaben Carsey <jaben.car...@intel.com> > >--- > > BaseTools/Scripts/PatchCheck.py | 9 ++++++--- > > 1 file changed, 6 insertions(+), 3 deletions(-) > > > >diff --git a/BaseTools/Scripts/PatchCheck.py > >b/BaseTools/Scripts/PatchCheck.py > >index 7b7fba8b7044..96b3cdf1fd8a 100755 > >--- a/BaseTools/Scripts/PatchCheck.py > >+++ b/BaseTools/Scripts/PatchCheck.py > >@@ -1,7 +1,7 @@ > > ## @file > > # Check a patch for various format issues > > # > >-# Copyright (c) 2015 - 2017, Intel Corporation. All rights reserved.<BR> > >+# Copyright (c) 2015 - 2018, Intel Corporation. All rights reserved.<BR> > > # > > # This program and the accompanying materials are licensed and made > > # available under the terms and conditions of the BSD License which > >@@ -528,6 +528,8 @@ class CheckGitCommits: > > print('Checking git commit:', commit) > > patch = self.read_patch_from_git(commit) > > self.ok &= CheckOnePatch(commit, patch).ok > >+ if not commits: > >+ print("Couldn't find commit matching: '{}'".format(rev_spec)) > > > > def read_commit_list_from_git(self, rev_spec, max_count): > > # Run git to get the commit patch > >@@ -536,7 +538,7 @@ class CheckGitCommits: > > cmd.append('--max-count=' + str(max_count)) > > cmd.append(rev_spec) > > out = self.run_git(*cmd) > >- return out.split() > >+ return out.split() if out else [] > > > > def read_patch_from_git(self, commit): > > # Run git to get the commit patch > >@@ -548,7 +550,8 @@ class CheckGitCommits: > > p = subprocess.Popen(cmd, > > stdout=subprocess.PIPE, > > stderr=subprocess.STDOUT) > >- return p.communicate()[0].decode('utf-8', 'ignore') > >+ Result = p.communicate() > >+ return Result[0].decode('utf-8', 'ignore') if Result[0] and > >Result[0].find("fatal")!=0 else None > > > > class CheckOnePatchFile: > > """Performs a patch check for a single file. > >-- > >2.16.2.windows.1 > > > >_______________________________________________ > >edk2-devel mailing list > >edk2-devel@lists.01.org > >https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
