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.

Sign up for GitHub

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

Jump to bottom

Parallel combinators test fails for features/plenary #301

Closed
Bearmarshal opened this issue Dec 17, 2015 · 5 comments
Closed

Parallel combinators test fails for features/plenary #301

Bearmarshal opened this issue Dec 17, 2015 · 5 comments
Assignees
@kikofernandez

Comments

@Bearmarshal
Copy link

Output from the terminal when running make test:

Parallel Combinators:
cmp: EOF i extract.out
     Tests passed: 4/5
     Tests failed: 1/5
         extract

For the record, I'm running Linux Ubuntu 14.04
@kikofernandez
Copy link
Contributor

Hi, this error happens sporadically and is related to #274. It's not really in the parallel combinators but a bug in Encore runtime. Thanks in any case. I'll keep it open for now but it's dependent on a fix to #274

@albertnetymk
Copy link
Contributor

I doubt that it's related to #274. I could reproduce it with --ponythreads 1.

Assertion failed: ERROR in 'test_from_string_to_int'

@kikofernandez kikofernandez mentioned this issue Dec 18, 2015
Merged
@kikofernandez kikofernandez self-assigned this Dec 18, 2015
@kikofernandez
Copy link
Contributor

you were all right! it was caused by not using the equals method on strings... probably forgot to update the file. PR ready to fix it! Thanks

EliasC added a commit that referenced this issue Dec 18, 2015
@EliasC
Merge pull request #307 from kikofernandez/fix/extract
55a50f1 
fix #301: extract.enc test not using String
@albertnetymk
Copy link
Contributor

@Bearmarshal I think this is fixed by now, if you can confirm.

@albertnetymk
Copy link
Contributor

Can't reproduce it on development, closing now. Reopen, if not so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kikofernandez kikofernandez

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@kikofernandez @TobiasWrigstad @albertnetymk @Bearmarshal