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

rakudo failing reverse.t and abs.t #436

Closed
p6rt opened this issue Nov 29, 2008 · 5 comments
Closed

rakudo failing reverse.t and abs.t #436

p6rt opened this issue Nov 29, 2008 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Nov 29, 2008

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

Searchable as RT60910$

@p6rt
Copy link
Author

p6rt commented Nov 29, 2008

From @pmichaud

Something changed between r33248 and 33263 that causes
Rakudo to fail the S29-list/reverse.t and S29-num/abs.t
spectests. The revisions in question had to do with
changing namespaces and multisub PMCs, but I don't know
enough of the details to fix or troubleshoot them.

Pm

@p6rt
Copy link
Author

p6rt commented Nov 30, 2008

From @masak

<masak> pmichaud, chromatic​: commenting out line 265 (added on r33263,
then as line 260) of src/pmc/namespace.pmc makes reverse.t and abc.t
pass again. I don't know enough to fix it, but that line's the cause.
<pmichaud> the && !old line?
<masak> aye
<masak> that was one of the two changes in r33263.
<pmichaud> well, I don't understand enough about namespace/multisub guts
to know why that's there... so I'll leave it for chromatic++ or
jonathan++ to work out.

@p6rt
Copy link
Author

p6rt commented Nov 30, 2008

@masak - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Nov 30, 2008

From @chromatic

On Friday 28 November 2008 16​:10​:14 Patrick R. Michaud (via RT) wrote​:

Something changed between r33248 and 33263 that causes
Rakudo to fail the S29-list/reverse.t and S29-num/abs.t
spectests. The revisions in question had to do with
changing namespaces and multisub PMCs, but I don't know
enough of the details to fix or troubleshoot them.

Fixed in r33386.

-- c

@p6rt
Copy link
Author

p6rt commented Dec 7, 2008

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

@p6rt p6rt closed this as completed Dec 7, 2008
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant