From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 10973 invoked by alias); 13 Sep 2017 13:15:00 -0000 Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org Received: (qmail 10937 invoked by uid 89); 13 Sep 2017 13:14:59 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS autolearn=ham version=3.3.2 spammy= X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 13 Sep 2017 13:14:58 +0000 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id D3745C058EB4; Wed, 13 Sep 2017 13:14:56 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com D3745C058EB4 Authentication-Results: ext-mx08.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx08.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=palves@redhat.com Received: from [127.0.0.1] (ovpn04.gateway.prod.ext.ams2.redhat.com [10.39.146.4]) by smtp.corp.redhat.com (Postfix) with ESMTP id D001E5D966; Wed, 13 Sep 2017 13:14:55 +0000 (UTC) Subject: Re: gettext.m4 update To: Kamil Rytarowski , gdb@sourceware.org, Sergio Durigan Junior References: <7925cdff-cd6d-2242-0c4b-55daa9e53664@gmx.com> From: Pedro Alves Message-ID: <9c14709c-0f65-b2c4-9b7e-a1cac8346f98@redhat.com> Date: Wed, 13 Sep 2017 13:15:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-SW-Source: 2017-09/txt/msg00029.txt.bz2 On 09/13/2017 12:42 PM, Kamil Rytarowski wrote: > On 11.09.2017 15:36, Kamil Rytarowski wrote: >> Hello, >> >> The current gettext.m4 bundled into GDB is too old for the NetBSD case. >> >> We mark GDB as BROKEN_GETTEXT_DETECTION and comment that it fails to add >> -lintl to the linker command line. >> >> Could we please update it to a newer version? >> >> gettext.m4 is located in https://ftp.gnu.org/pub/gnu/gettext/ >> > > I know what breaks the NetBSD buildbot [1]. > > The following rule (intl/Makefile.in) [2] breaks: > > .c.o: > $(COMPILE) $< > > .y.c: > $(YACC) $(YFLAGS) --output $@ $< > rm -f $*.h > > This means that if plural.y is newer than plural.c it will be regenerated. > > There is required Bison < 3.0 (the last one used for regeneration was > 1.35), which is already too old to keep it around. > > Git does not track mtime in files, and this enforces regeneration of > plural.c, at least in random checkouts. The NetBSD buildbot was > unfortunate to get plural.y newer: > > $ stat -f %Fm ./binutils-gdb-master/intl/plural.c > 1477324070.931854837 > $ stat -f %Fm ./binutils-gdb-master/intl/plural.y > 1477324070.931949821 > > > The cleanest and newfangled solution would be to upgrade to newer > gettext and enforce regeneration always. > > A kludge would be to insert touch(1) call before execution of intl/Makefile. > > I don't know about any good solutions in the middle. This date issue is likely to be an issue with any generated file checked in, all over the tree. Usually you won't notice if you have the right tools that regen succeeds. Maybe the bot should be doing a recursive touch(1) of all files after checkout, forcing the same mtime on all checked-out files, with 'touch -d ..' or 'touch -p ..' or 'touch -r .git/FETCH_HEAD' or something like that. Thanks, Pedro Alves