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

cmd/gofmt: not supported by windows #18026

Closed
alexbrainman opened this issue Nov 23, 2016 · 3 comments
Closed

cmd/gofmt: not supported by windows #18026

alexbrainman opened this issue Nov 23, 2016 · 3 comments
Assignees
Milestone

Comments

@alexbrainman
Copy link
Member

@alexbrainman alexbrainman commented Nov 23, 2016

What version of Go are you using (go version)?

go version devel +50c4dbc Wed Nov 23 00:44:01 2016 +0000 windows/386

What operating system and processor architecture are you using (go env)?

set GOARCH=386
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=386
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=c:\dev
set GORACE=
set GOROOT=c:\dev\go
set GOTOOLDIR=c:\dev\go\pkg\tool\windows_386
set GCCGO=gccgo
set GO386=
set CC=gcc
set GOGCCFLAGS=-m32 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\DOCUME1\brainman\LOCALS1\Temp\go-build246447175=/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

What did you do?

C:\dev\src\a>dir /b
foo.go

C:\dev\src\a>type foo.go
package main

import (
    "log"
    "os"
)

func main() {
    _, err := os.Stdin.Stat()
    if err != nil {
        log.Fatalln(err)
    }
}
C:\dev\src\a>go fmt
foo.go
chmod foo.go.662713035: not supported by windows
exit status 2

C:\dev\src\a>

What did you expect to see?

I expect to see no output from gofmt

What did you see instead?

gofmt failed with error message. Chmod is not implemented on Windows, so it is failing.

Alex

@bradfitz bradfitz added this to the Go1.8 milestone Nov 23, 2016
@bradfitz
Copy link
Contributor

@bradfitz bradfitz commented Nov 23, 2016

@griesemer, this is a regression from your #8984 fix (b188b4c).

@bradfitz
Copy link
Contributor

@bradfitz bradfitz commented Nov 23, 2016

@gopherbot
Copy link

@gopherbot gopherbot commented Nov 23, 2016

CL https://golang.org/cl/33477 mentions this issue.

@gopherbot gopherbot closed this in 75c1381 Nov 23, 2016
@golang golang locked and limited conversation to collaborators Nov 23, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.