Re: [edk2-devel] [PATCH 1/1] ShellPkg: Improve "ping" output by adding equals sign after time

["Michael D Kinney" <michael.d.kinney@intel.com>]

Hi Ard,

There have been some communication issues on status with Liming.  Miki and I think those are resolved now.

We are wanting to move to hard freeze ASAP to keep as close as possible to the original planned
release date. We will discuss with Liming when he is back online later today.

I recommend holding off on merging any non-critical changes until there is updated status from
Liming.

Thanks,

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ard Biesheuvel
> Sent: Tuesday, February 21, 2023 6:40 AM
> To: devel@edk2.groups.io; Gao, Zhichao <zhichao.gao@intel.com>
> Cc: Rebecca Cran <rebecca@quicinc.com>; Ni, Ray <ray.ni@intel.com>
> Subject: Re: [edk2-devel] [PATCH 1/1] ShellPkg: Improve "ping" output by adding equals sign after time
> 
> On Tue, 21 Feb 2023 at 14:58, Gao, Zhichao <zhichao.gao@intel.com> wrote:
> >
> > Sorry, I forgot this patch. Now it is the hard freeze for edk2. Would create the PR after edk2 202302 release.
> >
> 
> The hard freeze has been deferred until early march, so you can go
> ahead and merge this patch.
> 
> --
> Ard.
> 
> > > -----Original Message-----
> > > From: Rebecca Cran <rebecca@quicinc.com>
> > > Sent: Tuesday, February 21, 2023 8:45 AM
> > > To: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io; Ni, Ray
> > > <ray.ni@intel.com>
> > > Subject: Re: [edk2-devel] [PATCH 1/1] ShellPkg: Improve "ping" output by
> > > adding equals sign after time
> > >
> > > Could someone commit this please?
> > >
> > > On 12/5/22 19:17, Gao, Zhichao wrote:
> > > > Reviewed-by: Zhichao Gao <zhichao.gao@intel.com>
> > > >
> > > > Thanks,
> > > > Zhichao
> > > >
> > > >> -----Original Message-----
> > > >> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > >> Rebecca Cran
> > > >> Sent: Friday, December 2, 2022 5:50 AM
> > > >> To: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com>; Gao, Zhichao
> > > >> <zhichao.gao@intel.com>
> > > >> Cc: Rebecca Cran <rebecca@quicinc.com>
> > > >> Subject: [edk2-devel] [PATCH 1/1] ShellPkg: Improve "ping" output by
> > > >> adding equals sign after time
> > > >>
> > > >> The output of the "ping" command shows the time without a space
> > > >> between the label and the value. e.g.:
> > > >>
> > > >> 20 bytes from 192.168.0.1 : icmp_seq=1 ttl=1 time1~2ms
> > > >>
> > > >> Improve the readability and consistency by adding an equals sign for
> > > >> the time
> > > >> value:
> > > >>
> > > >> 20 bytes from 192.168.0.1 : icmp_seq=1 ttl=1 time=1~2ms
> > > >>
> > > >> Signed-off-by: Rebecca Cran <rebecca@quicinc.com>
> > > >> ---
> > > >>
> > > >>
> > > ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Comm
> > > >> andsLib.uni | 2 +-
> > > >>
> > > ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Comm
> > > >> andsLib.uni | 2 +-
> > > >>   2 files changed, 2 insertions(+), 2 deletions(-)
> > > >>
> > > >> diff --git
> > > >>
> > > a/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com
> > > >> mandsLib.uni
> > > >>
> > > b/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com
> > > >> mandsLib.uni
> > > >> index 73cf96d9e614..37fd4a11929c 100644
> > > >> ---
> > > >>
> > > a/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com
> > > >> mandsLib.uni
> > > >> +++
> > > >>
> > > b/ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com
> > > >> +++ mandsLib.uni
> > > >> @@ -52,7 +52,7 @@
> > > >>   #string STR_PING_NOROUTE_FOUND       #language en-US "There is no
> > > >> route to the destination '%B%s%N' from the source '%B%s%N' was
> > > >> found.\r\n"
> > > >>   #string STR_PING_START               #language en-US "Ping %s %d data
> > > >> bytes.\r\n"
> > > >>   #string STR_PING_TIMEOUT             #language en-US "Echo request
> > > >> sequence %d timeout.\r\n"
> > > >> -#string STR_PING_REPLY_INFO          #language en-US "%d bytes from %s :
> > > >> icmp_seq=%d ttl=%d time%d~%dms\r\n"
> > > >> +#string STR_PING_REPLY_INFO          #language en-US "%d bytes
> > > from %s :
> > > >> icmp_seq=%d ttl=%d time=%d~%dms\r\n"
> > > >>   #string STR_PING_STAT                #language en-US "\n%d packets
> > > >> transmitted, %d received, %d%% packet loss, time %dms\r\n"
> > > >>   #string STR_PING_RTT                 #language en-US "\nRtt(round trip time)
> > > >> min=%d~%dms max=%d~%dms avg=%d~%dms\r\n"
> > > >>
> > > >> diff --git
> > > >>
> > > a/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com
> > > >> mandsLib.uni
> > > >>
> > > b/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com
> > > >> mandsLib.uni
> > > >> index 30913ae5b583..cb9d39a33fbe 100644
> > > >> ---
> > > >>
> > > a/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com
> > > >> mandsLib.uni
> > > >> +++
> > > >>
> > > b/ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com
> > > >> +++ mandsLib.uni
> > > >> @@ -33,7 +33,7 @@
> > > >>   #string STR_PING6_NOSOURCE_INDOMAIN        #language en-US  "No
> > > >> sources in %s's multicast domain.\r\n"
> > > >>   #string STR_PING6_START                    #language en-US  "Ping %s %d data
> > > >> bytes\r\n"
> > > >>   #string STR_PING6_TIMEOUT                  #language en-US  "Echo request
> > > >> sequence %d timeout.\r\n"
> > > >> -#string STR_PING6_REPLY_INFO               #language en-US  "%d bytes
> > > >> from %s : icmp_seq=%d ttl=%d time%d~%dms\r\n"
> > > >> +#string STR_PING6_REPLY_INFO               #language en-US  "%d bytes
> > > >> from %s : icmp_seq=%d ttl=%d time=%d~%dms\r\n"
> > > >>   #string STR_PING6_STAT                     #language en-US  "\n%d packets
> > > >> transmitted, %d received, %d%% packet loss, time %dms\r\n"
> > > >>   #string STR_PING6_RTT                      #language en-US  "\nRtt(round trip
> > > time)
> > > >> min=%d~%dms max=%d~%dms avg=%d~%dms\r\n"
> > > >>
> > > >> --
> > > >> 2.30.2
> > > >>
> > > >>
> > > >>
> > > >>
> > > >>
> > > >
> >
> >
> >
> >
> >
> 
> 
> 
>