Tapa,
Is your thinking like this:
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 if (Status == EFI_NOT_FOUND) {
ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_ERR_NOT_FOUND),
gShellLevel3HiiHandle, L"alias");
ShellStatus = SHELL_NOT_FOUND;
} else { ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_ERR_UK),
gShellLevel3HiiHandle, L"alias", Status);
ShellStatus = SHELL_DEVICE_ERROR;
}
}
Then add a STR_GEN_ERR_NOT_FOUND string?
> -----Original Message-----
> From: Shah, Tapan [mailto:tapands...@hpe.com]
> Sent: Thursday, June 08, 2017 9:01 AM
> To: Ni, Ruiyu <ruiyu...@intel.com>
> Cc: Carsey, Jaben <jaben.car...@intel.com>; edk2-devel@lists.01.org
> Subject: RE: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion
> Importance: High
>
> EFI_NOT_FOUND should be handled with the token I mentioned for a clear
> message. All other errors should be printed using STR_GEN_ERR_UK.
>
> -----Original Message-----
> From: Ni, Ruiyu [mailto:ruiyu...@intel.com]
> Sent: Wednesday, June 07, 2017 5:14 PM
> To: Shah, Tapan <tapands...@hpe.com>
> Cc: Carsey, Jaben <jaben.car...@intel.com>; edk2-devel@lists.01.org
> Subject: Re: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion
>
> I did think about changing as you suggested. But other errors also may be
> DEVICE_ERROR in case the NV storage has some problem. So I did not change
> it.
> Sorry for not replying to you before checking in the code.
>
> Send from mobile phone
>
> > 在 2017年6月7日，下午11:35，Shah, Tapan <tapands...@hpe.com> 写
> 道：
> >
> > 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
