this fixes the pq registered twice issue (#1033)

generators/newapp/new.go

@@ -377,10 +377,8 @@ const GopkgTomlTmpl = `
     name = "github.com/gobuffalo/buffalo"
     unused-packages = false
 
-{{ if .opts.WithPop }}
   # DO NOT DELETE
   [[prune.project]] # pop
     name = "github.com/gobuffalo/pop"
     unused-packages = false
-{{ end }}
 `

internal/vbuffalo/dep.go

@@ -4,7 +4,6 @@ import (
 	"bytes"
 	"io/ioutil"
 	"os"
-	"os/exec"
 	"path/filepath"
 
 	"html/template"
@@ -43,7 +42,7 @@ func depEnsure() error {
 			return errors.WithStack(err)
 		}
 
-		return run(exec.Command("dep", "ensure", "-v"))
+		return run("dep", []string{"ensure", "-v"})
 	}
 
 	if addPrune {

internal/vbuffalo/vbuffalo.go

@@ -1,6 +1,7 @@
 package vbuffalo
 
 import (
+	"fmt"
 	"os"
 	"os/exec"
 	"path/filepath"
@@ -18,6 +19,7 @@ var binPath = filepath.Join(pwd, "bin", "vbuffalo")
 var app meta.App
 
 func init() {
+	fmt.Println("vbuffalo")
 	if runtime.GOOS == "windows" {
 		binPath += ".exe"
 	}
@@ -59,15 +61,13 @@ func execute() error {
 			args = append(args, "--tags", "sqlite")
 		}
 		args = append(args, "-o", binPath)
-		cmd := exec.Command("go", args...)
-		return run(cmd)
+		return run("go", args)
 	})
 	if err != nil {
 		return errors.WithStack(err)
 	}
 
-	cmd := exec.Command(binPath, os.Args[1:]...)
-	return run(cmd)
+	return run(binPath, os.Args[1:])
 }
 
 func cd(dir string, fn func() error) error {
@@ -76,12 +76,16 @@ func cd(dir string, fn func() error) error {
 	return fn()
 }
 
-func run(cmd *exec.Cmd) error {
-	// fmt.Println(strings.Join(cmd.Args, " "))
+func run(name string, args []string) error {
+	// fmt.Println("vbuffalo", name, strings.Join(args, " "))
+	cmd := exec.Command(name, args...)
 	cmd.Stdin = os.Stdin
 	cmd.Stdout = os.Stdout
 	cmd.Stderr = os.Stderr
-	return cmd.Run()
+	if err := cmd.Start(); err != nil {
+		return errors.WithStack(err)
+	}
+	return cmd.Wait()
 }
 
 func exists(f string) bool {