ShellPkg/Application/Shell/ShellParametersProtocol.h | 2 ++ ShellPkg/Application/Shell/ShellParametersProtocol.c | 2 ++ 2 files changed, 4 insertions(+)
Complete the list of values the functios ParseCommandLineToArgs() and UpdateArgcArgv() can return. The documentation was found to be incomplete while reviewing BZ#2395: https://edk2.groups.io/g/devel/message/51512 v2: - Shortened patch #1 subject to fit 72 chars - Added Zhichao Gao R-b tag Regards, Phil. Philippe Mathieu-Daude (2): ShellPkg: Document ParseCommandLineToArgs returns EFI_INVALID_PARAMETER ShellPkg: Document UpdateArgcArgv returns EFI_INVALID_PARAMETER ShellPkg/Application/Shell/ShellParametersProtocol.h | 2 ++ ShellPkg/Application/Shell/ShellParametersProtocol.c | 2 ++ 2 files changed, 4 insertions(+) -- 2.21.0 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#52420): https://edk2.groups.io/g/devel/message/52420 Mute This Topic: https://groups.io/mt/68831521/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Sorry for late response: > -----Original Message----- > From: Philippe Mathieu-Daude <philmd@redhat.com> > Sent: Thursday, December 19, 2019 10:10 PM > To: devel@edk2.groups.io > Cc: Ni, Ray <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; > Philippe Mathieu-Daude <philmd@redhat.com> > Subject: [PATCH v2 0/2] ShellPkg: Document the use of > EFI_INVALID_PARAMETER by two functions > > Complete the list of values the functios ParseCommandLineToArgs() and > UpdateArgcArgv() can return. > > The documentation was found to be incomplete while reviewing BZ#2395: > https://edk2.groups.io/g/devel/message/51512 > > v2: > - Shortened patch #1 subject to fit 72 chars The subject should be less than 72 chars (do not include 72 chars). Thanks, Zhichao > - Added Zhichao Gao R-b tag > > Regards, > > Phil. > > Philippe Mathieu-Daude (2): > ShellPkg: Document ParseCommandLineToArgs returns > EFI_INVALID_PARAMETER > ShellPkg: Document UpdateArgcArgv returns EFI_INVALID_PARAMETER > > ShellPkg/Application/Shell/ShellParametersProtocol.h | 2 ++ > ShellPkg/Application/Shell/ShellParametersProtocol.c | 2 ++ > 2 files changed, 4 insertions(+) > > -- > 2.21.0 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#52954): https://edk2.groups.io/g/devel/message/52954 Mute This Topic: https://groups.io/mt/68831521/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Hi Zhichao, On 1/7/20 8:01 AM, Gao, Zhichao wrote: > Sorry for late response: > >> -----Original Message----- >> From: Philippe Mathieu-Daude <philmd@redhat.com> >> Sent: Thursday, December 19, 2019 10:10 PM >> To: devel@edk2.groups.io >> Cc: Ni, Ray <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; >> Philippe Mathieu-Daude <philmd@redhat.com> >> Subject: [PATCH v2 0/2] ShellPkg: Document the use of >> EFI_INVALID_PARAMETER by two functions >> >> Complete the list of values the functios ParseCommandLineToArgs() and >> UpdateArgcArgv() can return. >> >> The documentation was found to be incomplete while reviewing BZ#2395: >> https://edk2.groups.io/g/devel/message/51512 >> >> v2: >> - Shortened patch #1 subject to fit 72 chars > > The subject should be less than 72 chars (do not include 72 chars). What is the rationale for this restriction? Some tool limitation? -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#52955): https://edk2.groups.io/g/devel/message/52955 Mute This Topic: https://groups.io/mt/68831521/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
> -----Original Message----- > From: Philippe Mathieu-Daudé [mailto:philmd@redhat.com] > Sent: Tuesday, January 7, 2020 3:10 PM > To: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io > Cc: Ni, Ray <ray.ni@intel.com> > Subject: Re: [PATCH v2 0/2] ShellPkg: Document the use of > EFI_INVALID_PARAMETER by two functions > > Hi Zhichao, > > On 1/7/20 8:01 AM, Gao, Zhichao wrote: > > Sorry for late response: > > > >> -----Original Message----- > >> From: Philippe Mathieu-Daude <philmd@redhat.com> > >> Sent: Thursday, December 19, 2019 10:10 PM > >> To: devel@edk2.groups.io > >> Cc: Ni, Ray <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; > >> Philippe Mathieu-Daude <philmd@redhat.com> > >> Subject: [PATCH v2 0/2] ShellPkg: Document the use of > >> EFI_INVALID_PARAMETER by two functions > >> > >> Complete the list of values the functios ParseCommandLineToArgs() and > >> UpdateArgcArgv() can return. > >> > >> The documentation was found to be incomplete while reviewing BZ#2395: > >> https://edk2.groups.io/g/devel/message/51512 > >> > >> v2: > >> - Shortened patch #1 subject to fit 72 chars > > > > The subject should be less than 72 chars (do not include 72 chars). > > What is the rationale for this restriction? Some tool limitation? Yes. There is a patch check tool in BaseTools\Scripts\ PatchCheck.py. And the patch #1 can't pass the check. Thanks, Zhichao -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#52957): https://edk2.groups.io/g/devel/message/52957 Mute This Topic: https://groups.io/mt/68831521/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Hi Zhichao, On 1/7/20 8:37 AM, Gao, Zhichao wrote: >> -----Original Message----- >> From: Philippe Mathieu-Daudé [mailto:philmd@redhat.com] >> Sent: Tuesday, January 7, 2020 3:10 PM >> To: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io >> Cc: Ni, Ray <ray.ni@intel.com> >> Subject: Re: [PATCH v2 0/2] ShellPkg: Document the use of >> EFI_INVALID_PARAMETER by two functions >> >> On 1/7/20 8:01 AM, Gao, Zhichao wrote: >>>> -----Original Message----- >>>> From: Philippe Mathieu-Daude <philmd@redhat.com> >>>> Sent: Thursday, December 19, 2019 10:10 PM >>>> To: devel@edk2.groups.io >>>> Cc: Ni, Ray <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; >>>> Philippe Mathieu-Daude <philmd@redhat.com> >>>> Subject: [PATCH v2 0/2] ShellPkg: Document the use of >>>> EFI_INVALID_PARAMETER by two functions >>>> >>>> Complete the list of values the functios ParseCommandLineToArgs() and >>>> UpdateArgcArgv() can return. >>>> >>>> The documentation was found to be incomplete while reviewing BZ#2395: >>>> https://edk2.groups.io/g/devel/message/51512 >>>> >>>> v2: >>>> - Shortened patch #1 subject to fit 72 chars >>> >>> The subject should be less than 72 chars (do not include 72 chars). >> >> What is the rationale for this restriction? Some tool limitation? > > Yes. There is a patch check tool in BaseTools\Scripts\ PatchCheck.py. And the patch #1 can't pass the check. Since the patch "BaseTools/PatchCheck.py: Ignore CR and LF characters in subject length" got merged as commit 2649a735b24, can you retry to merge this patch? Thanks, Phil. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#53425): https://edk2.groups.io/g/devel/message/53425 Mute This Topic: https://groups.io/mt/68831521/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Hi, I have tried with the commit. It pass the check. Thanks, Zhichao > -----Original Message----- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Philippe Mathieu-Daudé > Sent: Tuesday, January 21, 2020 7:24 PM > To: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io > Cc: Ni, Ray <ray.ni@intel.com> > Subject: Re: [edk2-devel] [PATCH v2 0/2] ShellPkg: Document the use of > EFI_INVALID_PARAMETER by two functions > > Hi Zhichao, > > On 1/7/20 8:37 AM, Gao, Zhichao wrote: > >> -----Original Message----- > >> From: Philippe Mathieu-Daudé [mailto:philmd@redhat.com] > >> Sent: Tuesday, January 7, 2020 3:10 PM > >> To: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io > >> Cc: Ni, Ray <ray.ni@intel.com> > >> Subject: Re: [PATCH v2 0/2] ShellPkg: Document the use of > >> EFI_INVALID_PARAMETER by two functions > >> > >> On 1/7/20 8:01 AM, Gao, Zhichao wrote: > >>>> -----Original Message----- > >>>> From: Philippe Mathieu-Daude <philmd@redhat.com> > >>>> Sent: Thursday, December 19, 2019 10:10 PM > >>>> To: devel@edk2.groups.io > >>>> Cc: Ni, Ray <ray.ni@intel.com>; Gao, Zhichao > >>>> <zhichao.gao@intel.com>; Philippe Mathieu-Daude <philmd@redhat.com> > >>>> Subject: [PATCH v2 0/2] ShellPkg: Document the use of > >>>> EFI_INVALID_PARAMETER by two functions > >>>> > >>>> Complete the list of values the functios ParseCommandLineToArgs() > >>>> and > >>>> UpdateArgcArgv() can return. > >>>> > >>>> The documentation was found to be incomplete while reviewing BZ#2395: > >>>> https://edk2.groups.io/g/devel/message/51512 > >>>> > >>>> v2: > >>>> - Shortened patch #1 subject to fit 72 chars > >>> > >>> The subject should be less than 72 chars (do not include 72 chars). > >> > >> What is the rationale for this restriction? Some tool limitation? > > > > Yes. There is a patch check tool in BaseTools\Scripts\ PatchCheck.py. And the > patch #1 can't pass the check. > > Since the patch "BaseTools/PatchCheck.py: Ignore CR and LF characters in > subject length" got merged as commit 2649a735b24, can you retry to merge this > patch? > > Thanks, > > Phil. > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#53592): https://edk2.groups.io/g/devel/message/53592 Mute This Topic: https://groups.io/mt/68831521/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
On 1/31/20 6:31 AM, Gao, Zhichao wrote: > Hi, > > I have tried with the commit. It pass the check. Good news, thanks for checking :) Is there anything else I should do to get this series applied? Thanks, Phil. >> -----Original Message----- >> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >> Philippe Mathieu-Daudé >> Sent: Tuesday, January 21, 2020 7:24 PM >> To: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io >> Cc: Ni, Ray <ray.ni@intel.com> >> Subject: Re: [edk2-devel] [PATCH v2 0/2] ShellPkg: Document the use of >> EFI_INVALID_PARAMETER by two functions >> >> Hi Zhichao, >> >> On 1/7/20 8:37 AM, Gao, Zhichao wrote: >>>> -----Original Message----- >>>> From: Philippe Mathieu-Daudé [mailto:philmd@redhat.com] >>>> Sent: Tuesday, January 7, 2020 3:10 PM >>>> To: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io >>>> Cc: Ni, Ray <ray.ni@intel.com> >>>> Subject: Re: [PATCH v2 0/2] ShellPkg: Document the use of >>>> EFI_INVALID_PARAMETER by two functions >>>> >>>> On 1/7/20 8:01 AM, Gao, Zhichao wrote: >>>>>> -----Original Message----- >>>>>> From: Philippe Mathieu-Daude <philmd@redhat.com> >>>>>> Sent: Thursday, December 19, 2019 10:10 PM >>>>>> To: devel@edk2.groups.io >>>>>> Cc: Ni, Ray <ray.ni@intel.com>; Gao, Zhichao >>>>>> <zhichao.gao@intel.com>; Philippe Mathieu-Daude <philmd@redhat.com> >>>>>> Subject: [PATCH v2 0/2] ShellPkg: Document the use of >>>>>> EFI_INVALID_PARAMETER by two functions >>>>>> >>>>>> Complete the list of values the functios ParseCommandLineToArgs() >>>>>> and >>>>>> UpdateArgcArgv() can return. >>>>>> >>>>>> The documentation was found to be incomplete while reviewing BZ#2395: >>>>>> https://edk2.groups.io/g/devel/message/51512 >>>>>> >>>>>> v2: >>>>>> - Shortened patch #1 subject to fit 72 chars >>>>> >>>>> The subject should be less than 72 chars (do not include 72 chars). >>>> >>>> What is the rationale for this restriction? Some tool limitation? >>> >>> Yes. There is a patch check tool in BaseTools\Scripts\ PatchCheck.py. And the >> patch #1 can't pass the check. >> >> Since the patch "BaseTools/PatchCheck.py: Ignore CR and LF characters in >> subject length" got merged as commit 2649a735b24, can you retry to merge this >> patch? >> >> Thanks, >> >> Phil. >> >> >> > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#53787): https://edk2.groups.io/g/devel/message/53787 Mute This Topic: https://groups.io/mt/68831521/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
On 2/5/20 12:04 AM, Philippe Mathieu-Daudé wrote: > On 1/31/20 6:31 AM, Gao, Zhichao wrote: >> Hi, >> >> I have tried with the commit. It pass the check. > > Good news, thanks for checking :) > > Is there anything else I should do to get this series applied? I see these patches were merged as ddb7050c6843 and 7f9e354a01d7 (my name mojibaked unfortunately). Thank to the anonymous maintainer who merged it! > Thanks, > > Phil. > >>> -----Original Message----- >>> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >>> Philippe Mathieu-Daudé >>> Sent: Tuesday, January 21, 2020 7:24 PM >>> To: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io >>> Cc: Ni, Ray <ray.ni@intel.com> >>> Subject: Re: [edk2-devel] [PATCH v2 0/2] ShellPkg: Document the use of >>> EFI_INVALID_PARAMETER by two functions >>> >>> Hi Zhichao, >>> >>> On 1/7/20 8:37 AM, Gao, Zhichao wrote: >>>>> -----Original Message----- >>>>> From: Philippe Mathieu-Daudé [mailto:philmd@redhat.com] >>>>> Sent: Tuesday, January 7, 2020 3:10 PM >>>>> To: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io >>>>> Cc: Ni, Ray <ray.ni@intel.com> >>>>> Subject: Re: [PATCH v2 0/2] ShellPkg: Document the use of >>>>> EFI_INVALID_PARAMETER by two functions >>>>> >>>>> On 1/7/20 8:01 AM, Gao, Zhichao wrote: >>>>>>> -----Original Message----- >>>>>>> From: Philippe Mathieu-Daude <philmd@redhat.com> >>>>>>> Sent: Thursday, December 19, 2019 10:10 PM >>>>>>> To: devel@edk2.groups.io >>>>>>> Cc: Ni, Ray <ray.ni@intel.com>; Gao, Zhichao >>>>>>> <zhichao.gao@intel.com>; Philippe Mathieu-Daude <philmd@redhat.com> >>>>>>> Subject: [PATCH v2 0/2] ShellPkg: Document the use of >>>>>>> EFI_INVALID_PARAMETER by two functions >>>>>>> >>>>>>> Complete the list of values the functios ParseCommandLineToArgs() >>>>>>> and >>>>>>> UpdateArgcArgv() can return. >>>>>>> >>>>>>> The documentation was found to be incomplete while reviewing >>>>>>> BZ#2395: >>>>>>> https://edk2.groups.io/g/devel/message/51512 >>>>>>> >>>>>>> v2: >>>>>>> - Shortened patch #1 subject to fit 72 chars >>>>>> >>>>>> The subject should be less than 72 chars (do not include 72 chars). >>>>> >>>>> What is the rationale for this restriction? Some tool limitation? >>>> >>>> Yes. There is a patch check tool in BaseTools\Scripts\ >>>> PatchCheck.py. And the >>> patch #1 can't pass the check. >>> >>> Since the patch "BaseTools/PatchCheck.py: Ignore CR and LF characters in >>> subject length" got merged as commit 2649a735b24, can you retry to >>> merge this >>> patch? >>> >>> Thanks, >>> >>> Phil. >>> >>> >>> >> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#54200): https://edk2.groups.io/g/devel/message/54200 Mute This Topic: https://groups.io/mt/68831521/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2024 Red Hat, Inc.