Opened 3 years ago

Last modified 3 years ago

#25288 new enhancement

Enable Cython caching again

Reported by: saraedum Owned by:
Priority: major Milestone: sage-8.5
Component: cython Keywords:
Cc: jdemeyer, roed, robertwb, vdelecroix Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #25292, #25293 Stopgaps:

Status badges

Description (last modified by saraedum)

In #17851 we disabled Cython caching. We should fix the underlying issues:

And then enable caching again :)

See #25292 for the Cython upgrade (when it comes out)

Change History (7)

comment:1 Changed 3 years ago by jdemeyer

  • Dependencies set to #25292
  • Description modified (diff)

comment:2 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:3 Changed 3 years ago by saraedum

  • Dependencies changed from #25292 to #25292, #25293
  • Description modified (diff)

comment:4 Changed 3 years ago by vdelecroix

  • Milestone changed from sage-8.3 to sage-8.4

update milestone 8.3 -> 8.4

comment:5 Changed 3 years ago by embray

  • Milestone changed from sage-8.4 to sage-8.5

Excited for this--hopefully we can get it in 8.5.

comment:6 Changed 3 years ago by jdemeyer

On #25292, you were doubting that Sage should upgrade to Cython 0.29 (which is a requirement for this ticket).

comment:7 Changed 3 years ago by embray

I have no problem if sage-the-distribution upgrades to Cython 0.29; I just think it shouldn't depend on it unconditionally. E.g. it would be good to use a version check before enabling features like this that depend on a specific Cython version.

Note: See TracTickets for help on using tickets.