Skip to content

Conversation

@ThrosturX
Copy link

Fixes issue #14 the easy way, leveraging the fact that cmd.Run() waits until the process has finished, thus closing the Stderr logger manually.

A possibly better fix would be the following, I guess it's up to the PO to decide.

writer := bufio.NewWriter(outfile)
defer writer.Flush()              
                                  
if err = cmd.Start(); err != nil {
    /* handle the error */
}                                 
go io.Copy(writer, stdoutBuff)

if err = cmd.Wait(); err != nil {
    /* handle the error */
}    

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant