emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: "Andreas Röhler" <andreas.roehler@easy-emacs.de>
To: emacs-orgmode@gnu.org
Cc: stardiviner <numbchild@gmail.com>, Jack Kamm <jackkamm@gmail.com>,
	python-mode@python.org
Subject: Re: [PATCH] Fix ob-python.el initiate session error with py-shell
Date: Thu, 20 Feb 2020 17:26:04 +0100	[thread overview]
Message-ID: <20aad61e-48e0-ba01-9255-793eacfdbc7e@easy-emacs.de> (raw)
In-Reply-To: <87d0a995hp.fsf@gmail.com>

,

On 20.02.20 06:45, stardiviner wrote:

Hi,
python-mode.el developer here.
>> Since python.el is built-in to Emacs, I think that things should
>> continue to work for python-mode.el users, even if we entirely switch
>> over to only using python.el.

Thanks supporting python-mode.el. Both python-modes should not conflict 
- except for the key-setting, which is taken by the last one loaded:

py-mode-map didn't work for some reasons, so both provide now 
python-mode-map.

>>
>> The only thing I'm not sure about, is whether the equivalent of
>> "python-shell-send-region" in python-mode.el will work with shells
>> started by python.el. If not, this could probably be addressed with a
>> patch to python-mode.el.

It should work. Will check it. Please feel free to send a bug-report 
resp. feature request to

https://gitlab.com/python-mode-devs/python-mode/issues

in case of python-mode.el related stuff.


>> You could test this by explicitly setting `org-babel-python-mode' to
>> `python' (it is probably set to `python-mode' on your system, which is
>> the default when python-mode.el is detected).
>>
>> I'll add a TODO for myself to explicitly mark python-mode-related
>> variables as deprecated. I'm also planning a major update to the Worg
>> documentation of ob-python when 9.4 comes out, and will mention the
>> deprecation there as well.
> Thanks for your work! Jack
>
> - -- 
> [ stardiviner ]

BTW there are some reasons, why python-mode.el still exists: no known 
indentation bugs, finer navigation commands etc. Maybe have a look at 
open python.el bugs and check them against python-mode.el

Best,

Andreas

  reply	other threads:[~2020-02-20 16:26 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-18 12:55 [PATCH] Fix ob-python.el initiate session error with py-shell stardiviner
2020-02-18 13:07 ` Bastien
2020-02-18 17:07 ` Jack Kamm
2020-02-18 17:18   ` Jack Kamm
2020-02-18 20:23     ` Bastien
2020-02-19  7:38     ` numbchild
2020-02-19 15:51       ` Jack Kamm
2020-02-19 16:03         ` Bastien
2020-02-20  5:45         ` stardiviner
2020-02-20 16:26           ` Andreas Röhler [this message]
2020-02-20 17:46             ` Bastien
2020-02-20 18:51               ` Jack Kamm
2020-02-20 19:08                 ` Bastien
2020-02-22 14:28               ` stardiviner

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.orgmode.org/

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

  git send-email \
    --in-reply-to=20aad61e-48e0-ba01-9255-793eacfdbc7e@easy-emacs.de \
    --to=andreas.roehler@easy-emacs.de \
    --cc=emacs-orgmode@gnu.org \
    --cc=jackkamm@gmail.com \
    --cc=numbchild@gmail.com \
    --cc=python-mode@python.org \
    /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/org-mode.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).