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

commit 24f0b3885aee8f3aaf2b7819bc258446f08b4d8e broke a few non-regression tests related to vec type #97

Open
LRGH opened this issue Mar 8, 2024 · 2 comments

Comments

@LRGH
Copy link
Contributor

LRGH commented Mar 8, 2024

python -m pytest generates three errors in test_cas_exp and two errors in test_cas_mapper.

A short example where the patch broke something is python -c 'from amoco.cas.expressions import *;print(vec([lab("eax",size=32),cst(0x10,32)])+3)' which outputted [(@eax+0x3),0x13]but now outputs T32.

@bdcht
Copy link
Owner

bdcht commented Mar 11, 2024

Fixed by fc91e1d. Would still need to make sure that all tests are independent of a local config now that I've removed travis-ci builds.

@LRGH
Copy link
Contributor Author

LRGH commented Mar 11, 2024

GitHub actions can quite nicely replace travis-ci.

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