From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca (simark.ca [158.69.221.121]) by sourceware.org (Postfix) with ESMTPS id DF3CB39B3C56 for ; Sun, 6 Sep 2020 00:16:26 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org DF3CB39B3C56 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=simark.ca Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=simark@simark.ca Received: from [10.0.0.11] (173-246-6-90.qc.cable.ebox.net [173.246.6.90]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by simark.ca (Postfix) with ESMTPSA id EB1941E509; Sat, 5 Sep 2020 20:16:25 -0400 (EDT) Subject: Re: [PATCH] Do not adjust mtime timezone on Windows To: Joel Brobecker , Tom Tromey Cc: gdb-patches@sourceware.org References: <20200903170244.2508330-1-tromey@adacore.com> <20200905204921.GA26780@adacore.com> From: Simon Marchi Message-ID: <7fc17040-78a9-311a-1c47-a1febe6333cb@simark.ca> Date: Sat, 5 Sep 2020 20:16:25 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 MIME-Version: 1.0 In-Reply-To: <20200905204921.GA26780@adacore.com> Content-Type: text/plain; charset=utf-8 Content-Language: fr Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-5.0 required=5.0 tests=BAYES_00, KAM_DMARC_STATUS, NICE_REPLY_A, SPF_HELO_PASS, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 06 Sep 2020 00:16:28 -0000 On 2020-09-05 4:49 p.m., Joel Brobecker wrote: > Hi Tom, > > On Thu, Sep 03, 2020 at 11:02:44AM -0600, Tom Tromey wrote: >> PR win32/25302 notes that gdb will crash when trying to "run" even a >> simple program on Windows. The essential bug here is that the BFD >> cache can easily be corrupted -- I have sent a separate patch for >> that. >> >> The particular reason that the cache is corrupted on Windows is that >> gnulib overrides "stat" to make it do timezone adjustment -- but BFD >> does not use this version of stat. The difference here triggers the >> latent cache bug, but can also cause other bugs as well; in particular >> it can cause spurious warnings about source files being newer. >> >> This patch simply removes the stat override on mingw, making gnulib >> and BFD agree. >> >> I tested this by backing out the local AdaCore changes to work around >> this bug and then verifying that I could reproduce it. Then, I >> applied this patch and verified that "run" works again. >> >> gnulib/ChangeLog >> 2020-09-03 Tom Tromey >> >> PR win32/25302: >> * update-gnulib.sh: Apply stat patch. >> * patches/0001-use-windows-stat: New file. >> * import/m4/stat.m4: Update. >> * configure: Rebuild. > > Thanks for helping us with this patch. > > This looks good to me. We will want this patch in before > we create the gdb-10-branch, but let's give people a few > more days to comment if they'd like -- let's say, if by > Thursday, we haven't received comments, then we push it. > > Thanks again! I wasn't involved in the discussions, so I'm not really aware of the various solutions that were considered. However, if that's the solution that was deemed the best, short term, then the patch LGTM. Simon