Ray, I don't see your check-in includes the change I proposed below. Tapan
-----Original Message----- From: Shah, Tapan Sent: Tuesday, June 06, 2017 5:03 PM To: 'Carsey, Jaben' <jaben.car...@intel.com>; Ni, Ruiyu <ruiyu...@intel.com>; edk2-devel@lists.01.org Subject: RE: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion Instead of using STR_GEN_ERR_UK here, it would be better to define a new string token as "alias: '<alias_name>' does not exist." > >> + ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN > >> + (STR_GEN_ERR_UK), > >> gShellLevel3HiiHandle, L"alias", Status); Reviewed-by: Tapan Shah <tapands...@hpe.com> -----Original Message----- From: Carsey, Jaben [mailto:jaben.car...@intel.com] Sent: Tuesday, June 06, 2017 4:37 PM To: Ni, Ruiyu <ruiyu...@intel.com>; edk2-devel@lists.01.org Cc: Shah, Tapan <tapands...@hpe.com> Subject: RE: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion I am good with more clear error information coming to the user. Reviewed-by: Jaben Carsey <jaben.car...@intel.com> > -----Original Message----- > From: Ni, Ruiyu > Sent: Tuesday, June 06, 2017 2:26 PM > To: Ni, Ruiyu <ruiyu...@intel.com>; edk2-devel@lists.01.org > Cc: Tapan Shah <tapands...@hpe.com>; Carsey, Jaben > <jaben.car...@intel.com> > Subject: Re: [edk2] [PATCH] ShellPkg/alias: Return status for alias > deletion > Importance: High > > Including Jaben for review. > > Send from mobile phone > > > 在 2017年6月6日,下午4:43,Ni, Ruiyu <ruiyu...@intel.com> 写道: > > > > Including Tapan. > > > > Thanks/Ray > > > >> -----Original Message----- > >> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf > >> Of > Ruiyu > >> Ni > >> Sent: Tuesday, June 6, 2017 4:29 PM > >> To: edk2-devel@lists.01.org > >> Subject: [edk2] [PATCH] ShellPkg/alias: Return status for alias > >> deletion > >> > >> Contributed-under: TianoCore Contribution Agreement 1.0 > >> Signed-off-by: Ruiyu Ni <ruiyu...@intel.com> > >> Cc: Tapan Shah <tapands...@hpe.com> > >> --- > >> ShellPkg/Library/UefiShellLevel3CommandsLib/Alias.c | 11 > >> ++++++++++- > >> 1 file changed, 10 insertions(+), 1 deletion(-) > >> > >> diff --git a/ShellPkg/Library/UefiShellLevel3CommandsLib/Alias.c > >> b/ShellPkg/Library/UefiShellLevel3CommandsLib/Alias.c > >> index 5a3b7e5913..7277bd4b90 100644 > >> --- a/ShellPkg/Library/UefiShellLevel3CommandsLib/Alias.c > >> +++ b/ShellPkg/Library/UefiShellLevel3CommandsLib/Alias.c > >> @@ -2,7 +2,7 @@ > >> Main file for Alias shell level 3 function. > >> > >> (C) Copyright 2015 Hewlett-Packard Development Company, L.P.<BR> > >> - Copyright (c) 2009 - 2016, Intel Corporation. All rights > >> reserved. <BR> > >> + Copyright (c) 2009 - 2017, 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 accompanies this distribution. The full text of the > >> license may be > found > >> at @@ -151,6 +151,15 @@ ShellCommandRunAlias ( > >> // delete an alias > >> // > >> Status = gEfiShellProtocol->SetAlias(Param1, NULL, TRUE, > >> FALSE); > >> + if (EFI_ERROR(Status)) { > >> + if (Status == EFI_ACCESS_DENIED) { > >> + ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN > >> + (STR_GEN_ERR_AD), > >> gShellLevel3HiiHandle, L"alias"); > >> + ShellStatus = SHELL_ACCESS_DENIED; > >> + } else { > >> + ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN > >> + (STR_GEN_ERR_UK), > >> gShellLevel3HiiHandle, L"alias", Status); > >> + ShellStatus = SHELL_DEVICE_ERROR; > >> + } > >> + } > >> } else if (ShellCommandLineGetCount(Package) == 3) { > >> // > >> // must be adding an alias > >> -- > >> 2.12.2.windows.2 > >> > >> _______________________________________________ > >> 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