From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11901 invoked by alias); 29 May 2012 13:02:05 -0000 Received: (qmail 11889 invoked by uid 22791); 29 May 2012 13:02:04 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=AWL,BAYES_00,RCVD_IN_HOSTKARMA_NO X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 29 May 2012 13:01:51 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 0DE231C697A; Tue, 29 May 2012 09:01:51 -0400 (EDT) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id WVtKFWgmgBrC; Tue, 29 May 2012 09:01:50 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id B4C5E1C6916; Tue, 29 May 2012 09:01:50 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id E2A28145616; Tue, 29 May 2012 06:01:44 -0700 (PDT) Date: Tue, 29 May 2012 13:02:00 -0000 From: Joel Brobecker To: Pierre Muller Cc: gdb-patches@sourceware.org, 'Jan Kratochvil' , 'Sergio Durigan Junior' Subject: Re: [RFA-v3] Add scripts to generate ARI web pages to gdb/contrib/ari directory Message-ID: <20120529130144.GM5492@adacore.com> References: <001201cd3547$377188b0$a6549a10$@muller@ics-cnrs.unistra.fr> <20120525194654.GA30472@host2.jankratochvil.net> <003c01cd3b3c$d85e07d0$891a1770$@muller@ics-cnrs.unistra.fr> <002101cd3c42$5473b670$fd5b2350$@muller@ics-cnrs.unistra.fr> <006701cd3d00$6b9566a0$42c033e0$@muller@ics-cnrs.unistra.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <006701cd3d00$6b9566a0$42c033e0$@muller@ics-cnrs.unistra.fr> User-Agent: Mutt/1.5.20 (2009-06-14) 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: 2012-05/txt/msg01012.txt.bz2 Hi Pierre, > As explained in my answer to Sergio, I would like to leave other fixes > to after initial commit to have a better history of changes relative > to the ss directory version on sourceware.org server. That would be fine. I think that's a good reason. > PS: could contrib get a separate ChangeLog file? I think we are trying to get as few ChangeLog files as possible. Any reason why you'd like to have your own ChangeLog file? It does not seem unreasonable since the changes in contrib/ari as probably expected to be few and far between once it's stabilized, and having them listed in the main ChangeLog might make it harder to have a quick list of the changes made in that directory over time... > 2012-05-28 Pierre Muller > > * contrib/ari/create-web-ari-in-src.sh: New file. > * contrib/ari/gdb_ari.sh: New file. > * contrib/ari/gdb_find.sh: New file. > * contrib/ari/update-web-ari.sh: New file. > ? contrib/ari/patch > Index: contrib/ari/create-web-ari-in-src.sh > =================================================================== > RCS file: contrib/ari/create-web-ari-in-src.sh > diff -N contrib/ari/create-web-ari-in-src.sh > --- /dev/null 1 Jan 1970 00:00:00 -0000 > +++ contrib/ari/create-web-ari-in-src.sh 28 May 2012 18:17:56 -0000 > @@ -0,0 +1,68 @@ > +#! /bin/sh > + > +# GDB script to create web ARI page directly from within gdb/ari directory. ^^^^^^^^^ gdb/contrib/ari ? > +set -x We'll probably want to remove that, at some point, and remove all the associated "set +x" in the other scripts. > +# update-web-ari.sh script wants four parameters > +# 1: directory of checkout src or gdb-RELEASE for release sources. > +# 2: a temp directory. > +# 3: a directory for generated web page. > +# 4: The name of the current package, must be gdb here. > +# Here we provide default values for these 4 parameters Are the parameters passed via the environment? > +# Default location of generate index.hmtl web page. > +if [ -z "${webdir}" ] ; then > + webdir=~/htdocs/www/local/ari > +fi Default location should be current working directory, IMO. > +# Launch update-web-ari.sh in same directory as current script. > +${scriptpath}/update-web-ari.sh ${srcdir} ${tempdir} ${webdir} gdb Same as "set -x", we'll probably want to get rid of the final parameter eventually, since we know it'll always be "gdb" for us. -- Joel