-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Output exit code in FAILED message #1805
base: master
Are you sure you want to change the base?
Conversation
I'm not complete happy with the interface of this design (putting the exit code in parens). I'd love to hear other ideas if people have them. |
Outputs the exit code after "FAILED" for any process that did not complete successfully. Output looks like "FAILED (<exitcode>): <outputs>"
Any idea when or if this will be merged? It would help a lot when using ninja on CI to have proper sucess/errors. Right now I depend on checking the output to see if the build is there to determine if a ninja subprocess failed or not. |
I was just needing this to debug a failing command that didn't have any output and used this rebased on top of the last release (some code has moved, but the basics still apply). Would indeed be nice to have this by default. |
Adding a vote to merge this PR, or another one that addresses the problem. It would have been very useful to us for debugging random command failures. |
Just used a custom built ninja with this patch to debug a gnarly compiler crash. Adding my vote to have this feature merged! |
it's a great idea, but I don't think this is the best way to write it |
Maybe the approach from #2540 is more straight forward? |
This fixes #1507
Outputs the exit code after "FAILED" for any process that did not
complete successfully.
Output looks like "FAILED (<exitcode>): <outputs>"