Opened 3 years ago

Closed 3 years ago

#27260 closed defect (fixed)

zn_poly tries to run "python" and "2to3"

Reported by: jdemeyer Owned by:
Priority: major Milestone: sage-8.7
Component: build Keywords:
Cc: Merged in:
Authors: Jeroen Demeyer Reviewers: Travis Scrimshaw
Report Upstream: Fixed upstream, but not in a stable release. Work issues:
Branch: 87fdcde (Commits, GitHub, GitLab) Commit: 87fdcde8865e9d57a3f5fce6d4273a5696438b4e
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

This breaks on systems where only python3 is installed but not python or 2to3.

Upstream patch: ​https://gitlab.com/sagemath/zn_poly/merge_requests/2

Change History (6)

comment:1 Changed 3 years ago by jdemeyer

  • Description modified (diff)
  • Summary changed from zn_poly tries to run "python" to zn_poly tries to run "python" and "2to3"

comment:2 Changed 3 years ago by jdemeyer

  • Branch set to u/jdemeyer/zn_poly_tries_to_run__python_

comment:3 Changed 3 years ago by jdemeyer

  • Authors set to Jeroen Demeyer
  • Commit set to 87fdcde8865e9d57a3f5fce6d4273a5696438b4e
  • Description modified (diff)
  • Report Upstream changed from N/A to Reported upstream. No feedback yet.
  • Status changed from new to needs_review

New commits:

​87fdcdeDrop dependency of zn_poly on "python" and "2to3"

comment:4 Changed 3 years ago by tscrim

  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

Upstream approves. LGTM.

comment:5 Changed 3 years ago by jdemeyer

  • Report Upstream changed from Reported upstream. No feedback yet. to Fixed upstream, but not in a stable release.

comment:6 Changed 3 years ago by vbraun

  • Branch changed from u/jdemeyer/zn_poly_tries_to_run__python_ to 87fdcde8865e9d57a3f5fce6d4273a5696438b4e
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.