unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Ken Manheimer" <ken.manheimer@gmail.com>
To: "Nick Roberts" <nickrob@snap.net.nz>
Cc: cyd@stupidchicken.com, emacs-devel@gnu.org, sdl.web@gmail.com,
	rms@gnu.org
Subject: Re: python.el fixes for Emacs 22
Date: Fri, 15 Feb 2008 19:22:56 -0500	[thread overview]
Message-ID: <2cd46e7f0802151622s8e191et63074c2cc6211610@mail.gmail.com> (raw)
In-Reply-To: <18358.9949.509332.71127@kahikatea.snap.net.nz>

On Feb 15, 2008 6:57 PM, Nick Roberts <nickrob@snap.net.nz> wrote:

>  > > I never installed that patch because I was uncertain about the authorship
>  > > of the parts of python-mode.el that I was taking from.
>  >
>  > i believe that any changes to the code for pdb tracking functionality were
>  > trivial, and that my copyright assignment would be sufficient for that part
>  > of it.  i can check with barry, who incorporated the stand-alone code i had
>  > written (as "pdbtrack.el") into python-mode.el
>
> About 90% of the patch is for pdbtrack, mostly taken verbatim after the line:
>
> ;; pdbtrack features
>
> The other 10% is generally defining defcustoms that pdbtrack uses.

i think it's a pretty safe bet that i could claim responsibility for
that code - and i've contacted barry just to be sure it's ok with him,
since he did the retrofitting to python-mode.el

>  > > I still have a local copy of the patch updated to the trunk and could
>  > > install it now if instructed to do so.
>  > >
>  > > There appear to be two options:
>  > >
>  > > 1) Install my patch in python.el which may in reality only involve your
>  > >   contribution to python-mode.el and may not require other signatures.
>  > >
>  > > 2) Replace python.el with a modified python-mode.el which would require
>  > >   more signatures (Barry Warsaw, Skip Montanero?) and more work.
>  >
>  >
>  > it sounds like you have done some substantial comparison between the two
>  > implementations of python-mode, at least when you were working on the
>  > patches.  can you characterize the differences between the two?  i never
>  > tried python.el very far, because it lacked pdb tracking - but i've always
>  > been favorably impressed with python-mode.el, and would be expect it to
>  > offer a good basis to work from.
>
> Firstly, as a disclaimer, I'm not a Python programmer.  My curiosity came from
> working on gud.el and M-x pdb.  So I can't characterise the differences between
> the two but I'm sure someone else on the list can.

i do hope someone could describe the differences.  i'm curious why
something besides python-mode.el was even necessary in the first
place.  just copyright assignment obstacles?

python-mode.el has been around almost as long as python has been
available on the internet.  though he might be horrified by the
gushing praise, tim peters, the original author, is one of the single
most skilled programmers around.  he no longer uses emacs, but his
python-mode has always worked outstandingly well.  barry warsaw, who
took up maintenance after tim, is no slouch as well, and has been
diligent.  other skilled, central python folk have been involved, and
i suspect python-mode.el offers a trove of good code, worth examining
for some kind of consolidation with python.el.  i don't have time,
myself, or i'd stop trying to sell the idea and just do it...

>  > i'm planning to contact barry and skip to see about copyright assignments -
>  > would that help?  would you be willing to revisit your patches and bring
>  > them up to date, if clearance for python-mode.el became possible?
>
> I think they are up to date.  I've just tried to take pdbtrack from
> python-mode.el and adapt it for python.el.  I can't remember the details
> but I think I made minor changes like making the hooks added to
> comint-output-filter-functions local.
>
> My intention was just to get pdbtrack into python.el and leave it to those
> who use Python to make improvements.

thanks much for doing that.  i hope the patch can land, now.
-- 
ken
http://myriadicity.net




  reply	other threads:[~2008-02-16  0:22 UTC|newest]

Thread overview: 81+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-11 13:34 python.el fixes for Emacs 22 Richard Stallman
2008-02-11 15:15 ` Stefan Monnier
2008-02-11 21:10   ` Richard Stallman
2008-02-12  3:04     ` Stefan Monnier
2008-02-12 17:44       ` Richard Stallman
2008-02-12 21:40         ` Stefan Monnier
2008-02-13 16:32           ` Richard Stallman
2008-02-13 19:19             ` Stefan Monnier
2008-02-14  0:36 ` Chong Yidong
2008-02-14  0:51   ` Miles Bader
2008-02-14  1:56   ` Leo
2008-02-14 18:11     ` Richard Stallman
2008-02-14 23:42       ` Leo
2008-02-15  0:01         ` Nick Roberts
2008-02-15  0:08           ` Leo
2008-02-15  0:17             ` Miles Bader
2008-02-15  2:10             ` Nick Roberts
2008-02-15 17:57             ` Ken Manheimer
2008-02-16  5:53             ` Richard Stallman
2008-02-16  8:18               ` Leo
2008-02-16  8:35                 ` Miles Bader
2008-02-16 14:45                 ` Stefan Monnier
2008-02-16 14:51                   ` Miles Bader
2008-02-17  3:44                     ` Stephen J. Turnbull
2008-02-17 13:22                 ` Richard Stallman
2008-02-17 13:26                 ` Piet van Oostrum
2008-02-15 12:59           ` Richard Stallman
2008-02-15 13:27             ` David Kastrup
2008-02-15 17:43               ` Ken Manheimer
2008-02-15 21:47                 ` Nick Roberts
2008-02-15 22:21                   ` Ken Manheimer
2008-02-15 23:57                     ` Nick Roberts
2008-02-16  0:22                       ` Ken Manheimer [this message]
2008-02-16  0:39                         ` python.el versus python-mode.el [was Re: python.el fixes for Emacs 22] Glenn Morris
2008-02-17 13:22                   ` python.el fixes for Emacs 22 Richard Stallman
2008-02-17 13:22                 ` Richard Stallman
2008-02-20  0:30                   ` Nick Roberts
2008-02-20  2:24                     ` pdbtrack [was Re: python.el fixes for Emacs 22] Glenn Morris
2008-02-20  2:34                       ` Nick Roberts
2008-02-20  2:56                         ` pdbtrack Glenn Morris
2008-02-21 22:40                         ` pdbtrack [was Re: python.el fixes for Emacs 22] Ken Manheimer
2008-02-21 23:35                           ` pdbtrack Glenn Morris
2008-02-22 18:00                             ` pdbtrack Ken Manheimer
2008-02-22 18:35                               ` pdbtrack Stefan Monnier
2008-02-23 23:09                                 ` pdbtrack Ken Manheimer
2008-02-22 20:08                               ` pdbtrack Nick Roberts
2008-02-23 23:16                                 ` pdbtrack Ken Manheimer
2008-02-24  4:49                                   ` pdbtrack Nick Roberts
2008-02-24 15:40                                     ` pdbtrack Stefan Monnier
2008-02-24 17:00                                       ` pdbtrack Ken Manheimer
2008-02-24 20:44                                         ` pdbtrack Stefan Monnier
2008-02-21  0:20                     ` python.el fixes for Emacs 22 Ken Manheimer
2008-02-21  4:02                       ` Stefan Monnier
2008-02-21  5:12                         ` Barry Warsaw
2008-02-21 22:28                           ` Richard Stallman
2008-02-21 23:00                             ` Barry Warsaw
2008-02-21 23:08                               ` Ken Manheimer
2008-02-21 23:12                                 ` Barry Warsaw
2008-02-22  1:49                                   ` Stefan Monnier
2008-02-22 13:44                                     ` Barry Warsaw
2008-02-22 15:13                                       ` skip
2008-02-22 15:30                                         ` Barry Warsaw
2008-02-22 22:57                               ` Richard Stallman
2008-02-22 22:57                               ` Richard Stallman
2008-02-22  3:23                             ` Stephen J. Turnbull
2008-02-22 13:45                               ` Barry Warsaw
2008-02-22 16:28                                 ` Stefan Monnier
2008-02-22 17:05                                   ` Barry Warsaw
2008-02-22 17:13                                     ` Ken Manheimer
2008-02-22 18:27                                     ` Stefan Monnier
2008-02-22 19:38                                       ` Barry Warsaw
2008-02-23 19:28                                         ` Richard Stallman
2008-02-23 20:16                                           ` skip
2008-02-22 22:57                                 ` Richard Stallman
2008-02-22 23:33                                   ` Barry Warsaw
2008-02-23 19:29                                     ` Richard Stallman
2008-02-23  0:09                                   ` skip
2008-02-25 13:53                                     ` Bernhard Herzog
2008-02-21  5:09                       ` Barry Warsaw
2008-02-21  5:22                       ` Nick Roberts
2008-02-15 16:51             ` Chong Yidong

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2cd46e7f0802151622s8e191et63074c2cc6211610@mail.gmail.com \
    --to=ken.manheimer@gmail.com \
    --cc=cyd@stupidchicken.com \
    --cc=emacs-devel@gnu.org \
    --cc=nickrob@snap.net.nz \
    --cc=rms@gnu.org \
    --cc=sdl.web@gmail.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.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).