From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [63.128.21.124]) by mx.groups.io with SMTP id smtpd.web12.1718.1610703328652721414 for ; Fri, 15 Jan 2021 01:35:28 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=czOgS92p; spf=pass (domain: redhat.com, ip: 63.128.21.124, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1610703327; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=RxxndjBU7q15wDmCWDEiJPXlfesS9zB1jKU/j7pvwDk=; b=czOgS92psH5f5t0HF9WDS5xqoIJgPl/0i120bd9SZVTR8NK6AekL1eIbWd9CfKCIU5KP/s kOnFsl8+jlSHnJzsBWXDwx9vS8qdElPx83JMcUpZjo911V60zVu/MUKVsn//JBECgnFKP0 nMnGrZVSwhS7omOckj2HLwWXG7OhB6c= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-160-kdncGG-_M-Gfu4VZrM9TnA-1; Fri, 15 Jan 2021 04:35:24 -0500 X-MC-Unique: kdncGG-_M-Gfu4VZrM9TnA-1 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 5F68F107ACFE; Fri, 15 Jan 2021 09:35:21 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-223.ams2.redhat.com [10.36.112.223]) by smtp.corp.redhat.com (Postfix) with ESMTP id 12DF26F44C; Fri, 15 Jan 2021 09:35:16 +0000 (UTC) Subject: Re: [edk2-devel] [edk2-platforms PATCH 0/3] add OrderedCollectionLib class resolution To: Ard Biesheuvel , devel@edk2.groups.io Cc: Agyeman Prince , Ard Biesheuvel , Chasel Chiu , Deepika Kethi Reddy , Jeremy Soller , Kathappan Esakkithevar , Kelly Steele , Leif Lindholm , Marcin Wojtas , Meenakshi Aggarwal , Michael D Kinney , Nate DeSimone , =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= , Rangasai V Chaganty , Thomas Abraham , Wenyi Xie , Yi Qian , Zailiang Sun References: <20210113082843.9095-1-lersek@redhat.com> From: "Laszlo Ersek" Message-ID: Date: Fri, 15 Jan 2021 10:35:16 +0100 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit On 01/15/21 10:22, Ard Biesheuvel wrote: > On 1/15/21 9:50 AM, Laszlo Ersek wrote: >> Hi All, >> >> sorry about the message; it's been only two days indeed. >> >> This patch series is trivial, it adds one lib class resolution to each >> of 22 platforms in edk2-platforms. >> >> I build-tested the platforms modified by this change, so I'm planning to >> push the series to edk2-platforms one week after posting it -- even if I >> get no feedback. Thus, the reason I'm sending this email isn't really to >> ping or urge reviewers, it's more like announcing my plan. >> >> Thanks, >> Laszlo >> >> >> On 01/13/21 09:28, Laszlo Ersek wrote: >>> The edk2 patch >>> >>> ShellPkg/ShellCommandLib: add ShellSortFileList >>> >>> for will make the >>> >>> ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf >>> >>> instance dependent on the OrderedCollectionLib class. Because the shell >>> binaries in several silicon and platform DSCs in edk2-platforms consume >>> the above UefiShellCommandLib instance, resolve the OrderedCollectionLib >>> class in those DSCs. >>> >>> Cc: Agyeman Prince >>> Cc: Ard Biesheuvel >>> Cc: Chasel Chiu >>> Cc: Deepika Kethi Reddy >>> Cc: Jeremy Soller >>> Cc: Kathappan Esakkithevar >>> Cc: Kelly Steele >>> Cc: Leif Lindholm >>> Cc: Marcin Wojtas >>> Cc: Meenakshi Aggarwal >>> Cc: Michael D Kinney >>> Cc: Nate DeSimone >>> Cc: Philippe Mathieu-Daudé >>> Cc: Rangasai V Chaganty >>> Cc: Thomas Abraham >>> Cc: Wenyi Xie >>> Cc: Yi Qian >>> Cc: Zailiang Sun >>> > > Where appropriate in the series, > > Acked-by: Ard Biesheuvel > > and > > Acked-by: Ard Biesheuvel much appreciated everyone! Laszlo