Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Tom Tromey <tromey@redhat.com>
To: Pedro Alves <pedro@codesourcery.com>
Cc: gdb-patches@sourceware.org
Subject: Re: RFC: add ability to "source" Python code
Date: Tue, 10 Feb 2009 01:29:00 -0000	[thread overview]
Message-ID: <m3fxinulmj.fsf@fleche.redhat.com> (raw)
In-Reply-To: <200902100000.22671.pedro@codesourcery.com> (Pedro Alves's message of "Tue\, 10 Feb 2009 00\:00\:22 +0000")

>>>>> "Pedro" == Pedro Alves <pedro@codesourcery.com> writes:

Pedro> Could you explain for the archives why this is better than having
Pedro> the script itself tell GDB that it's running python?  Either
Pedro> through something similar to a shebang, or starting the script with
Pedro> "python" (does that work?) ?  I think you've discussed this before,
Pedro> what were the limitations you found?

I think you may be thinking of invoking a python script using
"#!.../gdb -something".  This patch does not cover that case.  That is
a separate patch, which I haven't submitted.

Anyway, I think having "source" look at a #! line would not be very
helpful.  Most Python source files do not start with this.

Yes, you can write:

  python
  blah blah blah
  end

in a file and then source that.  IME, this is not always convenient.

In practice, though, I find I use the .py extension feature a lot more
than "-p".  I can remove the "-p" feature if that will make you and
Eli and Doug happy.

Pedro> Say that in the future we add support for another scripting language
Pedro> to GDB, and we want to keep the current python support working in
Pedro> parallel.  As a matter of example, let's call it "Python 3.0".  Would
Pedro> we want to add more "-p" switches to the source command?

I don't think Python 3.0 is a good example.  I think we'll only be
able to link GDB against a single version of Python.

If someone adds other scripting languages to GDB, then yes, I would
support new a option to "source".  I would also support having
"source" know about common file extensions for the new language.

Tom


  reply	other threads:[~2009-02-10  1:29 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-08  1:16 Tom Tromey
2009-02-08  1:34 ` Tom Tromey
2009-02-08  4:10   ` Eli Zaretskii
2009-02-08  4:08 ` Eli Zaretskii
2009-02-09  1:53   ` Doug Evans
2009-02-09  4:09     ` Eli Zaretskii
2009-02-10  1:37       ` Tom Tromey
2009-02-10  1:30     ` Tom Tromey
2009-02-09  1:35 ` Doug Evans
2009-02-10  0:00 ` Pedro Alves
2009-02-10  1:29   ` Tom Tromey [this message]
2009-02-10  2:36     ` Pedro Alves
2009-02-10  3:48       ` Daniel Jacobowitz
2009-02-10  9:34         ` Eli Zaretskii
2009-02-10 11:58           ` Thiago Jung Bauermann
2009-02-10 17:04             ` Tom Tromey
2009-02-11  2:25               ` Paul Pluzhnikov
2009-02-11  6:09               ` Joel Brobecker
2009-02-11 19:51                 ` Tom Tromey
2009-02-11 20:21                   ` Eli Zaretskii
2009-02-11 20:39                     ` Joel Brobecker
2009-02-11 21:06                       ` Eli Zaretskii
2009-02-11 21:26                         ` Matt Rice
2009-02-11 21:49                           ` Eli Zaretskii
2009-02-11 21:55                             ` Eli Zaretskii
2009-02-11 22:01                         ` Joel Brobecker
2009-02-12  3:59                           ` Eli Zaretskii
2009-02-12  6:27                             ` Joel Brobecker
2009-02-12 20:32                               ` Thiago Jung Bauermann
2009-02-12 22:38                               ` Eli Zaretskii
2009-02-13  8:42                                 ` Joel Brobecker
2009-02-13 15:23                                   ` Eli Zaretskii
2009-02-17  0:58                                     ` Joel Brobecker
2009-02-17  5:54                                       ` Eli Zaretskii
2009-02-17 20:37                                         ` Tom Tromey
2009-02-19 21:45                                           ` Joel Brobecker
2009-06-01  3:57                                             ` Thiago Jung Bauermann
2009-06-01  5:05                                               ` Paul Pluzhnikov
2009-06-01 15:33                                                 ` Eli Zaretskii
2009-06-01 15:46                                               ` Eli Zaretskii
2009-06-01 17:54                                                 ` Thiago Jung Bauermann
2009-06-10 23:10                                               ` Tom Tromey
2009-06-11 14:19                                                 ` Joel Brobecker
2009-07-03  7:21                                                 ` Paul Pluzhnikov
2010-01-15  7:21                                               ` Joel Brobecker
2010-01-15  9:13                                                 ` Joel Brobecker
2010-01-15 18:03                                                   ` Tom Tromey
2010-01-18  6:33                                                     ` Joel Brobecker
2010-01-18 17:48                                                       ` Eli Zaretskii
2010-01-19 10:32                                                         ` Joel Brobecker
2009-02-11 20:43                     ` Daniel Jacobowitz
2009-02-11 21:08                       ` Eli Zaretskii
2009-02-11 21:16                         ` Daniel Jacobowitz
2009-02-11 21:46                           ` Eli Zaretskii
2009-02-11 20:54                     ` Tom Tromey
2009-02-11 21:11                       ` Eli Zaretskii
2009-02-11 20:46                   ` Joel Brobecker
2009-02-11 20:58                     ` Tom Tromey

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=m3fxinulmj.fsf@fleche.redhat.com \
    --to=tromey@redhat.com \
    --cc=gdb-patches@sourceware.org \
    --cc=pedro@codesourcery.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