Ticket #3677 (closed defect: fixed)

Opened 20 months ago

Last modified 15 months ago

[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 Author(s):
Report Upstream: Reviewer(s):
Merged in: Work issues:

Description

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

trac_3677_bin.patch Download (1.0 KB) - added by gfurnish 15 months ago.

Change History

Changed 15 months ago by gfurnish

Changed 15 months 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.

Changed 15 months ago by gfurnish

  • cc mhansen, mabshoff added

Is it possible to get this reviewed for 3.2.2?

Changed 15 months 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

Changed 15 months 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

Changed 15 months ago by mabshoff

  • status changed from assigned to closed
  • resolution set to fixed

Merged in Sage 3.2.2.rc0

Note: See TracTickets for help on using tickets.