Skip to content

Commit 589f56c

Browse files
Merge pull request #6432 from thaJeztah/rm_windows_warning
cli/command/image: build: remove permissions warning on Windows
2 parents fa5b741 + af65ee4 commit 589f56c

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

cli/command/image/build.go

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ import (
88
"io"
99
"os"
1010
"path/filepath"
11-
"runtime"
1211
"strings"
1312

1413
"github.com/distribution/reference"
@@ -361,16 +360,6 @@ func runBuild(ctx context.Context, dockerCli command.Cli, options buildOptions)
361360
return err
362361
}
363362

364-
// Windows: show error message about modified file permissions if the
365-
// daemon isn't running Windows.
366-
if response.OSType != "windows" && runtime.GOOS == "windows" && !options.quiet {
367-
_, _ = fmt.Fprintln(dockerCli.Out(), "SECURITY WARNING: You are building a Docker "+
368-
"image from Windows against a non-Windows Docker host. All files and "+
369-
"directories added to build context will have '-rwxr-xr-x' permissions. "+
370-
"It is recommended to double check and reset permissions for sensitive "+
371-
"files and directories.")
372-
}
373-
374363
// Everything worked so if -q was provided the output from the daemon
375364
// should be just the image ID and we'll print that to stdout.
376365
if options.quiet {

0 commit comments

Comments
 (0)