Merge pull request #1288 from TomChv/fix/fs-naming

plan/task/mkdir.go

@@ -24,8 +24,8 @@ func (t *mkdirTask) Run(ctx context.Context, pctx *plancontext.Context, s solver
 		return nil, err
 	}
 
-	// Mode (int)
-	mode, err := v.Lookup("mode").Int64()
+	// Permissions (int)
+	permissions, err := v.Lookup("permissions").Int64()
 	if err != nil {
 		return nil, err
 	}
@@ -58,7 +58,7 @@ func (t *mkdirTask) Run(ctx context.Context, pctx *plancontext.Context, s solver
 
 	// Add Mkdir operation on input llb state
 	outputState := inputState.File(
-		llb.Mkdir(path, fs.FileMode(mode), mkdirOpts...),
+		llb.Mkdir(path, fs.FileMode(permissions), mkdirOpts...),
 		withCustomName(v, "Mkdir %s", path),
 	)
 

plan/task/writefile.go

@@ -48,7 +48,7 @@ func (t *writeFileTask) Run(ctx context.Context, pctx *plancontext.Context, s so
 		return nil, err
 	}
 
-	mode, err := v.Lookup("mode").Int64()
+	permissions, err := v.Lookup("permissions").Int64()
 
 	if err != nil {
 		return nil, err
@@ -67,7 +67,7 @@ func (t *writeFileTask) Run(ctx context.Context, pctx *plancontext.Context, s so
 	}
 
 	outputState := inputState.File(
-		llb.Mkfile(path, fs.FileMode(mode), contents),
+		llb.Mkfile(path, fs.FileMode(permissions), contents),
 		withCustomName(v, "WriteFile %s", path),
 	)
 
@@ -77,11 +77,11 @@ func (t *writeFileTask) Run(ctx context.Context, pctx *plancontext.Context, s so
 		return nil, err
 	}
 
-	fs := pctx.FS.New(result)
+	outputFS := pctx.FS.New(result)
 
 	output := compiler.NewValue()
 
-	if err := output.FillPath(cue.ParsePath("output"), fs.MarshalCUE()); err != nil {
+	if err := output.FillPath(cue.ParsePath("output"), outputFS.MarshalCUE()); err != nil {
 		return nil, err
 	}
 

stdlib/europa/dagger/engine/fs.cue

@@ -11,8 +11,8 @@ package engine
 	// It can be nested (e.g : "/foo" or "/foo/bar")
 	path: string
 
-	// Permissions to set
-	mode: *0o755 | int
+	// Permissions of the directory
+	permissions: *0o755 | int
 
 	// If set, it creates parents' directory if they do not exist
 	parents: *true | false
@@ -30,9 +30,6 @@ package engine
 	path: string
 	// Contents of the file
 	contents: string
-	// Output filesystem tree
-	// FIXME: this is a no-op. No output needed.
-	output: #FS
 }
 
 // Write a file to a filesystem tree, creating it if needed
@@ -46,8 +43,7 @@ package engine
 	// Contents to write
 	contents: string
 	// Permissions of the file
-	// FIXME: rename to 'permissions' for consistency
-	mode: int
+	permissions: int
 	// Output filesystem tree
 	output: #FS
 }

tests/tasks/mkdir/mkdir.cue

@@ -19,7 +19,7 @@ engine.#Plan & {
 			input:       mkdir.output
 			path:        "/test/foo"
 			contents:    "bar"
-			mode:     700
+			permissions: 700
 		}
 
 		readChecker: engine.#ReadFile & {

tests/tasks/mkdir/mkdir_failure_disable_parents.cue

@@ -20,7 +20,7 @@ engine.#Plan & {
 			input:       mkdir.output
 			path:        "/test/baz/foo"
 			contents:    "bar"
-			mode:     700
+			permissions: 700
 		}
 
 		readChecker: engine.#ReadFile & {

tests/tasks/mkdir/mkdir_parents.cue

@@ -19,7 +19,7 @@ engine.#Plan & {
 			input:       mkdir.output
 			path:        "/test/baz/foo"
 			contents:    "bar"
-			mode:     700
+			permissions: 700
 		}
 
 		readChecker: engine.#ReadFile & {

tests/tasks/writefile/writefile.cue

@@ -13,7 +13,7 @@ engine.#Plan & {
 			input:       pull.output
 			path:        "/testing"
 			contents:    "1,2,3"
-			mode:     700
+			permissions: 700
 		}
 		readfile: engine.#ReadFile & {
 			input: write.output

tests/tasks/writefile/writefile_failure_diff_contents.cue

@@ -13,7 +13,7 @@ engine.#Plan & {
 			input:       pull.output
 			path:        "/testing"
 			contents:    "1,2,3,4"
-			mode:     700
+			permissions: 700
 		}
 		readfile: engine.#ReadFile & {
 			input: write.output