Opened 9 years ago

Closed 9 years ago

#11893 closed defect (duplicate)

Fix problems with doctesting non-library python files

Reported by: jdemeyer Owned by:
Priority: blocker Milestone: sage-duplicate/invalid/wontfix
Component: scripts Keywords:
Cc: jhpalmieri Merged in:
Authors: Reviewers: John Palmieri, Leif Leonhardy
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by jhpalmieri)

This was supposed to be a follow-up to #9739, but it turns out it is unnecessary: the relevant patch was already merged at #9739.

Attachments (1)

trac_11893.patch (1.4 KB) - added by jdemeyer 9 years ago.

Download all attachments as: .zip

Change History (9)

Changed 9 years ago by jdemeyer

comment:1 Changed 9 years ago by jdemeyer

  • Status changed from new to needs_review

comment:2 Changed 9 years ago by leif

  • Milestone changed from sage-4.7.2 to sage-duplicate/invalid/wontfix
  • Status changed from needs_review to needs_info

This patch is already merged into Sage 4.7.2.alpha3 (otherwise I'd put it on another ticket):

changeset:   1682:7a9d0b9a03cc
user:        J. H. Palmieri <palmieri@math.washington.edu>
date:        Mon Aug 15 15:28:23 2011 -0700
summary:     Trac #9739: #9739: fix problems with doctesting non-library python files.

changeset:   1681:75a227112788
user:        Mitesh Patel <qed777@gmail.com>, J. H. Palmieri <palmieri@math.washington.edu>
date:        Fri Sep 03 23:44:52 2010 -0700
summary:     Trac #9739: #9739: Run doctests with unique temporary files, other small improvements

changeset:   1680:af4613048d64
user:        Leif Leonhardy <not.really@online.de>
date:        Wed Sep 28 19:13:01 2011 +0000
summary:     Added tag 4.7.2.alpha2 for changeset 8f3aaa1ebb9e

So please don't mess up #9739 which was closed, with all merged patches in the description.

comment:3 Changed 9 years ago by leif

I just wonder why the date in the log is that wrong.

comment:4 Changed 9 years ago by jhpalmieri

  • Status changed from needs_info to needs_review

I think this ticket can be closed, so I'm going to mark it as "positive review".

comment:5 Changed 9 years ago by jhpalmieri

  • Status changed from needs_review to positive_review

comment:6 Changed 9 years ago by jhpalmieri

  • Authors John Palmieri deleted
  • Description modified (diff)

comment:7 Changed 9 years ago by leif

  • Reviewers set to John Palmieri, Leif Leonhardy

comment:8 Changed 9 years ago by jdemeyer

  • Resolution set to duplicate
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.