Opened 15 years ago

Closed 13 years ago

#139 closed enhancement (fixed)

automatic dependency checking for pyrex files (implemented long ago--closed now)

Reported by: was Owned by: was
Priority: minor Milestone: sage-2.10.3
Component: user interface Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

Add code to devel/sage/setup.py so that a Pyrex file is pyrexed if it changes *or* if any pxd file that it cimports changes.

Change History (6)

comment:1 Changed 15 years ago by dmharvey

It would also be good to check any .pxi files that are included. Maybe even any .h files that are referenced, because the C file compilation will depend on this too.

comment:2 Changed 15 years ago by was

I've made a change so when you do a "sage -upgrade" all .pyx files get rebuilt. This is obviously slower, but will avoid a lot of stupid confusion for now.

comment:3 Changed 15 years ago by was

  • Checking for dependencies on .h files doesn't work at all.
  • Recursive dependencies don't work, i.e., if a depends on b and b on c, and

c changes, then a isn't rebuilt.

comment:4 Changed 14 years ago by mabshoff

  • Milestone set to sage-feature

comment:5 Changed 13 years ago by malb

Is this still a valid ticket? We do check dependencies now, right?

comment:6 Changed 13 years ago by was

  • Milestone changed from sage-feature to sage-2.10.3
  • Resolution set to fixed
  • Status changed from new to closed
  • Summary changed from automatic dependency checking for pyrex files to automatic dependency checking for pyrex files (implemented long ago--closed now)

I implemented this and forgot to close the ticket.

Note: See TracTickets for help on using tickets.