Opened 13 years ago

Closed 12 years ago

Last modified 6 years ago

#3677 closed defect (fixed)

[with patch, positive review] sage -tp does not take into account the current directory

Reported by: mhansen Owned by: gfurnish
Priority: minor Milestone: sage-3.2.2
Component: misc Keywords:
Cc: mhansen Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by chapoton)

At the end of testing when reporting the results, sage -t does not take into account the current directory. It produces output like this:

sage -t  devel/sage-combinat/sage/combinat/root_system/ambient_space.py
sage -t  devel/sage-combinat/sage/combinat/root_system/root_lattice_realization.py
sage -t  devel/sage-combinat/sage/combinat/root_system/root_space.py
sage -t  devel/sage-combinat/sage/combinat/root_system/root_system.py
sage -t  devel/sage-combinat/sage/combinat/root_system/weight_space.py

when it should be giving output like

sage -t  ambient_space.py
sage -t  root_lattice_realization.py
sage -t  root_space.py
sage -t  root_system.py
sage -t  weight_space.py

if I am in $SAGE_ROOT/devel/sage-combinat/sage/combinat/root_system .

Attachments (1)

trac_3677_bin.patch (1.0 KB) - added by gfurnish 12 years ago.

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by gfurnish

comment:1 Changed 12 years ago by gfurnish

  • Status changed from new to assigned
  • Summary changed from sage -t / sage -tp does not take into account the current directory to [with patch, needs review] sage -t / sage -tp does not take into account the current directory

This fixes this issue for sage -tp but not for sage -t.

comment:2 Changed 12 years ago by gfurnish

  • Cc mhansen mabshoff added

Is it possible to get this reviewed for 3.2.2?

comment:3 Changed 12 years ago by gfurnish

  • Cc mabshoff removed
  • Summary changed from [with patch, needs review] sage -t / sage -tp does not take into account the current directory to [with patch, needs review] sage -tp does not take into account the current directory

The "-t" case of this has been split to #4790

comment:4 Changed 12 years ago by mabshoff

  • Summary changed from [with patch, needs review] sage -tp does not take into account the current directory to [with patch, positive review] sage -tp does not take into account the current directory

Yep, this works Positive review.

Cheers,

Michael

comment:5 Changed 12 years ago by mabshoff

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

Merged in Sage 3.2.2.rc0

comment:6 Changed 6 years ago by chapoton

  • Description modified (diff)
  • Report Upstream set to N/A
Note: See TracTickets for help on using tickets.