Closed wkozaczuk closed 6 years ago
Regarding the problem with QEMU that led you to implement this patch @wkozaczuk , could you please open an issue about it, providing some more information? I wasn't aware that Capstan has QEMU problems on Mac.
@wkozaczuk thanks for this contribution!
Can you please update the commit message to comply with the proposal of @miha-plesko? Simple Redirect QEMU standard error to Capstan
title will do. You may also choose to add the text from PR into the commit, but that is not that important.
I hope we are not asking too much, but we'd like to ensure that commits are traceable.
I will be sending next patch to fix the QEMU/OSx related issue that show up in standard error
@wkozaczuk Sorry for being stubborn with this commit: thank you for changing the PR text, however what I was referring to was the actual commit title of this commit.
May I ask you to edit it (the same title as current PR title)?
@gberginc Done.
@wkozaczuk I think the commit message body is still missing:
Should be something like:
I am sorry for my total misunderstanding of you originally wanted. I amended the commit - I hope it will work like a charm this time.
Thanks for the contribution @wkozaczuk , it's much better now! 😄
Currently in case of QEMU error (which happens when composing an app on Mac OSX) capstan does not output any information and eventually quits with misleading error message:
In reality QEMU does not start due to some deprecated or incorrect parameters. In order for user to see the errors output by QEMU to standard error capstan needs to redirect it to its standard error.
After the changes user can see exact error which is much better:
Please note that the underlying problem is fixed in another followup pull request that I am ready to submit once this one is merged.
Signed-off-by: Waldemar Kozaczuk jwkozaczuk@gmail.com