Browse Source

cleanup

pull/169/head
aler9 6 years ago
parent
commit
4f7daacdda
  1. 18
      internal/externalcmd/cmd.go
  2. 2
      internal/externalcmd/cmd_unix.go
  3. 2
      internal/externalcmd/cmd_win.go
  4. 4
      internal/path/path.go

18
internal/externalcmd/externalcmd.go → internal/externalcmd/cmd.go

@ -8,14 +8,14 @@ const ( @@ -8,14 +8,14 @@ const (
retryPause = 5 * time.Second
)
// Environment is a ExternalCmd environment.
// Environment is a Cmd environment.
type Environment struct {
Path string
Port string
}
// ExternalCmd is an external command.
type ExternalCmd struct {
// Cmd is an external command.
type Cmd struct {
cmdstr string
restart bool
env Environment
@ -27,9 +27,9 @@ type ExternalCmd struct { @@ -27,9 +27,9 @@ type ExternalCmd struct {
done chan struct{}
}
// New allocates an ExternalCmd.
func New(cmdstr string, restart bool, env Environment) *ExternalCmd {
e := &ExternalCmd{
// New allocates an Cmd.
func New(cmdstr string, restart bool, env Environment) *Cmd {
e := &Cmd{
cmdstr: cmdstr,
restart: restart,
env: env,
@ -41,13 +41,13 @@ func New(cmdstr string, restart bool, env Environment) *ExternalCmd { @@ -41,13 +41,13 @@ func New(cmdstr string, restart bool, env Environment) *ExternalCmd {
return e
}
// Close closes an ExternalCmd.
func (e *ExternalCmd) Close() {
// Close closes an Cmd.
func (e *Cmd) Close() {
close(e.terminate)
<-e.done
}
func (e *ExternalCmd) run() {
func (e *Cmd) run() {
defer close(e.done)
for {

2
internal/externalcmd/externalcmd_unix.go → internal/externalcmd/cmd_unix.go

@ -7,7 +7,7 @@ import ( @@ -7,7 +7,7 @@ import (
"os/exec"
)
func (e *ExternalCmd) runInner() bool {
func (e *Cmd) runInner() bool {
cmd := exec.Command("/bin/sh", "-c", "exec "+e.cmdstr)
cmd.Env = append(os.Environ(),

2
internal/externalcmd/externalcmd_win.go → internal/externalcmd/cmd_win.go

@ -10,7 +10,7 @@ import ( @@ -10,7 +10,7 @@ import (
"github.com/kballard/go-shellquote"
)
func (e *ExternalCmd) runInner() bool {
func (e *Cmd) runInner() bool {
// on Windows the shell is not used and command is started directly
// variables are replaced manually in order to guarantee compatibility
// with Linux commands

4
internal/path/path.go

@ -155,7 +155,7 @@ type Path struct { @@ -155,7 +155,7 @@ type Path struct {
sourceTrackCount int
sourceSdp []byte
readers *readersMap
onDemandCmd *externalcmd.ExternalCmd
onDemandCmd *externalcmd.Cmd
describeTimer *time.Timer
sourceCloseTimer *time.Timer
sourceCloseTimerStarted bool
@ -244,7 +244,7 @@ func (pa *Path) run() { @@ -244,7 +244,7 @@ func (pa *Path) run() {
pa.startExternalSource()
}
var onInitCmd *externalcmd.ExternalCmd
var onInitCmd *externalcmd.Cmd
if pa.conf.RunOnInit != "" {
pa.Log("on init command started")
onInitCmd = externalcmd.New(pa.conf.RunOnInit, pa.conf.RunOnInitRestart, externalcmd.Environment{

Loading…
Cancel
Save