/cc @alexbrainman

os.Hostname() / syscall.ComputerName() call GetComputerName()

https://msdn.microsoft.com/ja-jp/library/windows/desktop/ms724295%28v=vs.85%29.aspx

https://github.com/golang/go/blob/master/src/syscall/ztypes_windows.go#L144

And MAX_COMPUTERNAME_LENGTH is defined as 15 in mingw header.

We should use GetComputerNameEx instead.

https://msdn.microsoft.com/ja-jp/library/windows/desktop/ms724301(v=vs.85).aspx

I have a CL that adds the *Ex equivalent functions to internal/syscall/windows (to not change syscall) but when I modify os.Hostname to use them instead, the bootstrap build of os fails because it can't import internal/syscall/windows for some reason.

The regular build of os, and it's tests seem to run fine if the patch is applied to a previously compiled tree.

@minux That fixed bootstrap, but:

• the dependency check in go/build fails as os has a new dependency
• the change hard-codes a windows specific package in the list of built pacakges by cmd/dist so might break other GOOS's

@cookieo9 Did you add internal/... into https://github.com/golang/go/blob/master/src/go/build/deps_test.go#L126 ?

Yup, its in the CL. https://go-review.googlesource.com/#/c/5852/

@kartlee please check if https://go-review.googlesource.com/#/c/5852 does fixes your issue. Thank you.

@alexbrainman This works great...Thanks for quickly addressing it.

-Karthik

@cookieo9 fix it. Thank him.

Alex

@cookieo9 Thanks for the fix.

@kartlee I also added new hostname test on windows https://go-review.googlesource.com/6070 Can you, please, update your source and try the test on pdx-srv2012-wv01 ? I am interested if test fail - see what your hostname command says. Thank you.

Alex

@alexbrainman Sorry for not getting back earlier. I will run them today and update the ticket.

BTW, we are still using go 1.2 and doesn't want to upgrade for our current product release, so I just applied the code upstream into our custom package's Hostname() and added a similar nightly build test to inform when the result between os.Hostname(..) and custom package's Hostname(..) match for machine name greater than 15 characters. This will help us to move to your change with go 1.5.

Thank you for confirming.