From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail8.parnet.fi (mail8.parnet.fi [77.234.108.134]) by sourceware.org (Postfix) with ESMTPS id A33B8385800D for ; Mon, 1 Aug 2022 20:24:52 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org A33B8385800D Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=martin.st Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=martin.st Received: from mail9.parnet.fi (mail9.parnet.fi [77.234.108.21]) by mail8.parnet.fi with ESMTP id 271KOiXh022372-271KOiXi022372; Mon, 1 Aug 2022 23:24:44 +0300 Received: from foo.martin.st (host-97-187.parnet.fi [77.234.97.187]) by mail9.parnet.fi (Postfix) with ESMTPS id EA568A1468; Mon, 1 Aug 2022 23:24:43 +0300 (EEST) Date: Mon, 1 Aug 2022 23:24:43 +0300 (EEST) From: =?ISO-8859-15?Q?Martin_Storsj=F6?= To: Nick Clifton cc: binutils@sourceware.org Subject: Re: [PATCH v3 2/2] ld: Support the -exclude-symbols option via COFF def files, with the EXCLUDE_SYMBOLS keyword In-Reply-To: <772c7816-f3c6-9b90-4e51-e9049521b716@redhat.com> Message-ID: <2ed37bd1-1cf1-291-842c-d012f76deac@martin.st> References: <20220728133737.4049560-1-martin@martin.st> <20220728133737.4049560-2-martin@martin.st> <772c7816-f3c6-9b90-4e51-e9049521b716@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed X-FE-Policy-ID: 3:14:2:SYSTEM X-Spam-Status: No, score=-3.6 required=5.0 tests=BAYES_00, JMQ_SPF_NEUTRAL, KAM_DMARC_STATUS, RCVD_IN_DNSWL_LOW, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: binutils@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Binutils mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Aug 2022 20:24:59 -0000 On Mon, 1 Aug 2022, Nick Clifton wrote: > Hi Martin, > >> This was requested in review. >> --- >> v3: Fixed the testcase to run successfully, added an x86_64 testcase, >> changed >> i*86 into i?86. > > Patch approved - please apply. I applied these two patches now - thanks! // Martin