From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from g4t3426.houston.hpe.com (g4t3426.houston.hpe.com [15.241.140.75]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 0687921A6F105 for ; Tue, 6 Jun 2017 15:02:40 -0700 (PDT) Received: from G1W8106.americas.hpqcorp.net (g1w8106.austin.hp.com [16.193.72.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by g4t3426.houston.hpe.com (Postfix) with ESMTPS id A67C884 for ; Tue, 6 Jun 2017 22:03:46 +0000 (UTC) Received: from G9W8668.americas.hpqcorp.net (16.220.49.27) by G1W8106.americas.hpqcorp.net (16.193.72.61) with Microsoft SMTP Server (TLS) id 15.0.1178.4; Tue, 6 Jun 2017 22:03:25 +0000 Received: from G4W10204.americas.hpqcorp.net (2002:10cf:5210::10cf:5210) by G9W8668.americas.hpqcorp.net (2002:10dc:311b::10dc:311b) with Microsoft SMTP Server (TLS) id 15.0.1178.4; Tue, 6 Jun 2017 22:03:24 +0000 Received: from NAM01-BY2-obe.outbound.protection.outlook.com (15.241.52.11) by G4W10204.americas.hpqcorp.net (16.207.82.16) with Microsoft SMTP Server (TLS) id 15.0.1178.4 via Frontend Transport; Tue, 6 Jun 2017 22:03:24 +0000 Received: from CS1PR84MB0024.NAMPRD84.PROD.OUTLOOK.COM (10.162.189.142) by CS1PR84MB0023.NAMPRD84.PROD.OUTLOOK.COM (10.162.189.141) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1157.12; Tue, 6 Jun 2017 22:03:22 +0000 Received: from CS1PR84MB0024.NAMPRD84.PROD.OUTLOOK.COM ([10.162.189.142]) by CS1PR84MB0024.NAMPRD84.PROD.OUTLOOK.COM ([10.162.189.142]) with mapi id 15.01.1157.012; Tue, 6 Jun 2017 22:03:22 +0000 From: "Shah, Tapan" To: "Carsey, Jaben" , "Ni, Ruiyu" , "edk2-devel@lists.01.org" Thread-Topic: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletion Thread-Index: AQHS3p8FvY0wpi4uv0yFy8KbBoSum6IXhMCAgADVC4CAAAMFAIAABn9w Date: Tue, 6 Jun 2017 22:03:22 +0000 Message-ID: References: <20170606082904.195040-1-ruiyu.ni@intel.com>, <734D49CCEBEEF84792F5B80ED585239D5B988FB6@SHSMSX104.ccr.corp.intel.com> <3DAAB008-D40F-48B1-B8AA-0F418CA19FE3@intel.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=hpe.com; x-originating-ip: [15.203.227.5] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; CS1PR84MB0023; 7:UdaajaZ4KjPokMKIaBSSYmBVptB3h+qclm2lgupFd8QdhqmJZKE8kQrnFf4oHthDHPl6ojxN9p3l8tw+w7pCD+1bxq02KStZWXVTg9/q8x/e3u9VMjcRL1kLVMA5OGglvMhsPN9yW92Rwp2rm1Mt+XxAHrSH2qxfzk++cF67GqYN1wfOxvM2Y6PcTUak97+Ye2wRB2G0jWxcpxysGOaHO3o73zKKAvucYeAvmDF2e7mfGnyUh4s+efL2vVoJblHUiDWO8ghCQux6u1WIZjBDsJ0Buzmb2xpTMgAZmKap1OXXwM8Eab/WwPgNWlwTgwkQQvMen20iruqPEpT4jgpgxg== x-ms-traffictypediagnostic: CS1PR84MB0023: x-ms-office365-filtering-correlation-id: 60b23c4e-bd3c-4803-1f18-08d4ad27d933 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254075)(48565401081)(201703131423075)(201703031133081); SRVR:CS1PR84MB0023; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(227479698468861)(162533806227266)(228905959029699); x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(8121501046)(5005006)(93006095)(93001095)(100000703101)(100105400095)(10201501046)(3002001)(6055026)(6041248)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(20161123560025)(20161123555025)(20161123564025)(6072148)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:CS1PR84MB0023; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:CS1PR84MB0023; x-forefront-prvs: 033054F29A x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(6009001)(39410400002)(39400400002)(39860400002)(39450400003)(39840400002)(39850400002)(377454003)(13464003)(77096006)(66066001)(86362001)(229853002)(25786009)(6506006)(33656002)(53546009)(478600001)(50986999)(2900100001)(2501003)(966005)(3660700001)(3280700002)(189998001)(54356999)(5660300001)(6116002)(2906002)(55016002)(3846002)(76176999)(93886004)(102836003)(6246003)(122556002)(74316002)(2950100002)(6306002)(14454004)(305945005)(81166006)(7736002)(8936002)(8676002)(7696004)(53936002)(6436002)(9686003)(38730400002); DIR:OUT; SFP:1102; SCL:1; SRVR:CS1PR84MB0023; H:CS1PR84MB0024.NAMPRD84.PROD.OUTLOOK.COM; FPR:; SPF:None; MLV:sfv; LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM MIME-Version: 1.0 X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Jun 2017 22:03:22.4447 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 105b2061-b669-4b31-92ac-24d304d195dc X-MS-Exchange-Transport-CrossTenantHeadersStamped: CS1PR84MB0023 X-OriginatorOrg: hpe.com Subject: Re: [PATCH] ShellPkg/alias: Return status for alias deletion X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 06 Jun 2017 22:02:40 -0000 Content-Language: en-US Content-Type: text/plain; charset="iso-2022-jp" Content-Transfer-Encoding: quoted-printable Instead of using STR_GEN_ERR_UK here, it would be better to define a new s= tring token as "alias: '' does not exist." > >> + ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_ERR_UK)= , > >> gShellLevel3HiiHandle, L"alias", Status); Reviewed-by: Tapan Shah -----Original Message----- From: Carsey, Jaben [mailto:jaben.carsey@intel.com]=20 Sent: Tuesday, June 06, 2017 4:37 PM To: Ni, Ruiyu ; edk2-devel@lists.01.org Cc: Shah, Tapan Subject: RE: [edk2] [PATCH] ShellPkg/alias: Return status for alias deletio= n I am good with more clear error information coming to the user. Reviewed-by: Jaben Carsey > -----Original Message----- > From: Ni, Ruiyu > Sent: Tuesday, June 06, 2017 2:26 PM > To: Ni, Ruiyu ; edk2-devel@lists.01.org > Cc: Tapan Shah ; Carsey, Jaben > > Subject: Re: [edk2] [PATCH] ShellPkg/alias: Return status for alias delet= ion > Importance: High >=20 > Including Jaben for review. >=20 > Send from mobile phone >=20 > > =1B$B:_=1B(B 2017=1B$BG/=1B(B6=1B$B7n=1B(B6=1B$BF|!$2<8a=1B(B4:43=1B$B!= $=1B(BNi, Ruiyu =1B$B > > > Including Tapan. > > > > Thanks/Ray > > > >> -----Original Message----- > >> From: edk2-devel [mailto:edk2-devel-bounces@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 deleti= on > >> > >> Contributed-under: TianoCore Contribution Agreement 1.0 > >> Signed-off-by: Ruiyu Ni > >> Cc: Tapan Shah > >> --- > >> 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.
> >> - Copyright (c) 2009 - 2016, Intel Corporation. All rights reserved. =
> >> + Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved. > >> +
> >> This program and the accompanying materials > >> are licensed and made available under the terms and conditions of th= e > BSD > >> License > >> which accompanies this distribution. The full text of the license m= ay be > found > >> at @@ -151,6 +151,15 @@ ShellCommandRunAlias ( > >> // delete an alias > >> // > >> Status =3D gEfiShellProtocol->SetAlias(Param1, NULL, TRUE, FALSE= ); > >> + if (EFI_ERROR(Status)) { > >> + if (Status =3D=3D EFI_ACCESS_DENIED) { > >> + ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_ERR_AD)= , > >> gShellLevel3HiiHandle, L"alias"); > >> + ShellStatus =3D SHELL_ACCESS_DENIED; > >> + } else { > >> + ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_ERR_UK)= , > >> gShellLevel3HiiHandle, L"alias", Status); > >> + ShellStatus =3D SHELL_DEVICE_ERROR; > >> + } > >> + } > >> } else if (ShellCommandLineGetCount(Package) =3D=3D 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