From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 12393 invoked by alias); 17 Mar 2009 22:43:57 -0000 Received: (qmail 12385 invoked by uid 22791); 17 Mar 2009 22:43:56 -0000 X-SWARE-Spam-Status: No, hits=-1.2 required=5.0 tests=AWL,BAYES_05,HK_OBFDOM X-Spam-Check-By: sourceware.org Received: from mailhost.u-strasbg.fr (HELO mailhost.u-strasbg.fr) (130.79.200.158) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 17 Mar 2009 22:43:51 +0000 Received: from baal.u-strasbg.fr (baal.u-strasbg.fr [IPv6:2001:660:2402::41]) by mailhost.u-strasbg.fr (8.14.2/jtpda-5.5pre1) with ESMTP id n2HMhiHL078338 ; Tue, 17 Mar 2009 23:43:44 +0100 (CET) Received: from mailserver.u-strasbg.fr (ms2.u-strasbg.fr [IPv6:2001:660:2402:d::11]) by baal.u-strasbg.fr (8.14.0/jtpda-5.5pre1) with ESMTP id n2HMhiiV078142 ; Tue, 17 Mar 2009 23:43:44 +0100 (CET) (envelope-from muller@ics.u-strasbg.fr) Received: from d620muller (lec67-4-82-230-53-140.fbx.proxad.net [82.230.53.140]) (user=mullerp mech=LOGIN) by mailserver.u-strasbg.fr (8.14.3/jtpda-5.5pre1) with ESMTP id n2HMhhSA030094 (version=TLSv1/SSLv3 cipher=RC4-MD5 bits=128 verify=NO) ; Tue, 17 Mar 2009 23:43:44 +0100 (CET) (envelope-from muller@ics.u-strasbg.fr) From: "Pierre Muller" To: "'Daniel Jacobowitz'" Cc: References: <000001c9a2fd$6b5332d0$41f99870$@u-strasbg.fr> <20090312163220.GB17979@caradoc.them.org> In-Reply-To: <20090312163220.GB17979@caradoc.them.org> Subject: RE: [RFA] AR Index, remove yacc-lex sources for boolean tests Date: Tue, 17 Mar 2009 23:26:00 -0000 Message-ID: <004f01c9a751$d6f04570$84d0d050$@u-strasbg.fr> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2009-03/txt/msg00341.txt.bz2 I tried several times to commit that approved patch, but I always get the same error: Could not open lock file `/cvs/gdbadmin/ss/,gdb_ari.sh,' I suppose that members allowed to commit to /cvs/src=20 are not necessarily the same as for /cvs/gdbadmin... Joel found out that this directory is writable only by group gdbadmin, and offered to commit it for me. Thanks Joel, the patch is below. Pierre Muller Pascal language support maintainer for GDB PS: the commit.txt file was just for the cvs commit -F commit.txt gdb_ari.sh as this directory has no ChangeLog. $ cat commit.txt * gdb_ari.sh (boolean, true, false): Do not consider lex and yacc sources. Pierre@d620-muller ~/ss $ cvs diff -up gdb_ari.sh Index: gdb_ari.sh =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D RCS file: /cvs/gdbadmin/ss/gdb_ari.sh,v retrieving revision 1.77 diff -u -p -r1.77 gdb_ari.sh --- gdb_ari.sh 24 Oct 2007 20:24:55 -0000 1.77 +++ gdb_ari.sh 17 Mar 2009 22:39:46 -0000 @@ -1364,7 +1364,9 @@ Do not use `boolean'\'', use `int'\'' i category["boolean"] =3D ari_regression } /(^|[^_[:alnum:]])boolean([^_[:alnum:]]|$)/ { - fail("boolean") + if (is_yacc_or_lex =3D=3D 0) { + fail("boolean") + } } BEGIN { doc["false"] =3D "\ @@ -1372,7 +1374,9 @@ Definitely do not use `false'\'' in bool category["false"] =3D ari_regression } /(^|[^_[:alnum:]])false([^_[:alnum:]]|$)/ { - fail("false") + if (is_yacc_or_lex =3D=3D 0) { + fail("false") + } } BEGIN { doc["true"] =3D "\ @@ -1380,7 +1384,9 @@ Do not try to use `true'\'' in boolean e category["true"] =3D ari_regression } /(^|[^_[:alnum:]])true([^_[:alnum:]]|$)/ { - fail("true") + if (is_yacc_or_lex =3D=3D 0) { + fail("true") + } } BEGIN { doc["if assignment"] =3D "\ > -----Message d'origine----- > De=A0: gdb-patches-owner@sourceware.org [mailto:gdb-patches- > owner@sourceware.org] De la part de 'Daniel Jacobowitz' > Envoy=E9=A0: Thursday, March 12, 2009 5:32 PM > =C0=A0: Pierre Muller > Cc=A0: gdb-patches@sourceware.org > Objet=A0: Re: [RFA] AR Index, remove yacc-lex sources for boolean tests >=20 > On Thu, Mar 12, 2009 at 11:29:26AM +0100, Pierre Muller wrote: > > This simple patch > > avoids testing lex and yacc files > > for 'boolean' 'false' and 'true'. > > > > Daniel, can I check this in? >=20 > Yes, go ahead. >=20 > -- > Daniel Jacobowitz > CodeSourcery