Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Pierre Muller" <pierre.muller@ics-cnrs.unistra.fr>
To: "'Joel Brobecker'" <brobecker@adacore.com>
Cc: "'Pedro Alves'" <palves@redhat.com>, <gdb-patches@sourceware.org>,
	       "'Jan Kratochvil'" <jan.kratochvil@redhat.com>,
	       "'Sergio Durigan Junior'" <sergiodj@redhat.com>
Subject: RE: [RFA-v4] Add scripts to generate ARI web pages to gdb/contrib/ari directory
Date: Thu, 14 Jun 2012 16:14:00 -0000	[thread overview]
Message-ID: <000001cd4a48$b4a97e80$1dfc7b80$@muller@ics-cnrs.unistra.fr> (raw)
In-Reply-To: <20120614160200.GL18729@adacore.com>

> >   Here is a new version of my patch to insert ARI web page generation
> >   script into GDB sources.
> 
> Thank you!
> 
> > As requested I removed the references to emails
> > for sourceware.
> 
> Nice.
> 
> > Otherwise I also made a small change to generate the web page in a
> > subdir called trunk/ari if no tag in found in CVS subdirectory and
> > branch/ari otherwise.
> 
> For this one, I do not think that this is a good idea. Not everyone
> works from a CVS sandbox anymore, so the assumption is going to break
> for those. But also, I don't see why we would need to produce the data
> in a different directory based on the type of sources.

  This is only because currently there are web links
inside the generated pages that should allow to switch from
trunk to branches (It would at least be useful that those link
still work when the script is called on the sourceware.org repository).
 
> We can let the caller decide where he wants the data to go...

  OK, this is already possible,
either by specifying a webdir environment variable
before calling create-web-ari-in-src.sh script
or by using directly
update-web-ari.sh script, you then need to specify all 4 args 
as explained in start of create-web-ari-in-src.sh script.

  I would just prefer the we do use another directory or a sub-directory,
because there is a lot of file generation, and copying of scripts
to destination dir.
  If webdir is current directory, it will be very hard to clean the
mess afterwards...

Pierre


  reply	other threads:[~2012-06-14 16:14 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-18 22:41 [RFA] " Pierre Muller
2012-05-25  8:09 ` PING " Pierre Muller
2012-05-25 19:47 ` Jan Kratochvil
2012-05-26 12:41   ` [RFA-v2] " Pierre Muller
2012-05-27  4:06     ` Sergio Durigan Junior
2012-05-27 19:53       ` Pierre Muller
2012-05-27 22:03         ` Sergio Durigan Junior
2012-05-28 18:34           ` [RFA-v3] " Pierre Muller
2012-05-28 18:38             ` Pierre Muller
2012-05-29 13:02             ` Joel Brobecker
2012-05-29 13:13               ` Pedro Alves
2012-05-31  6:56                 ` Pierre Muller
2012-05-31 15:59                   ` Joel Brobecker
2012-06-14 12:36                 ` [RFA-v4] " Pierre Muller
2012-06-14 16:02                   ` Joel Brobecker
2012-06-14 16:14                     ` Pierre Muller [this message]
2012-06-14 16:22                       ` Joel Brobecker
2012-08-21 10:27                         ` About RFA for " Pierre Muller
2012-08-21 22:36                           ` Sergio Durigan Junior
     [not found]                         ` <50336283.a2db440a.600c.105dSMTPIN_ADDED@mx.google.com>
2012-08-21 22:25                           ` Doug Evans
2012-09-26 22:15                   ` [RFA-v5] " Pierre Muller
2012-10-08 21:21                     ` Pierre Muller
2012-10-09  3:45                       ` Sergio Durigan Junior
2012-10-09  5:52                         ` Joel Brobecker
2012-10-22 21:04                     ` Joel Brobecker
2012-10-23 15:21                       ` Pierre Muller
2012-10-23 17:13                       ` Sergio Durigan Junior
2012-10-30 15:16                         ` Pierre Muller
     [not found]                         ` <4332.56673063642$1351610219@news.gmane.org>
2012-11-01 19:57                           ` Tom Tromey
2012-11-01 22:39                             ` Pierre Muller
2012-06-22 16:10                 ` [RFA-v3] " Tom Tromey
2012-05-26  0:12 ` [RFA] " Sergio Durigan Junior

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='000001cd4a48$b4a97e80$1dfc7b80$@muller@ics-cnrs.unistra.fr' \
    --to=pierre.muller@ics-cnrs.unistra.fr \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=jan.kratochvil@redhat.com \
    --cc=palves@redhat.com \
    --cc=sergiodj@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox