Bisa / factorio-init

Factorio init script
MIT License
415 stars 82 forks source link

"awk" invocation under "send_cmd)" hangs due to "-F" flag on "tail" #132

Closed NopeDK closed 5 years ago

NopeDK commented 6 years ago

As per title, https://github.com/Bisa/factorio-init/blob/20e60dd9b2c1bc15987cf067c73ba16b590195a2/factorio#L273

Solution: remove the "-F" flag.

New problem: If the output of the command is not available at "tail" time, mystical things happen

jadoc commented 6 years ago

Pull request #134 was meant to fix an issue related to waiting for output when the server is paused, but it also introduces a sleep when output is expected. It's a hack, but may help with this issue. Though, the entire output extraction logic is already a hack, so maybe this is the best that can be done.

Bisa commented 5 years ago

it's a hack indeed, if I come around to looking at this I hope the third invocation of this script will be better ;)