-
Notifications
You must be signed in to change notification settings - Fork 87
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
Make debugging easier by printing relevant errors #35
base: master
Are you sure you want to change the base?
Conversation
Hey @ShellCode33 there is issue with ur PR (Conflicts) |
It was indeed in conflict, I included a commit in the PR which shouldn't have been there. Should be ok now. |
This is not related to this PR but more to this one #36, you are probably using a pretty old version of qemu... Can't you upgrade it ? Otherwise the thing to do is to check qemu's version beforehand and use the appropriate parameter, which is kind of ugly :/ |
Sorry @ShellCode33 I don't understand, what is the new syntax ? |
Here is my version of qemu-system-arm:
|
Oh ok I see ! You are still using the old qemu's Just to be clear :
|
Hi ! Thanks for this amazing tool ! I came across an error and to be honest, debugging arm_now was an absolute nightmare... I spent almost 2 hours digging into the code just to realize the FS wasn't big enough...
This PR improves errors printing and should help future users to understand what's wrong with their setup