From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 48D50C6FA8E for ; Thu, 2 Mar 2023 20:08:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229662AbjCBUIi convert rfc822-to-8bit (ORCPT ); Thu, 2 Mar 2023 15:08:38 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41176 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229502AbjCBUIh (ORCPT ); Thu, 2 Mar 2023 15:08:37 -0500 Received: from secure.elehost.com (secure.elehost.com [185.209.179.11]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id EC56C1514C for ; Thu, 2 Mar 2023 12:08:35 -0800 (PST) X-Virus-Scanned: Debian amavisd-new at secure.elehost.com Received: from Mazikeen (cpebc4dfb928313-cmbc4dfb928310.cpe.net.cable.rogers.com [99.228.251.108] (may be forged)) (authenticated bits=0) by secure.elehost.com (8.15.2/8.15.2/Debian-22ubuntu3) with ESMTPSA id 322K86ec1977674 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 2 Mar 2023 20:08:07 GMT Reply-To: From: To: "'Junio C Hamano'" Cc: "'Git List'" References: <000001d94d26$1e52a910$5af7fb30$@nexbridge.com> <001a01d94d3f$04c2e460$0e48ad20$@nexbridge.com> In-Reply-To: Subject: RE: [Failure] Re: git 2.40.0-rc1 Date: Thu, 2 Mar 2023 15:08:29 -0500 Organization: Nexbridge Inc. Message-ID: <001e01d94d42$c5213b50$4f63b1f0$@nexbridge.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8BIT X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQJtn73P7CYGffBgedw5ASJ2ucV6kQHxJnHOAgZTViEB6mkEDq2QNrYA Content-Language: en-ca Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org On Thursday, March 2, 2023 3:00 PM, Junio C Hamano wrote: > writes: > >> On Thursday, March 2, 2023 1:44 PM, Junio C Hamano wrote: >>> writes: >>> >>>> Git 2.40.0-rc2 was a regression from rc1. The following new tests failed: > >I think you are reporting -rc1 relative to -rc0. > >>>> t3206 - virtually all subtests failed. An example of this failure is >>>> a results comparison in subtest 1 (seemingly relating to abbreviation): >>>> + test_cmp expect actual >>>> --- expect 2023-03-02 16:39:23 +0000 >>>> +++ actual 2023-03-02 16:39:23 +0000 >>>> @@ -1,4 +1,4 @@ >>>> -1: 4de457d = 1: 35b9b25 s/5/A/ >>>> -2: fccce22 = 2: de345ab s/4/A/ >>>> -3: 147e64e = 3: 9af6654 s/11/B/ >>>> -4: a63e992 = 4: 2901f77 s/12/B/ >>>> +1: 4de457d2c0d218f48d66f45f9b30f3aa62562105 = 1: >>>> 35b9b25f76d404d09a23e6c8efa96c3ce19e19aa s/5/A/ >>> >>>Are any big endian machines involved? >>> >>>cf. https://lore.kernel.org/git/Y%2F+paI8WGSmEbv%2Fw@pobox.com/ >> >> Oh yes. All NonStop variants are big-endian. That hasn't changed the >> SHA-1 signatures though. > >There is a stupid sequence that slipped through the review > > char c = structPtr->intMember; > > if (c < 0) > ... do something ... > >and where "char" is "unsigned char", the compiler thinks "do something" will never >trigger and from there everything goes haywire. > >The tip of 'master' already has a fix (and no other changes as of this writing), and >will be part of -rc2. Thank you for the really fast fix on this 😊. --Randall