From: Phil Muldoon <pmuldoon@redhat.com>
To: Tom Tromey <tromey@redhat.com>
Cc: Joel Brobecker <brobecker@adacore.com>,
iam ahal <hal9000ed2k@gmail.com>,
gdb-patches@sourceware.org
Subject: Re: [patch] GDB 7.2: new feature for "backtrace" that cuts path to file (remain filename)
Date: Tue, 28 Jun 2011 22:36:00 -0000 [thread overview]
Message-ID: <m3mxh1oa8a.fsf@redhat.com> (raw)
In-Reply-To: <m3sjqt67pe.fsf@fleche.redhat.com> (Tom Tromey's message of "Tue, 28 Jun 2011 14:07:25 -0600")
Tom Tromey <tromey@redhat.com> writes:
> Joel> Yeah, I am wondering which way would be best. It seems like a Python
> Joel> backtrace decorator would already work, or could be made to work.
>
> We have a plan to implement "frame filters" in Python. The idea here is
> basically pretty-printing for backtraces -- let libraries ship Python
> code to modify frames while they are being displayed. E.g., the Python
> interpreter could replace C frames with synthetic frames representing
> the Python stack.
>
> Full details are available somewhere -- either archer or gdb list
> archives. If you want to read them and can't find them, let me know,
> and I will either dig them up or write them again.
>
> Phil would have to say the status of this work.
This is long term work. I hope to have it for 7.4. The task is
complex, because I think the backtrace code right now is not too callback
friendly. That being said, every new Python feature requires one to
become somewhat of an expert in that area. This is happy, if sometimes
long work.
Okay, enough excuses. We need more GDB-Python hackers! I don't want to
disappear down this hole for too long as there are other tasks that
are shorter term that I think are more important, and bugs that also
need addressing. Tom and other have been doing a super job reviewing
existing Python patches, but I really need to do more of that. If only
not to collide (like I did this week!) with the few others working in
this space. Anyway, the backtrace work is neat stuff. It will be
implemented sooner rather than later.
> Joel> So, there are pluses and minuses on both ends. I am wondering
> Joel> what everyone else thinks...
>
> I am happy adding any vaguely sensible setting that people want to the
> core. Why not? The time more minimalism has long passed.
Me too. Feature rich at this point. Everyone wants something different
out of GDB, and debugging scenarios tend to be unique. But I think some
features offer a common-ground, and this is one of them.
> Whether this one meets the bar, I don't know. Is basename really the
> obvious transform to apply? What about just dropping the compilation
> directory?
Well in this context, user-driven needs, to me, are the best bar. If
this contributor has written this patch, with a specific need (not
super-specialised) in mind that is great. I wish I had the backtrace
Pythonic interface ready, but, OTOH, every use-case is great to mould
that functionality.
Cheers,
Phil
next prev parent reply other threads:[~2011-06-28 22:36 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-26 20:00 iam ahal
2011-06-26 20:49 ` Phil Muldoon
2011-06-27 16:00 ` Joel Brobecker
2011-06-27 16:18 ` Phil Muldoon
2011-06-28 20:08 ` Tom Tromey
2011-06-28 22:36 ` Phil Muldoon [this message]
2011-07-03 18:12 ` iam ahal
2011-07-03 21:13 ` Eli Zaretskii
2011-07-04 11:26 ` iam ahal
2011-07-04 12:05 ` Eli Zaretskii
2011-07-04 21:47 ` Joel Brobecker
2011-07-05 4:35 ` Eli Zaretskii
2011-07-19 14:43 ` Pedro Alves
2011-07-05 8:38 ` iam ahal
2011-07-19 14:19 ` Pedro Alves
2011-07-17 19:24 ` iam ahal
2011-07-19 13:28 ` iam ahal
2011-07-19 17:04 ` Eli Zaretskii
2011-07-24 21:12 ` iam ahal
2011-07-26 14:17 ` iam ahal
2011-07-28 15:34 ` Tom Tromey
2011-07-28 15:57 ` Tom Tromey
2011-07-28 16:36 ` Joel Brobecker
2011-07-28 17:39 ` Tom Tromey
2011-07-28 17:51 ` Tom Tromey
2011-07-29 12:01 ` Joel Brobecker
2011-07-29 12:36 ` Eli Zaretskii
2011-08-02 19:41 ` iam ahal
2011-08-03 17:45 ` Tom Tromey
2011-10-30 19:52 ` iam ahal
2011-11-02 19:06 ` Tom Tromey
2011-11-02 22:53 ` Doug Evans
2011-12-04 15:52 ` iam ahal
2011-12-04 16:55 ` Eli Zaretskii
2011-12-04 18:41 ` iam ahal
2011-12-04 19:01 ` Pedro Alves
2011-12-04 19:56 ` Eli Zaretskii
2011-12-04 21:00 ` Pedro Alves
2011-12-05 3:54 ` Eli Zaretskii
2011-12-05 5:17 ` Eli Zaretskii
2011-12-06 13:03 ` Pedro Alves
2011-12-06 14:04 ` Eli Zaretskii
2011-12-06 18:00 ` Doug Evans
2011-12-06 20:45 ` Tom Tromey
2011-12-07 8:00 ` Eli Zaretskii
2012-03-10 20:15 ` iam ahal
2012-03-11 1:22 ` asmwarrior
2012-03-12 13:10 ` iam ahal
2012-03-14 16:11 ` Tom Tromey
2012-03-14 16:27 ` Jan Kratochvil
2012-03-14 17:40 ` Eli Zaretskii
2012-03-15 22:46 ` Jan Kratochvil
2012-03-18 18:30 ` iam ahal
2012-03-18 18:35 ` Jan Kratochvil
2012-04-06 14:22 ` Jan Kratochvil
2012-03-18 20:46 ` Eli Zaretskii
2012-03-25 19:27 ` iam ahal
2012-03-25 19:31 ` Jan Kratochvil
2012-03-25 21:23 ` Eli Zaretskii
2011-12-06 12:50 ` Pedro Alves
2011-12-06 20:40 ` Tom Tromey
2011-12-06 23:02 ` Jan Kratochvil
2011-07-29 13:35 ` Jan Kratochvil
2011-08-01 18:04 ` Tom Tromey
2011-06-29 10:09 ` Andrew Burgess
2011-06-29 16:06 ` Joel Brobecker
2011-07-03 18:15 ` Daniel Jacobowitz
2011-06-28 20:08 ` Tom Tromey
2012-04-09 15:39 ` Jan Kratochvil
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=m3mxh1oa8a.fsf@redhat.com \
--to=pmuldoon@redhat.com \
--cc=brobecker@adacore.com \
--cc=gdb-patches@sourceware.org \
--cc=hal9000ed2k@gmail.com \
--cc=tromey@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