[app][git] run real command for clone

app/git/clone.go

@@ -1,5 +1,9 @@
 package git
 
+import (
+	"github.com/dyweb/Ayi/util"
+)
+
 // CloneFromURL will clone a repository based on short url or normal git clone url
 func CloneFromURL(repoURL string) error {
 	// get the remote from url
@@ -14,7 +18,8 @@ func CloneFromURL(repoURL string) error {
 
 // Clone clones a remote git repo
 func Clone(r Remote) error {
-	log.Info(r.GetSSH())
-	log.Info("git clone " + r.GetSSH() + " " + GetRepoBasePath() + "/" + r.Repo)
-	return nil
+	// log.Info("git clone " + r.GetSSH() + " " + GetCloneDirectory(r))
+	cmdStr := "git clone " + r.GetSSH() + " " + GetCloneDirectory(r)
+	log.Info(cmdStr)
+	return util.RunCommand(cmdStr)
 }

app/git/workspace.go

@@ -3,6 +3,7 @@ package git
 import (
 	"os"
 	"path/filepath"
+	"strings"
 
 	"github.com/spf13/viper"
 )
@@ -21,3 +22,13 @@ func GetRepoBasePath() string {
 	}
 	return ""
 }
+
+// GetCloneDirectory return path for clone destination
+func GetCloneDirectory(r Remote) string {
+	base := GetRepoBasePath()
+	if base == "" {
+		return ""
+	}
+	// git clone does not accept windows format path, so all /
+	return strings.Replace(base+"/"+r.Host+"/"+r.Owner+"/"+r.Repo, "\\", "/", -1)
+}

cmd/git.go

@@ -34,7 +34,7 @@ var gitCloneCmd = &cobra.Command{
 		repoURL := args[0]
 		err := git.CloneFromURL(repoURL)
 		if err != nil {
-			log.Warn(err.Error())
+			log.Error(err.Error())
 			return
 		}
 		// TODO: clone may need to return more information