Opened 13 years ago

Closed 13 years ago

#3721 closed defect (fixed)

[with patch, positive review] Use SAGE_TESTDIR for dsage unit tests

Reported by: tabbott Owned by: yi
Priority: minor Milestone: sage-3.1
Component: dsage Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

dsage tests currently ignore the SAGE_TESTDIR environment variable, resulting in permission denied errors for users who don't have write access to their Sage install.

I've attached a patch to fix this.

Attachments (1)

sage_scripts-dsage-testdir.patch (1.3 KB) - added by tabbott 13 years ago.

Download all attachments as: .zip

Change History (4)

Changed 13 years ago by tabbott

comment:1 Changed 13 years ago by mabshoff

  • Milestone set to sage-3.1.1

comment:2 Changed 13 years ago by mabshoff

  • Milestone changed from sage-3.1.1 to sage-3.1
  • Summary changed from [with patch; needs review] Use SAGE_TESTDIR for dsage unit tests to [with patch, positive review] Use SAGE_TESTDIR for dsage unit tests

Patch looks good to me. Positive review.

Cheers,

Michael

comment:3 Changed 13 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from new to closed

Merged in Sage 3.1.alpha0

Note: See TracTickets for help on using tickets.