From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from wout3-smtp.messagingengine.com (wout3-smtp.messagingengine.com [64.147.123.19]) by mx.groups.io with SMTP id smtpd.web11.15484.1683722781257716459 for ; Wed, 10 May 2023 05:46:21 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="signature has expired" header.i=@bsdio.com header.s=fm1 header.b=tGQBNDeA; spf=pass (domain: bsdio.com, ip: 64.147.123.19, mailfrom: rebecca@bsdio.com) Received: from compute2.internal (compute2.nyi.internal [10.202.2.46]) by mailout.west.internal (Postfix) with ESMTP id 3039232000CC; Wed, 10 May 2023 08:46:20 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute2.internal (MEProxy); Wed, 10 May 2023 08:46:20 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdio.com; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to; s=fm1; t= 1683722779; x=1683809179; bh=pfGxht6Sby1ip+YhiDuu9oPAHbaNDGAlbds CCuQcWmg=; b=tGQBNDeA2Fbe3MmZiAAaH1qRVJOOMCbfXXR3r6cZtovOjEmT99B PbSjOIfhwBoe2lAwJ4KJM1mPiRZGduIKqHjHfg/COeqey2H1UPKyfqn2eeJIF5id 3gXx/sCSjG1JLYpKo07bPoT8SVFZTTxkOkkXDLgWCIuxnTiIWDqrbNGWsG0FDQSK b1lK48hFv8iOcwkYaUNOQCzB/ZUmGz9z54LXB4l6uX6z82I0b1NK7bN8bF0vWk3y enLMyThaQcJNVGiCiKi8HeiGOR7vK86rqNO8T3E6OAFI5LdFlhdIM0gysfWvD2zk 4nOUbE7HktUEiczlSOG/nEAIl7EeKns5k/A== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; t= 1683722779; x=1683809179; bh=pfGxht6Sby1ip+YhiDuu9oPAHbaNDGAlbds CCuQcWmg=; b=PLdBZtdN0/IfA3j1+RDJogU4w4BHQkmD0UAcvCJEjkdJ4L7hOyL +jATqK2kJ84V8+1jdBN2LcRea9dpY8GDWWUJUfdn0vGpyf3fAM2Bmlse+XasyQNG W2nO+dWNRVTLIfdMVH59hK3FWfahsr6CClg780H+rWcL8CdGhvsvSC2rNBsnFjKf 29Iz8zeqtrVrCnU0H32ec6nmzXE3BgCdbPINohBQdf3nJ8Q3DoP7+DaRrB7yf1dR HtuQXKo1fNOviHBxcPmXMspgKXGgxPTSA/MyypXUcfIx1v2eirda+QDgH6t/Bo+4 MumyW1zzqElHtnkNjam6UcQfnHExVrGUNQA== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrfeegiedgfeehucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepkfffgggfuffvvehfhfgjtgfgsehtjeertddtfeejnecuhfhrohhmpeftvggs vggttggrucevrhgrnhcuoehrvggsvggttggrsegsshguihhordgtohhmqeenucggtffrrg htthgvrhhnpeehleekgfdutedugeehgeffgffhteefhffhteetfeevhefgvdejhedthfff ieffleenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpe hrvggsvggttggrsegsshguihhordgtohhm X-ME-Proxy: Feedback-ID: i5b994698:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 10 May 2023 08:46:18 -0400 (EDT) Message-ID: <3173f4a8-5079-ac73-7ce2-1802152a14f1@bsdio.com> Date: Wed, 10 May 2023 06:46:17 -0600 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [edk2-stable202305 PATCH 1/1] BaseTools: Revert Set the CLANGDWARF OBJCOPY path in tools_def.template To: Ard Biesheuvel Cc: devel@edk2.groups.io, Liming Gao , Bob Feng , Yuwei Chen References: <20230510123141.1880782-1-rebecca@bsdio.com> From: "Rebecca Cran" In-Reply-To: Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 5/10/23 06:37, Ard Biesheuvel wrote: > On Wed, 10 May 2023 at 14:31, Rebecca Cran wrote: >> This reverts commit 11f62f4cc09f16d265da1a737dabfd8ed65f8c00. >> >> While GCC uses objcopy for the OBJCOPY command, it's not needed for the >> CLANGDWARF toolchain and can be left as echo. >> > Are you sure you want to change this for IA32 and X86 as well? I'm not > sure what the significance is, I suppose it sets the debuglink on the > copies of the dll files? Looking back to the tools_def.template from edk2-stable202302, we set OBJCOPY_PATH for CLANG38 IA32 and X64, but not for CLANGPDB or CLANGDWARF. I was thinking it's safer to keep existing behavior for now. -- Rebecca Cran