GNU bug report logs -
#70815
[PATCH] ; Enahnce python-tests.el to adapt different python interpreters
Previous Next
Reported by: Lin Sun <sunlin7.mail <at> gmail.com>
Date: Tue, 7 May 2024 06:54:02 UTC
Severity: normal
Tags: patch
Done: Eli Zaretskii <eliz <at> gnu.org>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
Mattias EngdegÄrd wrote:
>
> 4 juni 2024 kl. 16.29 skrev kobarity <kobarity <at> gmail.com>:
>
> > The attached is the revised patch. In order to bind
> > `python-shell-interpreter' consistently, new macros
> > `python-tests-with-shell-interpreter' and
> > `python-tests-with-temp-buffer-with-shell-interpreter' were
> > introduced. The patch is larger because some of the indentations have
> > been changed by using new macros. I have fixed
> > `python-shell-interpreter' was not bound correctly in some ERTs, so
> > you might find new ERTs that fail with Python 2 on Mac.
>
> It works here.
>
> > 1. Set EMACS_PYTHON_INTERPRETER environment variable to "python" and
> > run ERTs. The expectation is that the 7 ERTs that previously
> > resulted in errors will be skipped.
>
> Yes.
>
> > 2. Change the order of the interpreters in
> > `python-tests-get-shell-interpreter' to "python" "python3"
> > "python2" and run ERTs. The expectation is that the 7 ERTs that
> > previously failed will succeed.
>
> Yes.
>
> > However, if the interpreter specified by EMACS_PYTHON_INTERPRETER is
> > not found, it is likely that EMACS_PYTHON_INTERPRETER is incorrectly
> > configured, and I would prefer to let it fail rather than let it be
> > skipped.
>
> Yes, which is what happens with your patch, and I agree that this is the best outcome.
Thanks for the early confirmation.
> Would you like me to push your patch?
Thanks again. I will wait a few days to see if I can get feedback
from others.
This bug report was last modified 1 year and 40 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.