Fix 'govvv list', add regression test

Signed-off-by: Ahmet Alp Balkan <ahmetalpbalkan@gmail.com>
ahmetb · Aug 18, 2016 · 826804e · 826804e
1 parent 905d46c
commit 826804e
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/integration-test/test.bats b/integration-test/test.bats
@@ -40,6 +40,12 @@
     [[ "$output" =~ ^-X\ .* ]]
 }
 
+@test "govvv list" {
+    run govvv list ./integration-test/app-empty
+    echo "$output"
+    [ "$status" -eq 0 ]
+}
+
 @test "govvv build - program with no compile-time variables" {
     tmp="${BATS_TMPDIR}/a.out"
     run govvv build -o "$tmp" ./integration-test/app-empty  

diff --git a/main.go b/main.go
@@ -47,9 +47,13 @@ func main() {
 		fmt.Print(ldflags)
 		return
 	}
-	args, err = addLdFlags(args[1:], ldflags)
-	if err != nil {
-		log.Fatalf("failed to add ldflags to args: %v", err)
+	args = args[1:] // rm executable name
+
+	if args[0] == "build" || args[0] == "install" {
+		args, err = addLdFlags(args, ldflags)
+		if err != nil {
+			log.Fatalf("failed to add ldflags to args: %v", err)
+		}
 	}
 
 	if findArg(args, flDryRun) != -1 {