Cython dependency checking broken

[jdemeyer]

Install the PARI spkg from #14539:

./sage -i http://boxen.math.washington.edu/home/jdemeyer/spkg/pari-2.5.4.p0.spkg

Then rebuild the library:

./sage -b

Before #13031, this used to rebuild all files depending on PARI (good!), with #13031 absolutely nothing happens (bad!).

spkg: http://sage.math.washington.edu/home/robertwb/patches/cython-0.19.p1.spkg

apply attachment: 14544-cythonize-deps.patch to the Sage library

CC: @robertwb

Component: cython

Author: Robert Bradshaw

Reviewer: Jeroen Demeyer

Merged: sage-5.10.beta2

Issue created by migration from https://trac.sagemath.org/ticket/14544

[robertwb]

comment:1

Attachment: 14544-cythonize-deps.patch.gz

This requires a fix to the Sage library as well as cython/cython@5574592

[robertwb]

comment:2

New spkg up at http://sage.math.washington.edu/home/robertwb/patches/cython-0.19.p1.spkg

[jdemeyer]

Author: Robert Bradshaw

[jdemeyer]

Merged: sage-5.10.beta2

[jdemeyer]

Reviewer: Jeroen Demeyer

[jdemeyer]

comment:4

Looks good to me.

[SnarkBoojum]

comment:5

It's not that closed in fact : what does deps.patch do? It isn't documented in SPKG.txt, and the log message only says who wrote it but not why. And since it is an upstream modification, was it forwarded?

[jdemeyer]

comment:6

The author of the patch is upstream. Anyway, #14569 solved your problem.