From ded9e0d6dfbcfe0f2ccd448baf0acb4378365bac Mon Sep 17 00:00:00 2001
From: Solomon Hykes <solomon@dotcloud.com>
Date: Tue, 4 Feb 2014 18:41:37 +0000
Subject: [PATCH] builder: clearly display ONBUILD triggers during a build.

Docker-DCO-1.1-Signed-off-by: Solomon Hykes <solomon@docker.com> (github: shykes)
---
 buildfile.go                  | 10 +++++-----
 integration/buildfile_test.go |  4 ++--
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/buildfile.go b/buildfile.go
index c420e492d9..2a4b163bec 100644
--- a/buildfile.go
+++ b/buildfile.go
@@ -112,8 +112,8 @@ func (b *buildFile) CmdFrom(name string) error {
 	if nTriggers := len(b.config.OnBuild); nTriggers != 0 {
 		fmt.Fprintf(b.errStream, "# Executing %d build triggers\n", nTriggers)
 	}
-	for _, step := range b.config.OnBuild {
-		if err := b.BuildStep(step); err != nil {
+	for n, step := range b.config.OnBuild {
+		if err := b.BuildStep(fmt.Sprintf("onbuild-%d", n), step); err != nil {
 			return err
 		}
 	}
@@ -697,8 +697,7 @@ func (b *buildFile) Build(context io.Reader) (string, error) {
 		if len(line) == 0 || line[0] == '#' {
 			continue
 		}
-		fmt.Fprintf(b.outStream, "Step %d : %s\n", stepN, line)
-		if err := b.BuildStep(line); err != nil {
+		if err := b.BuildStep(fmt.Sprintf("%d", stepN), line); err != nil {
 			return "", err
 		}
 		stepN += 1
@@ -715,7 +714,8 @@ func (b *buildFile) Build(context io.Reader) (string, error) {
 }
 
 // BuildStep parses a single build step from `instruction` and executes it in the current context.
-func (b *buildFile) BuildStep(expression string) error {
+func (b *buildFile) BuildStep(name, expression string) error {
+	fmt.Fprintf(b.outStream, "Step %s : %s\n", name, expression)
 	tmp := strings.SplitN(expression, " ", 2)
 	if len(tmp) != 2 {
 		return fmt.Errorf("Invalid Dockerfile format")
diff --git a/integration/buildfile_test.go b/integration/buildfile_test.go
index f0baebc01e..6a7da70558 100644
--- a/integration/buildfile_test.go
+++ b/integration/buildfile_test.go
@@ -855,8 +855,8 @@ func TestBuildOnBuildTrigger(t *testing.T) {
 	onbuild run touch foobar
 	`,
 		nil, nil,
-		},
-	t, nil, true,
+	},
+		t, nil, true,
 	)
 	if err != nil {
 		t.Fatal(err)