From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-3.0 required=3.0 tests=AWL,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_HI,T_RP_MATCHES_RCVD shortcircuit=no autolearn=ham autolearn_force=no version=3.4.0 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by dcvr.yhbt.net (Postfix) with ESMTP id B889D1FADF for ; Mon, 15 Jan 2018 02:37:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753485AbeAOCh4 convert rfc822-to-8bit (ORCPT ); Sun, 14 Jan 2018 21:37:56 -0500 Received: from elephants.elehost.com ([216.66.27.132]:53437 "EHLO elephants.elehost.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752904AbeAOChz (ORCPT ); Sun, 14 Jan 2018 21:37:55 -0500 X-Virus-Scanned: amavisd-new at elehost.com Received: from pangea (CPE00fc8d49d843-CM00fc8d49d840.cpe.net.cable.rogers.com [99.229.179.249]) (authenticated bits=0) by elephants.elehost.com (8.15.2/8.15.2) with ESMTPSA id w0F2bkTU010378 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Sun, 14 Jan 2018 21:37:47 -0500 (EST) (envelope-from rsbecker@nexbridge.com) From: "Randall S. Becker" To: "'Johannes Sixt'" Cc: , "'Joachim Schmitz'" References: <004a01d38cab$705262a0$50f727e0$@nexbridge.com> <001a01d38d57$d36c7d10$7a457730$@nexbridge.com> <59d3adab-4a95-4ef5-2d8f-ef4c7b797156@kdbg.org> <004a01d38d7f$3306e810$9914b830$@nexbridge.com> In-Reply-To: <004a01d38d7f$3306e810$9914b830$@nexbridge.com> Subject: RE: [BUG] test_must_fail: does not correctly detect failures - Was Git 2.16.0-rc2 Test Summary on NonStop Date: Sun, 14 Jan 2018 21:37:42 -0500 Message-ID: <007901d38da9$d517e9e0$7f47bda0$@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: AQJL2DF5YDw1sAX4TtGn7MOoKgkH9AFqQNrWAa1IcAICMDCvx6JZY76A Content-Language: en-ca Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org On January 14, 2018 4:33 PM, I wrote: > On January 14, 2018 1:41 PM Johannes Sixt wrote: > > Sent: > > Am 14.01.2018 um 17:50 schrieb Randall S. Becker: > > > Follow-up: This looks like the completion code from perl on NonStop > > > is not the same as expected by git in the case of failures. I need > > > to debug this to get more details to the team. We have had > > > completion issues before relating to interpretation problems between > > > perl, bash, and git in this platform, so I'm assuming this to be > > > likely again but need to > > track down the specifics. > > > Can anyone point me to where the detection is within git or the execv? > > > > Perhaps you are looking for wait_or_whine() in run-command.c? But this > > function cannot do anything if perl alread exits with an exotic code > > (> > > 128 even though no signal was received). > > The exotic error code coming back from perl is 162. I can muck with it, if > there was a value more useful to git. > * 553 } else if (WIFEXITED(status)) { > * 554 code = WEXITSTATUS(status); > (eInspect 3,887):p code > $4 = 162 > > The perl code uses die to terminate with a non-specific non-zero error code. > What it seems is that there is an assumed value that the git tests depend on, > but perl.org describes the following: > > "die raises an exception. Inside an eval the error message is stuffed into $@ > and the eval is terminated with the undefined value. If the exception is > outside of all enclosing evals, then the uncaught exception prints LIST to > STDERR and exits with a non-zero value. If you need to exit the process with > a specific exit code, see exit." > > So it seems that we might need to either not use die or change the tests not > to care about the exit code for specific tests involving perl. Suggestions? Sadly, while changing the funky 162 completion code to 255 works fine for t9001, it causes a bunch of other tests to fail (parts of 1308 and most of 1404). I can't tall whether this is test suite or code related but I'm caught in the middle. Going to the platform developers may eventually get the fix for t9001 (fixing perl), but otherwise, I'm not sure why changing 162 to 255 causes 1308 and 1404 to blow so badly. In any event, I'm putting this away for a few days due to $DAYJOB. Cheers, Randall