From a1522ec01c21807e657e840760a8186f482f2271 Mon Sep 17 00:00:00 2001 From: Erik Hollensbe Date: Fri, 15 Aug 2014 09:29:35 -0700 Subject: [PATCH] builder: move the evaluator package code directly into builder Docker-DCO-1.1-Signed-off-by: Erik Hollensbe (github: erikh) --- builder/builder.go | 11 +++++------ builder/{evaluator => }/dispatchers.go | 2 +- builder/{evaluator => }/evaluator.go | 4 ++-- builder/{evaluator => }/internals.go | 2 +- builder/job.go | 3 +-- builder/{evaluator => }/support.go | 2 +- 6 files changed, 11 insertions(+), 13 deletions(-) rename builder/{evaluator => }/dispatchers.go (99%) rename builder/{evaluator => }/evaluator.go (98%) rename builder/{evaluator => }/internals.go (99%) rename builder/{evaluator => }/support.go (98%) diff --git a/builder/builder.go b/builder/builder.go index d99d1ad9b6..e655f9da60 100644 --- a/builder/builder.go +++ b/builder/builder.go @@ -1,17 +1,16 @@ package builder import ( - "github.com/docker/docker/builder/evaluator" "github.com/docker/docker/runconfig" ) -// Create a new builder. -func NewBuilder(opts *evaluator.BuildOpts) *evaluator.BuildFile { - return &evaluator.BuildFile{ +// Create a new builder. See +func NewBuilder(opts *BuildOpts) *BuildFile { + return &BuildFile{ Dockerfile: nil, Config: &runconfig.Config{}, Options: opts, - TmpContainers: evaluator.UniqueMap{}, - TmpImages: evaluator.UniqueMap{}, + TmpContainers: UniqueMap{}, + TmpImages: UniqueMap{}, } } diff --git a/builder/evaluator/dispatchers.go b/builder/dispatchers.go similarity index 99% rename from builder/evaluator/dispatchers.go rename to builder/dispatchers.go index 23e16b000e..fe0b5be579 100644 --- a/builder/evaluator/dispatchers.go +++ b/builder/dispatchers.go @@ -1,4 +1,4 @@ -package evaluator +package builder // This file contains the dispatchers for each command. Note that // `nullDispatch` is not actually a command, but support for commands we parse diff --git a/builder/evaluator/evaluator.go b/builder/evaluator.go similarity index 98% rename from builder/evaluator/evaluator.go rename to builder/evaluator.go index dbf4e30839..01805f892d 100644 --- a/builder/evaluator/evaluator.go +++ b/builder/evaluator.go @@ -1,4 +1,4 @@ -// evaluator is the evaluation step in the Dockerfile parse/evaluate pipeline. +// builder is the evaluation step in the Dockerfile parse/evaluate pipeline. // // It incorporates a dispatch table based on the parser.Node values (see the // parser package for more information) that are yielded from the parser itself. @@ -17,7 +17,7 @@ // before and after each step, such as creating an image ID and removing temporary // containers and images. Note that ONBUILD creates a kinda-sorta "sub run" which // includes its own set of steps (usually only one of them). -package evaluator +package builder import ( "bytes" diff --git a/builder/evaluator/internals.go b/builder/internals.go similarity index 99% rename from builder/evaluator/internals.go rename to builder/internals.go index 9519c683b3..b1d5b21f03 100644 --- a/builder/evaluator/internals.go +++ b/builder/internals.go @@ -1,4 +1,4 @@ -package evaluator +package builder // internals for handling commands. Covers many areas and a lot of // non-contiguous functionality. Please read the comments. diff --git a/builder/job.go b/builder/job.go index 98a24df898..e1233e0a08 100644 --- a/builder/job.go +++ b/builder/job.go @@ -8,7 +8,6 @@ import ( "strings" "github.com/docker/docker/archive" - "github.com/docker/docker/builder/evaluator" "github.com/docker/docker/daemon" "github.com/docker/docker/engine" "github.com/docker/docker/pkg/parsers" @@ -86,7 +85,7 @@ func (b *BuilderJob) CmdBuild(job *engine.Job) engine.Status { sf := utils.NewStreamFormatter(job.GetenvBool("json")) - opts := &evaluator.BuildOpts{ + opts := &BuildOpts{ Daemon: b.Daemon, Engine: b.Engine, OutStream: &utils.StdoutFormater{ diff --git a/builder/evaluator/support.go b/builder/support.go similarity index 98% rename from builder/evaluator/support.go rename to builder/support.go index b543676ecf..08cfa8defa 100644 --- a/builder/evaluator/support.go +++ b/builder/support.go @@ -1,4 +1,4 @@ -package evaluator +package builder import ( "regexp"