From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Fri, 26 Apr 2019 15:17:42 -0700 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 1774D30BD2BA; Fri, 26 Apr 2019 22:17:42 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-121-104.rdu2.redhat.com [10.10.121.104]) by smtp.corp.redhat.com (Postfix) with ESMTP id E2369608AC; Fri, 26 Apr 2019 22:17:39 +0000 (UTC) Subject: Re: [edk2-devel] [edk2][edk2-libc/master V2]Request to add new edk2-libc repository To: devel@edk2.groups.io, michael.d.kinney@intel.com Cc: "Carsey, Jaben" , "Daryl McDaniel (edk2-lists@mc2research.org)" , "leif.lindholm@linaro.org" , "Andrew Fish (afish@apple.com)" References: From: "Laszlo Ersek" Message-ID: Date: Sat, 27 Apr 2019 00:17:38 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.45]); Fri, 26 Apr 2019 22:17:42 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 04/26/19 02:55, Michael D Kinney wrote: > Hello, > > New in V2 > ========= > * Add details to Readme.md on how the history for these > three packages was exported from the edk2 repository > using git filter-branch. > * Remove redundant content in edk2-libc Readme.md and > Maintainters.txt and instead point to content in edk2 > repository. > * Mark the sections of Maintainers.txt that need further > discussion as TBD. > * Update installation and build instructions in the > Readme.txt files in AppPkg and StdLib. > * Fix a warning in StdLib when building with VS2015. > ======== > > There were no objections to the following RFC to add > a new edk2-libc repository. > > https://edk2.groups.io/g/devel/message/35211 > > I have entered the following Feature Request Bugzilla > > https://bugzilla.tianocore.org/show_bug.cgi?id=1734 > > I have posted a version of the edk2-libc repository for > review at the following location. It includes all the > history for the three packages from the edk2 repository. > > https://github.com/mdkinney/edk2-libc/tree/master_V2 > > Please review this branch. There are 3 additional commits: > > * Update the Readme.md and Maintainers.txt that scopes them > to this new edk2-libc repository. > > https://github.com/mdkinney/edk2-libc/commit/62cfdf929f15edba7f7bcbbdb01e4dc50d2c81a7 Reviewed-by: Laszlo Ersek > * Update Readme.txt in AppPkg and StdLib to describe how to > install both the edk2-libc repository and the edk2 > repository and set PACKAGES_PATH to support builds. > > https://github.com/mdkinney/edk2-libc/commit/d004f93626b3bb962c24500db6ec7284db074c9a In both README files here, I've noticed the following: - typo: "The EADK is depends on the EDK II source tree and need to be installed with" should be "The EADK depends on the EDK II source tree and needs to be installed with" - another typo: "git clone https://github.com/mdkinney/edk2-libc.git" should be "git clone https://github.com/tianocore/edk2-libc.git" With those fixed in both READMEs: Reviewed-by: Laszlo Ersek > > * Fix a minor issue that removes warnings when building > edk2-libc packages with VS2015 toolchains. > > https://github.com/mdkinney/edk2-libc/commit/145d262e94182a6b4631e0b6df5bc5d67e209931 Acked-by: Laszlo Ersek Thanks, Laszlo > > Best regards, > > Mike > > >