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

Test failure on i686 #8

Open
adigitoleo opened this issue Mar 18, 2024 · 2 comments
Open

Test failure on i686 #8

adigitoleo opened this issue Mar 18, 2024 · 2 comments

Comments

@adigitoleo
Copy link

I get a test failure when building v9.2 on i686, with the traceback of:

Traceback (most recent call last):
  File "/builddir/tab-9.2/test/go.py", line 68, in <module>
    go(sort=("-s" in sys.argv), tests=sys.argv[1:])
  File "/builddir/tab-9.2/test/go.py", line 53, in go
    run(i, txt[1], txt[2], log, infile=txt[0].replace('\n',''), sort=sort)
  File "/builddir/tab-9.2/test/go.py", line 37, in run
    raise Exception("Test failed for: %s, '%s' -- output is '%s'" % (filename, arg, out))
Exception: Test failed for: uniques.test.in, 'z = { x=cut(@,"\t"), uint(x~0)/50 -> uniques.x~3, uniques_estimate.x~3 }, lines(sort.z, merge.first.second.z)' -- output is '38	221	330
39	589	834
40	528	783
608
'

Is 32-bit arch supported?

@ivan-tkatchev
Copy link
Owner

ivan-tkatchev commented Mar 18, 2024

Yes, 32-bit architecture is supported. This is a test problem - uniques_estimate uses a hash under the hood that differs for 32 and 64 bit architectures.

I just pushed a test fix to master, please confirm that tests pass.

@adigitoleo
Copy link
Author

adigitoleo commented Mar 18, 2024

Yes, it seems like that fixed it. Tests pass on the 32-bit runners now: https://github.com/void-linux/void-packages/actions/runs/8333452792/job/22804931888?pr=49358

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

2 participants