From: Pedro Alves <pedro@codesourcery.com>
To: gdb-patches@sourceware.org
Cc: "Pierre Muller" <muller@ics.u-strasbg.fr>
Subject: Re: [RFA] testsuite/gdb.base/fullname.exp fix
Date: Sun, 21 Jun 2009 01:27:00 -0000 [thread overview]
Message-ID: <200906210227.48175.pedro@codesourcery.com> (raw)
In-Reply-To: <000501c9f12b$68c860e0$3a5922a0$@u-strasbg.fr>
On Friday 19 June 2009 23:15:08, Pierre Muller wrote:
>
> fullname.exp currently fails compilation
> of windows targets because it tries to recompile
> the executable, while it is still running under GDB.
Might this be a new failure? We have a few bfd_cache_close_all
calls around to prevent this. Hmm, probably not. Anyone
thinks we should do something to prevent this from happening?
AFAICS, this particular case is something like:
> gdb filefoo
(gdb opens the exec_file does things, and the calls bfd_cache_close_all.)
(gdb) b main << this reads file memory and reopens the bfd, nothing closes it.
Maybe we could always call bfd_cache_close_all after executing a
command... Dunno, maybe that's going too far? Probably will still be
defeated by the new dwarf memory mapped files support if that ever
is ported to Windows (I think Cygwin has now some magic to be able
to remove open files? Maybe I'm dreaming.)
On Friday 19 June 2009 23:15:08, Pierre Muller wrote:
> 2009-06-19 Pierre Muller <muller@ics.u-strasbg.fr>
>
> * fullname.exp: Exit gdb before recompile as some operating systems
> lock the executable while it is running.
>
This is OK if it isn't a new failure, and no one has replies to the
above questions, and if you move the "why" note from the changelong entry
to the .exp file.
The changelog header indicates "who" and "when". The changelog entry
itself indicates "what". The "why" belongs in the code.
--
Pedro Alves
next prev parent reply other threads:[~2009-06-21 1:27 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-19 22:15 Pierre Muller
2009-06-21 1:27 ` Pedro Alves [this message]
2009-06-21 13:48 ` Eli Zaretskii
2009-06-21 16:45 ` Pierre Muller
2009-06-21 23:17 ` Eli Zaretskii
2009-06-25 22:29 ` Pierre Muller
2009-06-27 14:39 ` Eli Zaretskii
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=200906210227.48175.pedro@codesourcery.com \
--to=pedro@codesourcery.com \
--cc=gdb-patches@sourceware.org \
--cc=muller@ics.u-strasbg.fr \
/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