Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backtrace printer prints first file all the way, not correct file #2550

Closed
p6rt opened this issue Nov 6, 2011 · 5 comments
Closed

Backtrace printer prints first file all the way, not correct file #2550

p6rt opened this issue Nov 6, 2011 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Nov 6, 2011

Migrated from rt.perl.org#103034 (status was 'resolved')

Searchable as RT103034$

@p6rt
Copy link
Author

p6rt commented Nov 6, 2011

From @masak

<masak> moritz​: I think I've found a bug in the backtrace printer​:
https://gist.github.com/1343548
<moritz> masak​: yes, I know. But I don't think the backtrace printer is to blame
<masak> oh?
<moritz> masak​: I think the wrong file is come out wrongly out of the
backtrace annotations already
<masak> moritz​: but there's a fix, right?
<moritz> the handling of file names in Backtrace.pm is so trivial that
I'd be shocked if it were wrong
<masak> ok.
<masak> moritz​: as a bot, I'm confused by the above as to what to do.
would an RT ticket be beneficial, or just noise? as a developer, I
would much prefer for it to report the correct module -- I'm fine
with the error not being in code you wrote, as long as it's fixed
eventually.
<jnthn> masakbot submit rt
* masak submits rakudobug
<masak> whoa.
<masak> creepy.
<jnthn> ...works faster than p6eval

@p6rt
Copy link
Author

p6rt commented Aug 7, 2012

From @moritz

Fixed now. Writing a proper test for that will be fun :-)

@p6rt
Copy link
Author

p6rt commented Aug 7, 2012

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Jan 12, 2013

From @moritz

Now tested in integration/error-reporting.t

@p6rt
Copy link
Author

p6rt commented Jan 12, 2013

@moritz - Status changed from 'open' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant