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

which python version should we use for best out come? #1

Closed
zerishpho opened this issue Apr 7, 2021 · 3 comments
Closed

which python version should we use for best out come? #1

zerishpho opened this issue Apr 7, 2021 · 3 comments

Comments

@zerishpho
Copy link

prime@Primes-MacBook-Pro harlogger-master % ./remote_har_listener.py -o output.har
Traceback (most recent call last):
File "/Users/prime/Downloads/harlogger-master/harlogger-master/./remote_har_listener.py", line 151, in
main()
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/click/core.py", line 829, in call
return self.main(*args, **kwargs)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/click/core.py", line 782, in main
rv = self.invoke(ctx)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/click/core.py", line 1066, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/click/core.py", line 610, in invoke
return callback(*args, **kwargs)
File "/Users/prime/Downloads/harlogger-master/harlogger-master/./remote_har_listener.py", line 109, in main
assert line == b'[connected]'
AssertionError

@doronz88
Copy link
Owner

doronz88 commented Apr 7, 2021

Should work with every python3.x.
The script executes idevicesyslog to parse the log lines from the device.
This assert line doesn't really matter, but is there just for sanity check since I expect the first line from it to be "[connected]".
Can you try and print the line you're getting instead? Or just comment out and verify?

On newer releases I'll replace this dependency with: https://github.com/doronz88/pymobiledevice3 which I'm currently working on to reduce dependency on external executables.

Please update what solution fixes this.

@doronz88
Copy link
Owner

It's now using pymobiledevice3 which is a much better implementation and should fix all enviornment related bugs. I'm closing this issue. If the problem persist, please open another.

@zerishpho
Copy link
Author

zerishpho commented Apr 20, 2021 via email

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