From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=66.187.233.73; helo=mx1.redhat.com; envelope-from=lersek@redhat.com; receiver=edk2-devel@lists.01.org Received: from mx1.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) (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 EE58C210DA7B9 for ; Mon, 6 Aug 2018 09:41:26 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com [10.11.54.3]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 7DAB8818A6A2; Mon, 6 Aug 2018 16:41:25 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-121-82.rdu2.redhat.com [10.10.121.82]) by smtp.corp.redhat.com (Postfix) with ESMTP id DF6D2101041D; Mon, 6 Aug 2018 16:41:24 +0000 (UTC) From: Laszlo Ersek To: "Gao, Liming" References: <20180726004415.13381-1-lersek@redhat.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E2C4BDF@SHSMSX104.ccr.corp.intel.com> <7fcaa55c-abf5-25ff-2a08-6168ee10fd01@redhat.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E2C6726@SHSMSX104.ccr.corp.intel.com> <44ddfa8b-7af5-47a7-d584-6777213706e3@redhat.com> Cc: edk2-devel-01 Message-ID: Date: Mon, 6 Aug 2018 18:41:24 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: <44ddfa8b-7af5-47a7-d584-6777213706e3@redhat.com> X-Scanned-By: MIMEDefang 2.78 on 10.11.54.3 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.8]); Mon, 06 Aug 2018 16:41:25 +0000 (UTC) X-Greylist: inspected by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.8]); Mon, 06 Aug 2018 16:41:25 +0000 (UTC) for IP:'10.11.54.3' DOMAIN:'int-mx03.intmail.prod.int.rdu2.redhat.com' HELO:'smtp.corp.redhat.com' FROM:'lersek@redhat.com' RCPT:'' Subject: Re: [PATCH 0/6] BaseTools/Source/C: take EXTRA_OPTFLAGS and EXTRA_LDFLAGS from the caller X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.27 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 06 Aug 2018 16:41:27 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Hi Liming, On 08/06/18 17:18, Laszlo Ersek wrote: > On 08/06/18 16:48, Gao, Liming wrote: >> Laszlo: >> Thanks for your detail information. I understand EXTRA_OPTFLAGS. >> So, its name is OK to me. >> >> On Pccts, it is the third party code. I would like to make the >> minimal change. So, I ask whether we not touch it. > > OK, thank you, I'll look into that. I started writing up a summary for the stake-holders of , explaining that some source code that goes into the VfrCompile utility is native to the edk2 project, while the code that *generates* the lexer and parser source code for VfrCompile comes from the PCCTS project, and is used only temporarily. And, that this should be a good enough reason to ignore PCCTS, because in upstream the maintainers prefer not touching PCCTS source. However: our git history for "BaseTools/Source/C/VfrCompile/Pccts" does not corroborate this preference. Consider: (a) 1 30fdf1140b8d [2009-07-17] Check In tool source code based on Build tool project revision r1655. 2 b69fd59e6f1a [2014-08-25] Fix nmake cleanall bugs. 3 5ddccf34c4f5 [2015-07-08] BaseTools: Fix build on FreeBSD and allow use of non-gcc system compiler 4 819a2394f17f [2016-01-11] BaseTools/VfrCompile: honor CC if it is set 5 4ac14ceae076 [2016-09-08] BaseTools VfrCompile Pccts: Update GCC Flags to the specific one with BUILD_ prefix Commits #3 through #5 modify the same set of files as my patches 4-6 -- the "antlr" and "dlg" makefiles. (b) 6 99e55970ff07 [2016-10-20] BaseTools: Fix typos in comments and variables This is from Gary's series [edk2] [PATCH 00/33] Fix typos in comments and variables and it modifies "dlg" source code. (c) 7 bab5ad2fd14b [2016-11-08] BaseTools/VfrCompile: Add checks for array access 8 77dee0b1859d [2016-11-08] BaseTools/VfrCompile: Avoid freeing freed memory in classes 9 d55638362727 [2016-11-08] BaseTools/VfrCompile/Pccts: Add virtual destructor for class DLGInputStream 10 fef15ecd20dd [2016-11-08] BaseTools/VfrCompile/Pccts: Make assignment operator not returning void These four commits (#7 through #10) are from Hao's series [edk2] [PATCH v2 00/53] Resolve issues for C source codes in BaseTools and they modify PCCTS headers. (d) 11 5b26adf03a0b [2016-12-20] BaseTools: fix format-security build warnings 12 8230d45bba51 [2016-12-20] BaseTools: fix format type build warnings Commits #11 and #12 are from Heyi's series [edk2] [PATCH 0/4] Fix GCC build warnings for BaseTools and they modify the "antlr" source code. (e) 13 0a64f49fde09 [2016-12-23] BaseTools/Pccts: Resolve GCC sting format mismatch build warning This patch is again from Hao, and it modifies utility code in PCCTS that is built into both "dlg" and "antlr" (namely, "set.c"). (f) 14 a5b84d3480b4 [2018-01-02] BaseTools: eliminate unused expression result 15 4e97974c1e52 [2018-01-02] BaseTools: silence parentheses-equality warning These are from a series that Zenith432 posted without a cover letter. They modify "antlr" and "dlg" source code. The above examples imply that we have modified both the makefiles and the source code under PCCTS, over time. Do you still prefer that I drop those parts of my series? I can attempt to do that, but then I cannot tell the RHBZ#1540244 stakeholders that we "generally" avoid patching the bundled PCCTS instance -- because, we do patch it whenever necessary. Thanks! Laszlo