Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

window get #2

Closed
zytzjx opened this issue Jun 30, 2017 · 4 comments
Closed

window get #2

zytzjx opened this issue Jun 30, 2017 · 4 comments

Comments

@zytzjx
Copy link

zytzjx commented Jun 30, 2017

github.com/go-cmd/cmd

..\src\github.com\go-cmd\cmd\cmd.go:123: undefined: syscall.Kill
..\src\github.com\go-cmd\cmd\cmd.go:173: unknown field 'Setpgid' in struct literal of type syscall.SysProcAttr

How to Fix it?

window10 x64
my environment:
set GOARCH=amd64
set GOBIN=D:\Go\bin
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=E:\Works\GoLang
set GORACE=
set GOROOT=D:\Go
set GOTOOLDIR=D:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\jeffery\AppData\Local\Temp\go-build544971735=/tmp/go-build -gno-record-gcc-switches
set CXX=g++
set CGO_ENABLED=1
set PKG_CONFIG=pkg-config
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2

@daniel-nichter
Copy link
Member

I don't know; I don't use Windows. The sys* stuff is low-level and system-specific. I don't know what the Windows equivalent is, or if there is an equivalent. You probably know more about Go on Windows, so if you can find the solution and submit a patch, please do so.

@MedAnd
Copy link

MedAnd commented Jan 16, 2018

@daniel-nichter
Copy link
Member

I'd rather not mess around with this low-level stuff just to make it work on platform this package is not intended to support (Windows). The use of low-level sys* stuff is a necessary evil for Linux; I wish it wasn't necessary but Go doesn't expose the needed functionality through higher-level means. So the less we mess with the low-level stuff the better.

@daniel-nichter
Copy link
Member

Windows should work as of v1.1.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants