[Patches] [ python-Patches-1571379 ] make trace.py --ignore-dir work

SourceForge.net noreply at sourceforge.net
Sun Feb 11 19:38:43 CET 2007


Patches item #1571379, was opened at 2006-10-05 08:10
Message generated for change (Comment added) made by montanaro
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1571379&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Library (Lib)
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Clinton Roy (clintonroy)
Assigned to: Skip Montanaro (montanaro)
Summary: make trace.py --ignore-dir work

Initial Comment:
The co_filename attribute is only ever the relative
filename, where as the __file__ value of the frame
globals is absolute, which lets the ignore() stuff do
its thing.

----------------------------------------------------------------------

>Comment By: Skip Montanaro (montanaro)
Date: 2007-02-11 12:38

Message:
Logged In: YES 
user_id=44345
Originator: NO

Checked in on SVN head as rev 53735.  Checked in on 2.4 branch as rev
53736.  Will also backport to 2.5 once I figure out why svn thinks I have a
conflict in Misc/NEWS.


----------------------------------------------------------------------

Comment By: Clinton Roy (clintonroy)
Date: 2006-10-08 07:02

Message:
Logged In: YES 
user_id=31446

Hi Skip,

This patch only addresses --ignore-dir working, I didn't
initally look at the other uses.

Now that I have, they're all passed to linecache, which 
itself looks in $prefix for a matching filename, so it only
needs the relative filename to be passed.

My patch specifically just targets the filename param passed
to ignore.names() so that it can actually work.

cheers,

----------------------------------------------------------------------

Comment By: Skip Montanaro (montanaro)
Date: 2006-10-07 05:25

Message:
Logged In: YES 
user_id=44345

Without digging too deeply, can you explain why only this one instance of
using 
code.co_filename needs to be changed?


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1571379&group_id=5470


More information about the Patches mailing list