Skip to content

Commit

Permalink
Fix logging
Browse files Browse the repository at this point in the history
  • Loading branch information
fabpot committed Jan 23, 2022
1 parent 6e7b0a8 commit a2f0d18
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion commands/local_new.go
Original file line number Diff line number Diff line change
Expand Up @@ -327,7 +327,7 @@ func runComposer(c *console.Context, dir string, args []string, debug bool) erro
env = append(env, "SYMFONY_DOCKER=1")
}

if err := php.Composer(dir, args, env, out, err, os.Stderr); err.ExitCode() != 0 {
if err := php.Composer(dir, args, env, out, err, os.Stderr, terminal.Logger); err.ExitCode() != 0 {
terminal.Println(buf.String())
terminal.Logger.Debug().Msg(buf.String())
return err
Expand Down
9 changes: 4 additions & 5 deletions local/php/composer.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ import (
"strings"

"github.com/pkg/errors"
"github.com/rs/zerolog"
"github.com/symfony-cli/symfony-cli/util"
"github.com/symfony-cli/terminal"
)

const DefaultComposerVersion = 2
Expand All @@ -56,24 +56,23 @@ func (c ComposerResult) ExitCode() int {
return c.code
}

func Composer(dir string, args, env []string, stdout, stderr, logger io.Writer) ComposerResult {
func Composer(dir string, args, env []string, stdout, stderr, logger io.Writer, debugLogger zerolog.Logger) ComposerResult {
e := &Executor{
Dir: dir,
BinName: "php",
Stdout: stdout,
Stderr: stderr,
SkipNbArgs: -1,
ExtraEnv: env,
Logger: terminal.Logger,
Logger: debugLogger,
}

composerBin := "composer1"
if composerVersion() == 2 {
composerBin = "composer2"
}
path, err := e.findComposer(composerBin)
if err != nil || !isComposerPHPScript(path) {
fmt.Println(" WARNING: Unable to find Composer, downloading one. It is recommended to install Composer yourself at https://getcomposer.org/download/")
fmt.Fprintln(logger, " WARNING: Unable to find Composer, downloading one. It is recommended to install Composer yourself at https://getcomposer.org/download/")
// we don't store it under bin/ to avoid it being found by findComposer as we want to only use it as a fallback
binDir := filepath.Join(util.GetHomeDir(), "composer")
if path, err = downloadComposer(binDir); err != nil {
Expand Down
3 changes: 2 additions & 1 deletion main.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ import (
"os"
"time"

"github.com/rs/zerolog"
"github.com/symfony-cli/console"
"github.com/symfony-cli/symfony-cli/commands"
"github.com/symfony-cli/symfony-cli/local/php"
Expand Down Expand Up @@ -73,7 +74,7 @@ func main() {
}
// called via "symfony composer"?
if len(args) >= 2 && args[1] == "composer" {
res := php.Composer("", args[2:], []string{}, os.Stdout, os.Stderr, ioutil.Discard)
res := php.Composer("", args[2:], []string{}, os.Stdout, os.Stderr, ioutil.Discard, zerolog.Nop())
terminal.Eprintln(res.Error())
os.Exit(res.ExitCode())
}
Expand Down

0 comments on commit a2f0d18

Please sign in to comment.