Opened 9 years ago

Last modified 9 years ago

#14331 closed task

remove explict references to SAGE_ROOT in the doctesting framework — at Version 5

Reported by: ohanar Owned by: jason
Priority: major Milestone: sage-5.9
Component: misc Keywords:
Cc: jdemeyer, roed Merged in:
Authors: R. Andrew Ohana Reviewers: David Roe
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by ohanar)

The new doctesting framework expects the sage library's source code to be in SAGE_ROOT/devel/sage which will no longer be the case once we switch layouts. Using the SAGE_SRC environment variable will make the transition much easier.

Apply trac14331.patch then trac14331_missed.patcho the sage library.

Change History (7)

Changed 9 years ago by ohanar

apply to sage library

comment:1 Changed 9 years ago by ohanar

  • Cc jdemeyer roed added
  • Description modified (diff)
  • Status changed from new to needs_review

comment:2 Changed 9 years ago by roed

  • Reviewers set to David Roe
  • Status changed from needs_review to positive_review

Looks good to me.

comment:3 Changed 9 years ago by fbissey

Definitely want this in. We didn't spend so much time on #13432 to be undone by the patch bomb from the new doctesting frame work. We want it ASAP. Thanks for spotting it Andrew and thanks for the review David.

Hopefully it doesn't interfere with any of the follow up of #12415.

Changed 9 years ago by ohanar

apply to sage library

comment:4 Changed 9 years ago by ohanar

  • Status changed from positive_review to needs_work

comment:5 Changed 9 years ago by ohanar

  • Description modified (diff)
  • Status changed from needs_work to needs_review

I missed one disguised reference in my first patch, so I've attached a fix for that.

Note: See TracTickets for help on using tickets.