Possible bug: SAGE_PYTHON_VERSION not set

[John Cremona]

In a new install of version 9.2 (on ubuntu) I was using sage -pip
install to install several packages which I need for LMFDB
development. All but one of these are normal pip packages which
install fine, but one is not: it is specified in the requirements.txt
file as
git+https://github.com/jwbober/conrey-dirichlet-characters.git@master#egg=dirichlet_conrey

The installation of this (which has always worked fine up to now)
bailed out with the error:

Collecting dirichlet_conrey
Cloning https://github.com/jwbober/conrey-dirichlet-characters.git
(to revision master) to /tmp/pip-install-r1tfb33j/dirichlet-conrey
ERROR: Command errored out with exit status 1:
command: /usr/local/sage/sage-9.2/local/bin/python3 -c 'import
sys, setuptools, tokenize; sys.argv[0] =
'"'"'/tmp/pip-install-r1tfb33j/dirichlet-conrey/setup.py'"'"';
__file__='"'"'/tmp/pip-install-r1tfb33j/dirichlet-conrey/setup.py'"'"';f=getattr(tokenize,
'"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"',
'"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))'
egg_info --egg-base /tmp/pip-pip-egg-info-xvkfdork
cwd: /tmp/pip-install-r1tfb33j/dirichlet-conrey/
Complete output (1 lines):
ERROR: The environment variable SAGE_PYTHON_VERSION must be defined.

I worked out that after putting

export SAGE_PYTHON_VERSION=3

and rerunning the pip install, all was well. Should Sage's
environment (e.g. what you see typing env after sage -sh) have this
set?

John

[Dima Pasechnik]

IIRC we have removed this variable lately, it was not documented, and so we felt no need to do a deprecation.

The upstream ought to remove it.

--
You received this message because you are subscribed to the Google Groups "sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/sage-devel/CAD0p0K4YDCJ3C%2BkFQ24QCJW1oUz9Gw27wZDqZURS8LKUQH4pgQ%40mail.gmail.com.

[Dima Pasechnik]

On Mon, Nov 16, 2020 at 3:54 PM Dima Pasechnik <dim...@gmail.com> wrote:
>
> IIRC we have removed this variable lately, it was not documented, and so we felt no need to do a deprecation.
>
> The upstream ought to remove it.

PR for upstream here:
https://github.com/jwbober/conrey-dirichlet-characters/pull/12

[Matthias Koeppe]

On Monday, November 16, 2020 at 7:55:15 AM UTC-8 Dima Pasechnik wrote:

IIRC we have removed this variable lately,

in https://trac.sagemath.org/ticket/30546

 

[John Cremona]

Thanks for that -- I had not realised that it was an upstream issue.   I'll get Bober to merge your PR -- thanks for doing that Dima.