Merge pull request #4 from riltech/fix/process_logging

Roverr · web-flow · commit 8c5f01fd4156 · 2020-04-26T11:10:03.000+02:00
diff --git a/main.go b/main.go
@@ -59,7 +59,7 @@ func NewStream(
 		logrus.Error(err)
 		return nil, ""
 	}
-	process := NewProcess(keepFiles, audio, loggingOpts)
+	process := NewProcess(keepFiles, audio)
 	cmd := process.Spawn(path, URI)
 
 	// Create nil pointer in case logging is not enabled
diff --git a/process.go b/process.go
@@ -23,9 +23,8 @@ type ProcessLoggingOpts struct {
 
 // Process is the main type for creating new processes
 type Process struct {
-	keepFiles   bool
-	audio       bool
-	loggingOpts ProcessLoggingOpts
+	keepFiles bool
+	audio     bool
 }
 
 // Type check
@@ -35,9 +34,8 @@ var _ IProcess = (*Process)(nil)
 func NewProcess(
 	keepFiles bool,
 	audio bool,
-	loggingOpts ProcessLoggingOpts,
 ) *Process {
-	return &Process{keepFiles, audio, loggingOpts}
+	return &Process{keepFiles, audio}
 }
 
 // getHLSFlags are for getting the flags based on the config context

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ func NewStream(`
`59`	`59`	`logrus.Error(err)`
`60`	`60`	`return nil, ""`
`61`	`61`	`}`
`62`		`- process := NewProcess(keepFiles, audio, loggingOpts)`
	`62`	`+ process := NewProcess(keepFiles, audio)`
`63`	`63`	`cmd := process.Spawn(path, URI)`
`64`	`64`
`65`	`65`	`// Create nil pointer in case logging is not enabled`