Opened 5 years ago

Closed 5 years ago

#21076 closed enhancement (fixed)

make pyx a new style package

Reported by: vdelecroix Owned by:
Priority: critical Milestone: sage-7.3
Component: packages: optional Keywords:
Cc: vbraun, jdemeyer Merged in:
Authors: Vincent Delecroix Reviewers: Jeroen Demeyer
Report Upstream: N/A Work issues:
Branch: e7bb30b (Commits, GitHub, GitLab) Commit: e7bb30b5387e1f122e3227960e6d70abb3ed635e
Dependencies: Stopgaps:

Status badges

Description (last modified by vdelecroix)

The installation of the old style package pyx breaks pip for encoding reason. 

$ sage - i pyx
SNIP installation log
$ sage -pip uninstall pyx
Traceback (most recent call last):
  File "/opt/sage/local/bin/pip", line 5, in <module>
    from pkg_resources import load_entry_point
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 2927, in <module>
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 2913, in _call_aside
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 2940, in _initialize_master_working_set
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 626, in _build_master
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 619, in __init__
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 675, in add_entry
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 1982, in find_on_path
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 2370, in from_location
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 2687, in _reload_version
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 2335, in _version_from_file
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 2503, in _get_metadata
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 1873, in get_metadata_lines
  File "build/bdist.linux-x86_64/egg/pkg_resources/__init__.py", line 1863, in get_metadata
  File "/opt/sage/local/lib/python/codecs.py", line 302, in decode
    (result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf8' codec can't decode byte 0xf6 in position 165: invalid start byte in /opt/sage/local/lib/python2.7/site-packages/PyX-0.10-py2.7.egg-info

We propose to make pyx a new style package. Upstream tarball at

​https://pypi.python.org/packages/27/04/3e13fab75ea4d7e390af41b732a00709557c9b46a1d598b40a68bc312503/pyx-0.12.1.tar.gz

See the discussion at

​https://groups.google.com/forum/#!topic/sage-devel/TVMnUmj8U0k

and also #20472

Change History (7)

comment:1 Changed 5 years ago by vdelecroix

  • Description modified (diff)

comment:2 Changed 5 years ago by vdelecroix

Sadly pyx is now Python 3 only and we need to do a version specific installation 

$ sage -pip install pyx==0.12.1

comment:3 Changed 5 years ago by vdelecroix

  • Description modified (diff)

comment:4 Changed 5 years ago by vdelecroix

  • Branch set to u/vdelecroix/21076
  • Commit set to e7bb30b5387e1f122e3227960e6d70abb3ed635e
  • Status changed from new to needs_review

New commits:

​e7bb30bTrac 21076: make pyx a new style package

comment:5 Changed 5 years ago by vdelecroix

  • Description modified (diff)
  • Summary changed from make pyx a new style pip package to make pyx a new style package

comment:6 Changed 5 years ago by jdemeyer

  • Reviewers set to Jeroen Demeyer
  • Status changed from needs_review to positive_review
  • Type changed from defect to enhancement

comment:7 Changed 5 years ago by vbraun

  • Branch changed from u/vdelecroix/21076 to e7bb30b5387e1f122e3227960e6d70abb3ed635e
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.