From nobody Thu May 2 04:46:15 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+66602+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+66602+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1603715276; cv=none; d=zohomail.com; s=zohoarc; b=gvdJnpWhJSi9ORnoBZO8bfPfDky238xEueKHk1PBRqR3OnhUhnKbUTquStdbTTlYYAQk4s5Jzyf8DEZEXws1sagQp5EIDqXzVhxd1Pz08Q0nt7/Rfzf/HG5nFgDiTQ+cLWxRZYW9MCpMZMCIjoxtNtc6qt72J1K/1Qzmg9hABzs= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1603715276; h=Content-Transfer-Encoding:Cc:Date:From:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Sender:Subject:To; bh=WFWM/umyBPlmtbIeP6/d4b564bxEUobLv9Z+S+nPNgk=; b=PoRdoxypA5PTsV93cJrjsg6MXMhAWNJ9nmor9uIXhcxzRNLE952DlDye0nbEf8p3oj3q5XNBm8hneMi0AcpuAkWT92T56Q2fBsOoy0b2wKKiSLRXEB1X8fKfB7V0U1lWMiIT8kmQoyODq9nlMMEj8IuoXdK8ihvQkXbiP4QXgzA= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+66602+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 160371527631829.742395571119005; Mon, 26 Oct 2020 05:27:56 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id icoDYY1788612x5uSoositEG; Mon, 26 Oct 2020 05:27:54 -0700 X-Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by mx.groups.io with SMTP id smtpd.web12.11776.1603715268519148157 for ; Mon, 26 Oct 2020 05:27:48 -0700 IronPort-SDR: weNz5pSWlRPXiKWSGFuPK9Z9vsSB7zrJP+PEr8MlWMOB/W65Ax8L0R8EQeEhZgeKPE9kcW/ejG fRy7R26onV/Q== X-IronPort-AV: E=McAfee;i="6000,8403,9785"; a="232101328" X-IronPort-AV: E=Sophos;i="5.77,419,1596524400"; d="scan'208";a="232101328" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Oct 2020 05:27:47 -0700 IronPort-SDR: ZkKxa9pL0/ITdazNkuP/AXYvP/PxVBGtPNwOpsyCUJMvtQFKjt50NJH1NfyfOmF+SCMwDcLbYX Gpyxv6qE427w== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,419,1596524400"; d="scan'208";a="322528348" X-Received: from shwdepsi1121.ccr.corp.intel.com ([10.239.158.66]) by orsmga006.jf.intel.com with ESMTP; 26 Oct 2020 05:27:46 -0700 From: "Bob Feng" To: devel@edk2.groups.io Cc: Liming Gao , Yuwei Chen Subject: [edk2-devel] [Patch] BaseTools: Remove the dependency on the build intermediate file Date: Mon, 26 Oct 2020 20:27:44 +0800 Message-Id: <20201026122744.45096-1-bob.c.feng@intel.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,bob.c.feng@intel.com X-Gm-Message-State: X70T1eUBq8txCXbvBmjyHYKYx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1603715274; bh=wOINS+cwNyinkrIpY7O54/EwTC7mm326UyMp0M35Bsk=; h=Cc:Date:From:Reply-To:Subject:To; b=xU5ZcvNHKsCq4WGeFn61lmIQq+AovqQgypp8DeW+/ubcazOvTxkciHn4+BnDzol/hAX Q72nfKtuCznQymXuLiZQtKjSBMPHg+TDlbuqDUUAwr5MF50gl0watF/rkVV1yzb38a6cu LOgGY19RmlwZz0CZnRFaa80yOMQ7ss53+uw= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" When generating compressed section, the build tool rely on the build intermediate files, which were generated in last build, to get the file list. This method will cause the incremental build to generate incorrect build result. To reproduce this incremental build error, you can do: 1. build Ovmf 2. change the module OvmfPkg\AcpiTables a source file Facp.aslc name from Facp.aslc to Facpxxx.aslc. 3. change the Facp.aslc file name in [sources] section of AcpiTables.inf 4. incremental build Ovmf you will see the in AcpiTables module Makefile, the corresponding Facp.acpi file is not changed. This patch is to make the build always get file list from the INF. Signed-off-by: Bob Feng Cc: Liming Gao Cc: Yuwei Chen --- BaseTools/Source/Python/GenFds/Section.py | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/BaseTools/Source/Python/GenFds/Section.py b/BaseTools/Source/P= ython/GenFds/Section.py index 2acb70f412..0382b2a759 100644 --- a/BaseTools/Source/Python/GenFds/Section.py +++ b/BaseTools/Source/Python/GenFds/Section.py @@ -139,25 +139,10 @@ class Section (SectionClassObject): if File.Ext =3D=3D Suffix: FileList.append(File.Path) =20 if (not IsMakefile and Suffix is not None and os.path.exists(FfsIn= f.EfiOutputPath)) or (IsMakefile and Suffix is not None): - # - # Get Makefile path and time stamp - # - MakefileDir =3D FfsInf.EfiOutputPath[:-len('OUTPUT')] - Makefile =3D os.path.join(MakefileDir, 'Makefile') - if not os.path.exists(Makefile): - Makefile =3D os.path.join(MakefileDir, 'GNUmakefile') - if os.path.exists(Makefile): - # Update to search files with suffix in all sub-dirs. - Tuple =3D os.walk(FfsInf.EfiOutputPath) - for Dirpath, Dirnames, Filenames in Tuple: - for F in Filenames: - if os.path.splitext(F)[1] =3D=3D Suffix: - FullName =3D os.path.join(Dirpath, F) - if os.path.getmtime(FullName) > os.path.getmti= me(Makefile): - FileList.append(FullName) if not FileList: SuffixMap =3D FfsInf.GetFinalTargetSuffixMap() if Suffix in SuffixMap: FileList.extend(SuffixMap[Suffix]) =20 --=20 2.20.1.windows.1 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#66602): https://edk2.groups.io/g/devel/message/66602 Mute This Topic: https://groups.io/mt/77813761/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-