From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-x235.google.com (mail-wm0-x235.google.com [IPv6:2a00:1450:400c:c09::235]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 8CAE681F3F for ; Fri, 2 Dec 2016 03:27:12 -0800 (PST) Received: by mail-wm0-x235.google.com with SMTP id a197so13861858wmd.0 for ; Fri, 02 Dec 2016 03:27:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=QFVQ/cj5H102AfAQ1pguHiEC8/rsDCYmTjZo0iFXvVA=; b=gi6zFZAC2qEVi0RkJ/vQvix8P6GgOxHqvcMbx5/MPj2OzBZRzZHoiGi3uf5AfM8xhr jBse1G+ODX7ko4rR+jSKm3tKcIQ+w3qYYKIOoIVjYm16/OFSrnWkmieZzjY3vrRAMnNa Wk2pJ4XzeSgWF0QqR2Q3EMXJMlARsBRCOVuq8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=QFVQ/cj5H102AfAQ1pguHiEC8/rsDCYmTjZo0iFXvVA=; b=b8wR1gbmpyDgmeExjfX61ayaxM51WhfDsmgvA4HOCpR7posBsQzk8PcTiCWKCjSczW pRGznkVU2jC69sv+ThdP5+k3N2eoJqm+75xsbqIBzvwYoo81QfGlYxgbXDPu3iu8IkU2 s6M7Lmizu+0yMoraTJtpoTZGiwERRj6gDuu/LeBhERArLWGy2SKsel3Cn9/15ZAWTUTM 6IK0P1nhtB3ICS9byCosmYO8GowZdSjLr6TQLa6QFX3zbSJK0Hv2AR+qmL4fFx4NEuav WtkU8BzdBIlKbv7ff7ltmo8SFJg/F5rXugFXp/OSZJdsdL7apcMwyRVUjEQLI+fz22m3 y2zA== X-Gm-Message-State: AKaTC03UvGIMUpaXOcxFAjDWiKPQb/53qGnKYDua/0yniUlbTsBNmzejuCX09g9q7R1/VGia X-Received: by 10.28.4.200 with SMTP id 191mr2297983wme.23.1480678030686; Fri, 02 Dec 2016 03:27:10 -0800 (PST) Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id 81sm2730566wmw.7.2016.12.02.03.27.09 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 02 Dec 2016 03:27:10 -0800 (PST) Date: Fri, 2 Dec 2016 11:27:08 +0000 From: Leif Lindholm To: Jordan Justen Cc: edk2-devel@lists.01.org, Ruiyu Ni , Ard Biesheuvel , Andrew Fish , Michael D Kinney , Laszlo Ersek , David Wei Message-ID: <20161202112708.GT27069@bivouac.eciton.net> References: <20161201155326.21716-1-leif.lindholm@linaro.org> <148062112294.26637.9118356424650979744@jljusten-ivb> MIME-Version: 1.0 In-Reply-To: <148062112294.26637.9118356424650979744@jljusten-ivb> User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [PATCH 0/7] Various: Remove EDK2 use of IntelFrameworkModulePkg legacy libs X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 02 Dec 2016 11:27:12 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Dec 01, 2016 at 11:38:42AM -0800, Jordan Justen wrote: > For the future, I'd recommend adding Cc's for package owners to the > associated patch commit message. I guess what you're actually asking for is to have only the parts of series I want your review on sent to you? That's a fair point, but having Cc: tags of package maintainers in the commit log is just noise. And we are indeed seeing a lot of those at the moment. This seems like an excellent time to bring back the idea of unifying the */Contributions.txt and codifying some of this behaviour better. (None of them mention Cc: - and they are in fact all bit-identical.) Rather than derailing this thread, how about I send out a patch that does just that, and then we can discuss modifications to the document? > Series Reviewed-by: Jordan Justen Many thanks - I'll apply for your packages. Regards, Leif > On 2016-12-01 07:53:19, Leif Lindholm wrote: > > LzmaCustomDecompressLib and PeiDxeDebugLibReportStatusCode were copied > > from IntelFrameworkModulePkg to MdeModulePkg, but the originals were > > kept for compatibility. > > > > Nevertheless, new code should be using the MdeModulePkg versions, so > > change all references in in-tree platforms. > > > > Since the patches are individually independent, I plan to push them > > myself as Reviewed-by:s appear. Laszlo/Mike - Are you OK with me pusing > > 1 and 2 myself before the whole series is reviewed? > > > > Changes from RFC: > > - Broken down into per-package patches. > > - Received Reviewed-by:s added. > > > > Leif Lindholm (7): > > OvmfPkg: Remove use of IntelFrameworkModulePkg legacy libs > > QuarkSocPkg: Remove use of IntelFrameworkModulePkg legacy libs > > BeagleBoardPkg: Remove use of IntelFrameworkModulePkg legacy libs > > EmbeddedPkg: Remove use of IntelFrameworkModulePkg legacy libs > > DuetPkg: Remove use of IntelFrameworkModulePkg legacy libs > > EmulatorPkg: Remove use of IntelFrameworkModulePkg legacy libs > > Vlv2TbltDevicePkg: Remove use of IntelFrameworkModulePkg legacy libs > > > > BeagleBoardPkg/BeagleBoardPkg.dsc | 4 ++-- > > DuetPkg/DuetPkgIa32.dsc | 4 ++-- > > DuetPkg/DuetPkgX64.dsc | 4 ++-- > > EmbeddedPkg/EmbeddedPkg.dsc | 2 +- > > EmulatorPkg/EmulatorPkg.dsc | 4 ++-- > > OvmfPkg/OvmfPkgIa32.dsc | 4 ++-- > > OvmfPkg/OvmfPkgIa32X64.dsc | 4 ++-- > > OvmfPkg/OvmfPkgX64.dsc | 4 ++-- > > QuarkSocPkg/QuarkSocPkg.dsc | 2 +- > > Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc | 4 ++-- > > Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 6 +++--- > > Vlv2TbltDevicePkg/PlatformPkgX64.dsc | 6 +++--- > > 12 files changed, 24 insertions(+), 24 deletions(-) > > > > -- > > 2.10.2 > > > > _______________________________________________ > > edk2-devel mailing list > > edk2-devel@lists.01.org > > https://lists.01.org/mailman/listinfo/edk2-devel