search

cyoung / stratux

Aviation weather and traffic receiver based on RTL-SDR.
BSD 3-Clause "New" or "Revised" License
1.06k stars 363 forks source link

dump1090 errors not logged #102

Closed bradanlane closed 8 years ago

bradanlane commented 8 years ago

If dump1090 generates any error messages that do not abort the process, they are lost.

This is not a problem under normal operations but users who may experience power related oddities (and we have seen them) will not get any evidence in the log files.

The only way to detect the errors are by killing off the stratux process and then manually starting dump1090 manually. This is difficult now that dump1090 is started inside of the main gen_gdl90 code.

jpoirier commented 8 years ago

We can grab stderr (and stdio ?) from the running process and pass it to the logger.

bradanlane commented 8 years ago

I attempted this but not being completely fluent with golang, I wasn't able to get the proper results.

I'll leave this open in the hopes someone can make the necessary changes.

jpoirier commented 8 years ago

Thanks for giving it a shot. I'll try to add this within the next few days.

On Thu, Nov 12, 2015 at 7:08 AM, bradanlane notifications@github.com wrote:

I attempted this but not being completely fluent with golang, I wasn't able to get the proper results.

I'll leave this open in the hopes someone can make the necessary changes.

— Reply to this email directly or view it on GitHub https://github.com/cyoung/stratux/issues/102#issuecomment-156098657.

jpoirier commented 8 years ago

It's simpler (five or six lines of code) to just pipe the dump1090 stdio and stderr to the current stratux log. My last commit is included in the pending push.