Switch from "winio" to "npipe".

Switch from "winio" to "npipe".

This is a hopeful fix to the linked bug. The problem is Go runtime panic
that manifested in coordination with an upgrade to Go 1.8. There are
several GitHub issue threads referencing similar issues with Windows and
"winio".

We have to disable the test because "npipe" is actually not a fully-
correct Go Listener implementation, and the test fails because of this.
This is okay, though, as this code has run for quite a while on Windows
using "npipe" without issue.

BUG=chromium:702105
TEST=None
R=vadimsh@chromium.org

Review-Url: https://codereview.chromium.org/2770013003
Committed: https://github.com/luci/luci-go/commit/6cb138dd7f4d900c66b22f22b9bbde1b0c4ff975

[iannucci1, 2017-03-24 02:03:42]

iannucci@google.com changed reviewers:
+ iannucci@google.com

[iannucci1, 2017-03-24 02:03:43]

lgtm

[dnj, 2017-03-24 02:03:52]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2017-03-24 02:04:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Vadim Sh., 2017-03-24 02:04:19]

lgtm

[commit-bot: I haz the power, 2017-03-24 02:10:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490321032802430, "parent_rev":
"11dcd1399b9104bb91d65370687c2d582c1770dc", "commit_rev":
"6cb138dd7f4d900c66b22f22b9bbde1b0c4ff975"}

[commit-bot: I haz the power, 2017-03-24 02:10:35]

Description was changed from

==========
Switch from "winio" to "npipe".

This is a hopeful fix to the linked bug. The problem is Go runtime panic
that manifested in coordination with an upgrade to Go 1.8. There are
several GitHub issue threads referencing similar issues with Windows and
"winio".

We have to disable the test because "npipe" is actually not a fully-
correct Go Listener implementation, and the test fails because of this.
This is okay, though, as this code has run for quite a while on Windows
using "npipe" without issue.

BUG=chromium:702105
TEST=None
R=vadimsh@chromium.org
==========

to

==========
Switch from "winio" to "npipe".

This is a hopeful fix to the linked bug. The problem is Go runtime panic
that manifested in coordination with an upgrade to Go 1.8. There are
several GitHub issue threads referencing similar issues with Windows and
"winio".

We have to disable the test because "npipe" is actually not a fully-
correct Go Listener implementation, and the test fails because of this.
This is okay, though, as this code has run for quite a while on Windows
using "npipe" without issue.

BUG=chromium:702105
TEST=None
R=vadimsh@chromium.org

Review-Url: https://codereview.chromium.org/2770013003
Committed:
https://github.com/luci/luci-go/commit/6cb138dd7f4d900c66b22f22b9bbde1b0c4ff975
==========

[commit-bot: I haz the power, 2017-03-24 02:10:37]

Committed patchset #1 (id:1) as
https://github.com/luci/luci-go/commit/6cb138dd7f4d900c66b22f22b9bbde1b0c4ff975