I would suggest to keep FILE/DIR tokens as-is to avoid more code changes. New token for alias... to match the wording it can be "%s: <alias_name> not found."
-----Original Message----- From: Carsey, Jaben [mailto:[email protected]] Sent: Thursday, June 08, 2017 11:16 AM To: Shah, Tapan <[email protected]>; Ni, Ruiyu <[email protected]> Cc: [email protected] Subject: RE: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion Wonder if we should refactor FILE_NOT_FOUND, DIR_NOT_FOUND, and this into "%s '%s' not found"? > -----Original Message----- > From: Shah, Tapan [mailto:[email protected]] > Sent: Thursday, June 08, 2017 9:14 AM > To: Carsey, Jaben <[email protected]>; Ni, Ruiyu > <[email protected]> > Cc: [email protected] > Subject: RE: [edk2] [PATCH] ShellPkg/alias: Return status for alias > deletion > Importance: High > > Correct. > > STR_GEN_ERR_NOT_FOUND should be "alias: '<alias_name>' does not > exist." > > > -----Original Message----- > From: Carsey, Jaben [mailto:[email protected]] > Sent: Thursday, June 08, 2017 11:10 AM > To: Shah, Tapan <[email protected]>; Ni, Ruiyu <[email protected]> > Cc: [email protected] > Subject: RE: [edk2] [PATCH] ShellPkg/alias: Return status for alias > deletion > > 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:[email protected]] > > Sent: Thursday, June 08, 2017 9:01 AM > > To: Ni, Ruiyu <[email protected]> > > Cc: Carsey, Jaben <[email protected]>; [email protected] > > 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:[email protected]] > > Sent: Wednesday, June 07, 2017 5:14 PM > > To: Shah, Tapan <[email protected]> > > Cc: Carsey, Jaben <[email protected]>; [email protected] > > 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 <[email protected]> > 写 > > 道： > > > > > > 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' <[email protected]>; Ni, Ruiyu > > > <[email protected]>; [email protected] > > > 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 <[email protected]> > > > > > > -----Original Message----- > > > From: Carsey, Jaben [mailto:[email protected]] > > > Sent: Tuesday, June 06, 2017 4:37 PM > > > To: Ni, Ruiyu <[email protected]>; [email protected] > > > Cc: Shah, Tapan <[email protected]> > > > 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 <[email protected]> > > > > > >> -----Original Message----- > > >> From: Ni, Ruiyu > > >> Sent: Tuesday, June 06, 2017 2:26 PM > > >> To: Ni, Ruiyu <[email protected]>; [email protected] > > >> Cc: Tapan Shah <[email protected]>; Carsey, Jaben > > >> <[email protected]> > > >> 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 <[email protected]> 写道 > ： > > >>> > > >>> Including Tapan. > > >>> > > >>> Thanks/Ray > > >>> > > >>>> -----Original Message----- > > >>>> From: edk2-devel [mailto:[email protected]] On > > >>>> Behalf Of > > >> Ruiyu > > >>>> Ni > > >>>> Sent: Tuesday, June 6, 2017 4:29 PM > > >>>> To: [email protected] > > >>>> Subject: [edk2] [PATCH] ShellPkg/alias: Return status for alias > > >>>> deletion > > >>>> > > >>>> Contributed-under: TianoCore Contribution Agreement 1.0 > > >>>> Signed-off-by: Ruiyu Ni <[email protected]> > > >>>> Cc: Tapan Shah <[email protected]> > > >>>> --- > > >>>> 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 > > >>>> [email protected] > > >>>> https://lists.01.org/mailman/listinfo/edk2-devel > _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
