Skip to content
This repository has been archived by the owner on Mar 2, 2023. It is now read-only.

All tests fail #2

Closed
benlaurie opened this issue Aug 20, 2018 · 7 comments
Closed

All tests fail #2

benlaurie opened this issue Aug 20, 2018 · 7 comments
Labels
duplicate This issue or pull request already exists

Comments

@benlaurie
Copy link
Contributor

If I run the tests, there are 249 tests and 249 failures. For example:

target = '/usr/local/google/home/benl/git-work/qsym/tests/syscall/mremap'

    def test_syscall(target):
>       assert run_single_test(target)
E       AssertionError: assert False
E        +  where False = run_single_test('/usr/local/google/home/benl/git-work/qsym/tests/syscall/mremap')

test_syscall.py:19: AssertionError
@mxmssh
Copy link

mxmssh commented Aug 20, 2018

There is file called pin.log in the test folder, could you please print it out here. I had the same issue.

@insuyun
Copy link
Contributor

insuyun commented Aug 20, 2018

I think this would be the problem of ptrace_scope. Please check new README or pin.log that @mxmssh mentioned.

@mxmssh
Copy link

mxmssh commented Aug 20, 2018

In my case it was simple, I just forgot to run python setup.py build before python setup.py install
It would be great to add this step in the manual, people might get confused :)

@insuyun
Copy link
Contributor

insuyun commented Aug 20, 2018

Thanks.
I just pushed c538ebd that fixes documentation to use pip instead of python setup.py install.
This will build libqsym.so.

@BinLeeBit
Copy link

When I run tests, there are "E: Unable to load /../qsym/qsym/pintool/obj-intel64/libqsym.so" in pin.log. I just use python setup.py build instead of python setup.py install in README.md. It can run, but slowly.

@insuyun
Copy link
Contributor

insuyun commented Aug 22, 2018

@BinLeeBit
I think you are looking at old README.md.
I changed it to use pip install . instead of python setup.py install.
Then it will work. (I hope)
You can use python setup.py build and python setup.py install to install this project, too.
Sorry for confusion.

@insuyun insuyun added the duplicate This issue or pull request already exists label Sep 13, 2018
@insuyun
Copy link
Contributor

insuyun commented Sep 13, 2018

This issue is also same with #4.

@insuyun insuyun closed this as completed Sep 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

4 participants