Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Joel Brobecker <brobecker@adacore.com>
To: Tom Tromey <tromey@redhat.com>
Cc: Eli Zaretskii <eliz@gnu.org>,
	bauerman@br.ibm.com, drow@false.org, 	pedro@codesourcery.com,
	gdb-patches@sourceware.org
Subject: Re: RFC: add ability to "source" Python code
Date: Thu, 19 Feb 2009 21:45:00 -0000	[thread overview]
Message-ID: <20090219200536.GA25486@adacore.com> (raw)
In-Reply-To: <m3vdr83izz.fsf@fleche.redhat.com>

> Eli> I'm okay with this, but I think Tom had some valid reasons for
> Eli> having Python scripts that don't have a certain extension.
> 
> It is a convenience when using scratch files.
> In practice I don't use -p much.  I don't mind removing it.

Ok, we can try without, and see if we get any request?

> I considered this early on and rejected it, because "source" is
> clearer and "felt natural".

Yeah ... I would have to agree with that.

So, where do you think we should go from now?
a tri-state setting:

  1. Use filename extension and evaluate accordingly.
     Error if the language was not compiled in.

  2, Use filename extension and evaluate accordingly.
     Fallback to GDB scripting if the language was not compiled in.

  3. No filename extension recognition, always treat sourced files
     as GDB scripts.

Would that be acceptable to everyone?

Otherwise, I'd be OK with a two state that just deactivates filename
extension recognition, and implements option (2) above when turned on,
but for some reason this fallback, which causes GDBs of the same version
to behavior differently, feels VERY wrong to me. So I'd like to have
a setting that allows me to provide behavior (1).

-- 
Joel


  reply	other threads:[~2009-02-19 20:05 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
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 [this message]
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=20090219200536.GA25486@adacore.com \
    --to=brobecker@adacore.com \
    --cc=bauerman@br.ibm.com \
    --cc=drow@false.org \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=pedro@codesourcery.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