Merge lp:~sergiusens/snappy/reviewNOT into lp:~snappy-dev/snappy/snappy-moved-to-github

[Sergio Schvezov]

Disabling review tools

[Alexander Sack (asac)]

i am happy with the content of this change!

[John Lenton (chipaca)]

review: Approve

[lp:~sergiusens/snappy/reviewNOT]

448. By Sergio Schvezov on 2015-05-07

Disabling review tools by sergiusens approved by chipaca

=== modified file 'cmd/snappy/cmd_build.go'
--- cmd/snappy/cmd_build.go	2015-04-20 10:40:12 +0000
+++ cmd/snappy/cmd_build.go	2015-05-07 17:10:52 +0000
@@ -19,8 +19,6 @@
 
 import (
 	"fmt"
-	"os"
-	"os/exec"
 
 	"launchpad.net/snappy/snappy"
 )
@@ -53,16 +51,20 @@
 		return err
 	}
 
-	_, err = exec.LookPath(clickReview)
-	if err != nil {
-		fmt.Fprintf(os.Stderr, "Warning: could not review package (%s not available)\n", clickReview)
-	}
-
-	cmd := exec.Command(clickReview, snapPackage)
-	cmd.Stdout = os.Stdout
-	cmd.Stderr = os.Stderr
-	// we ignore the error for now
-	_ = cmd.Run()
+	/*
+		Disabling review tools run until the output reflects reality more closely
+
+		_, err = exec.LookPath(clickReview)
+		if err != nil {
+			fmt.Fprintf(os.Stderr, "Warning: could not review package (%s not available)\n", clickReview)
+		}
+
+		cmd := exec.Command(clickReview, snapPackage)
+		cmd.Stdout = os.Stdout
+		cmd.Stderr = os.Stderr
+		// we ignore the error for now
+		_ = cmd.Run()
+	*/
 
 	fmt.Printf("Generated '%s' snap\n", snapPackage)
 	return nil