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

FAIL: base-test on riscv64 #57

Open
ncopa opened this issue Apr 24, 2024 · 1 comment · May be fixed by #58
Open

FAIL: base-test on riscv64 #57

ncopa opened this issue Apr 24, 2024 · 1 comment · May be fixed by #58

Comments

@ncopa
Copy link

ncopa commented Apr 24, 2024

base-test fails on alpine linux edge riscv64:

========================================
   marisa 0.2.6: tests/test-suite.log
========================================

# TOTAL: 5
# PASS:  4
# SKIP:  0
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0

.. contents:: :depth: 2

FAIL: base-test
===============

base-test.cc:14: TestTypes(): 21: Assertion `MARISA_WORD_SIZE == (sizeof(std::size_t) * 8)' failed.
FAIL base-test (exit status: 255)
@ncopa
Copy link
Author

ncopa commented Apr 24, 2024

Seems like there are a number of similar reports with non-optimal suggestions for a solution:

Apparently this is a difficult problem to solve :)

ncopa added a commit to ncopa/marisa-trie that referenced this issue Apr 24, 2024
@ncopa
Fix detection of MARISA_WORD_SIZE
1e16775 
Detect the MARISA_WORD_SIZE independent of architecture.

Fixes: s-yata#40
Fixes: s-yata#57
Fixes: s-yata#44
Fixes: s-yata#46
Fixes: s-yata#56
@ncopa ncopa linked a pull request Apr 24, 2024 that will close this issue
Open
algitbot pushed a commit to alpinelinux/aports that referenced this issue Apr 24, 2024
@ncopa
community/libmarisa: fix wordsize detection
cdfcf37 
Fixes detection of word size on riscv64 and loongarch64.

ref: s-yata/marisa-trie#57
ncopa added a commit to ncopa/marisa-trie that referenced this issue Apr 30, 2024
@ncopa
Fix detection of MARISA_WORD_SIZE
6925413 
Detect the MARISA_WORD_SIZE independent of architecture.

Fixes: s-yata#40
Fixes: s-yata#57
Fixes: s-yata#44
Fixes: s-yata#46
Fixes: s-yata#56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix detection of MARISA_WORD_SIZE ncopa/marisa-trie
1 participant
@ncopa