Docker/documize
Docker/documize
1
0
Fork 0
mirror of https://github.com/documize/community.git synced 2025-07-19 21:29:42 +02:00
Code Releases Activity

Remove redundant logging dependency

Browse source
This commit is contained in:
Harvey Kandola 2019-01-09 11:04:23 +00:00
parent bc28b36627
commit 659eb99c0f
1 changed files with 2 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

5
core/osutil/command.go
View file

@ -14,10 +14,9 @@ package osutil
import ( import (
"bytes" "bytes"
"errors" "errors"
"fmt"
"os/exec" "os/exec"
"time" "time"
"github.com/documize/community/core/log"
) )
var errTimeout = errors.New("conversion timelimit exceeded") var errTimeout = errors.New("conversion timelimit exceeded")
@ -39,7 +38,7 @@ func CommandWithTimeout(command *exec.Cmd, timeout time.Duration) ([]byte, error
select { select {
case <-time.After(timeout): case <-time.After(timeout):
if err := command.Process.Kill(); err != nil { if err := command.Process.Kill(); err != nil {
log.Error("failed to kill: ", err) fmt.Errorf("failed to kill: ", err)
} }
<-done // prevent memory leak <-done // prevent memory leak
//fmt.Println("DEBUG timeout") //fmt.Println("DEBUG timeout")