Skip to content

Commit

Permalink
fix: remove PTY we use virtual tty in packages
Browse files Browse the repository at this point in the history
Signed-off-by: Ales Verbic <verbotenj@blinklabs.io>
  • Loading branch information
verbotenj committed Sep 26, 2024
1 parent d69d4c8 commit f54547d
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 5 deletions.
1 change: 0 additions & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ go 1.21
require (
github.com/Masterminds/sprig/v3 v3.3.0
github.com/blinklabs-io/gouroboros v0.94.3
github.com/creack/pty v1.1.23
github.com/docker/docker v27.3.1+incompatible
github.com/docker/go-connections v0.5.0
github.com/hashicorp/go-version v1.7.0
Expand Down
2 changes: 0 additions & 2 deletions go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,6 @@ github.com/cenkalti/backoff/v4 v4.2.1/go.mod h1:Y3VNntkOUPxTVeUxJ/G5vcM//AlwfmyY
github.com/containerd/log v0.1.0 h1:TCJt7ioM2cr/tfR8GPbGf9/VRAX8D2B4PjzCpfX540I=
github.com/containerd/log v0.1.0/go.mod h1:VRRf09a7mHDIRezVKTRCrOq78v577GXq3bSa3EhrzVo=
github.com/cpuguy83/go-md2man/v2 v2.0.4/go.mod h1:tgQtvFlXSQOSOSIRvRPT7W67SCa46tRHOmNcaadrF8o=
github.com/creack/pty v1.1.23 h1:4M6+isWdcStXEf15G/RbrMPOQj1dZ7HPZCGwE4kOeP0=
github.com/creack/pty v1.1.23/go.mod h1:08sCNb52WyoAwi2QDyzUCTgcvVFhUzewun7wtTfvcwE=
github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc h1:U9qPSI2PIWSS1VwoXQT9A3Wy9MM3WgvqSxFWenqJduM=
github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
Expand Down
3 changes: 1 addition & 2 deletions pkgmgr/package.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ import (
"regexp"
"strings"

"github.com/creack/pty"
"github.com/hashicorp/go-version"
"gopkg.in/yaml.v3"
)
Expand Down Expand Up @@ -648,7 +647,7 @@ func (p Package) runHookScript(cfg Config, hookScript string) error {
cmd.Stdout = os.Stdout
cmd.Stderr = os.Stderr
// We won't be reading or writing, so throw away the PTY file
_, err = pty.Start(cmd)
err = cmd.Start()
if err != nil {
return fmt.Errorf("failed to run hook script: %s", err)
}
Expand Down

0 comments on commit f54547d

Please sign in to comment.