From c41c23813c15219a78e4c2451f83bcdb74c6e071 Mon Sep 17 00:00:00 2001
From: Ian Campbell <ijc@docker.com>
Date: Mon, 4 Mar 2019 17:29:06 +0000
Subject: [PATCH] Disable `docker system dial-stdio` on Windows

The `conn` here is `*winio.win32MessageBytePipe` which does not have a
`CloseRead` method (it does have `CloseWrite`) resulting in:

    docker@WIN-NUC0 C:\Users\docker>.\docker-windows-amd64.exe system dial-stdio
    the raw stream connection does not implement halfCloser

Also disable the path which uses this for cli-plugins on Windows.

Signed-off-by: Ian Campbell <ijc@docker.com>
---
 cli-plugins/plugin/plugin.go | 7 ++++++-
 cli/command/system/cmd.go    | 8 +++++++-
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/cli-plugins/plugin/plugin.go b/cli-plugins/plugin/plugin.go
index 99ef36d845..c8e4f9f465 100644
--- a/cli-plugins/plugin/plugin.go
+++ b/cli-plugins/plugin/plugin.go
@@ -4,6 +4,7 @@ import (
 	"encoding/json"
 	"fmt"
 	"os"
+	"runtime"
 	"sync"
 
 	"github.com/docker/cli/cli"
@@ -74,7 +75,11 @@ func PersistentPreRunE(cmd *cobra.Command, args []string) error {
 		}
 		// flags must be the original top-level command flags, not cmd.Flags()
 		options.opts.Common.SetDefaultOptions(options.flags)
-		err = options.dockerCli.Initialize(options.opts, withPluginClientConn(options.name))
+		var initopts []command.InitializeOpt
+		if runtime.GOOS != "windows" {
+			initopts = append(initopts, withPluginClientConn(options.name))
+		}
+		err = options.dockerCli.Initialize(options.opts, initopts...)
 	})
 	return err
 }
diff --git a/cli/command/system/cmd.go b/cli/command/system/cmd.go
index 6accb98f0c..1bb5374ccb 100644
--- a/cli/command/system/cmd.go
+++ b/cli/command/system/cmd.go
@@ -1,6 +1,8 @@
 package system
 
 import (
+	"runtime"
+
 	"github.com/docker/cli/cli"
 	"github.com/docker/cli/cli/command"
 	"github.com/spf13/cobra"
@@ -19,8 +21,12 @@ func NewSystemCommand(dockerCli command.Cli) *cobra.Command {
 		NewInfoCommand(dockerCli),
 		newDiskUsageCommand(dockerCli),
 		newPruneCommand(dockerCli),
-		newDialStdioCommand(dockerCli),
 	)
+	if runtime.GOOS != "windows" {
+		cmd.AddCommand(
+			newDialStdioCommand(dockerCli),
+		)
+	}
 
 	return cmd
 }