Opened 6 years ago

Closed 5 years ago

Last modified 4 years ago

#17607 closed enhancement (fixed)

Add a python 3 package

Reported by: ohanar Owned by:
Priority: major Milestone: sage-6.5
Component: packages: optional Keywords:
Cc: vbraun, wstein, wluebbe, aapitzsch, jdemeyer Merged in:
Authors: R. Andrew Ohana Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: df5f216 (Commits) Commit:
Dependencies: Stopgaps:

Description (last modified by ohanar)

To assist in porting to python 3 (and supposedly to use once we have a working python 3 port).

Tarball: https://www.python.org/ftp/python/3.4.3/Python-3.4.3.tgz (renamed to tar.gz)

Change History (13)

comment:1 Changed 6 years ago by ohanar

This is missing a few of the patches that were in the python 2.7 package, mainly because parts of the codebase has changed significantly and I didn't want to spend the time resolving the differences. Most of the ones cut had to do with cygwin support, the main exception being Jereon's patch for #13579.

comment:2 Changed 5 years ago by git

  • Commit changed from 90cdd2bfc398fd687e5aca7cd6744454ca7987b9 to df5f21622a1728d2c6bb58181bc030b1b0bfb3c1

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

df5f216make it possible to try to build sage with python 3

comment:3 Changed 5 years ago by ohanar

  • Status changed from new to needs_review

comment:4 follow-up: Changed 5 years ago by vbraun

tarball?

comment:5 in reply to: ↑ 4 Changed 5 years ago by ohanar

  • Description modified (diff)

Replying to vbraun:

tarball?

derp.

comment:6 Changed 5 years ago by vbraun

  • Reviewers set to Volker Braun

comment:7 Changed 5 years ago by vbraun

  • Branch changed from u/ohanar/python3 to df5f21622a1728d2c6bb58181bc030b1b0bfb3c1
  • Resolution set to fixed
  • Status changed from needs_review to closed

comment:8 Changed 5 years ago by ncohen

  • Commit df5f21622a1728d2c6bb58181bc030b1b0bfb3c1 deleted
(GRRR|…)~/.Sage/build/pkgs$ head python3/checksums.ini -n 1
tarball=Python-VERSION.tar.gz
(GRRR|…)~/.Sage/build/pkgs$ head python2/checksums.ini -n 1
tarball=python-VERSION.tar.gz

Really guys...

comment:9 Changed 5 years ago by ncohen

  • Cc jdemeyer added

comment:10 follow-up: Changed 5 years ago by jhpalmieri

Small and big question: First, when I install this on OS X 10.10.3 (using sage -i python3), I see

configure: WARNING: unrecognized options: --enable-unicode, --disable-toolbox-glue

I assume that's minor, but it should probably be cleaned up.

Second, after installing, if I do sage --python, I get

/Users/palmieri/Desktop/Sage_stuff/git/sage/src/bin/sage-env: line 411: 33785 Abort trap: 6           "$SAGE_ROOT/local/bin/python" -c 'import pkg_resources; pkg_resources.get_distribution("matplotlib").version' 2> /dev/null
Fatal Python error: Py_Initialize: unable to load the file system codec
  File "/Users/palmieri/Desktop/Sage_stuff/git/sage/local/lib/python/encodings/__init__.py", line 123
    raise CodecRegistryError,\
                            ^
SyntaxError: invalid syntax
/usr/local/bin/sage: line 134: 33763 Abort trap: 6           "$SAGE_ROOT/src/bin/sage" "$@"

This breaks Sage completely. I don't mind having an optional package which doesn't build for me, but I don't think we should have one that breaks Sage.

comment:11 in reply to: ↑ 10 Changed 5 years ago by ohanar

John, would you make a follow up ticket and cc me?

comment:12 Changed 5 years ago by jhpalmieri

Done: it's #18521.

comment:13 Changed 4 years ago by jdemeyer

Funny, you added a site_packages (sic!) directory to PYTHONPATH. That directory doesn't exist, but site-packages exists. See #21688

Last edited 4 years ago by jdemeyer (previous) (diff)
Note: See TracTickets for help on using tickets.