This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [RFC] Create also ARI web page using CVS source in gdb/contrib/ari


  The current change doesn't work because the
whole checked out sources gets removed at the end of the
update-web-ari
not the end of the script, but the trap command, which is

~/ss % grep -n trap   update-web-ari
71:    trap "cd /tmp; rm -rf ${tmpdir}; exit" 0 1 2 15

  Should I modify ss/update-web-ari so that it
generate a temp-file that reports if the CVS sources
contain a contrib/ari/update-web-ari.sh file
and prevent tempdir deletion in that case?

  Any cleaner solution welcome,


  Pierre Muller


> -----Message d'origine-----
> De?: gdb-patches-owner@sourceware.org [mailto:gdb-patches-
> owner@sourceware.org] De la part de Pierre Muller
> Envoyé?: mercredi 7 novembre 2012 17:15
> À?: 'Joel Brobecker'
> Cc?: gdb-patches@sourceware.org; 'Tom Tromey'
> Objet?: RE: [RFC] Create also ARI web page using CVS source in
> gdb/contrib/ari
> 
>   OK,
>   I committed the change to ss CVS directory,
> the gdbadmin ss directory is also update, which means that
> the first ari/test/index.html should be generated tonight.
> 
>   If anything goes wrong, you are going to get the email...
> If that happens, could you please forward it to me?
> 
> 
> 
> 
> > -----Message d'origine-----
> > De?: gdb-patches-owner@sourceware.org [mailto:gdb-patches-
> > owner@sourceware.org] De la part de Joel Brobecker
> > Envoyé?: mercredi 7 novembre 2012 16:07
> > À?: Pierre Muller
> > Cc?: gdb-patches@sourceware.org; 'Tom Tromey'
> > Objet?: Re: [RFC] Create also ARI web page using CVS source in
> > gdb/contrib/ari
> >
> > >   This is the purpose of the change below, after suggestions from
Joel.
> > > For now, it just adds a new directory on the sourceware.org web server
> > > http://sourceware.org/gdb/current/ari/test
> >
> > As said privately, it is fine to have a temporary staging area for
> > testing the new ARI scripts. In the interest of allowing you to make
> > progress a little faster, please feel free also to self-approve changes
> > that you feel confident about. (we're always quite happy to provide
> > feedback whenever you feel necessary, of course, and we will send
> > comments if there is something we notice).
> 
>   I will send a first bunch of fixes shortly...
> Mainly about ChangeLog formatting issues as usual...
> 
> Pierre



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]