os/exec: LookPath uses a hard coded "/", assuming this is broken on Windows

[martinrode]

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

$ go version
go version go1.20.4 darwin/arm64

Does this issue reproduce with the latest release?

Yes

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

My assumption here is that os.LookPath will not work the same on Windows as it does on Linux / Mac. It will try to check the PATH environment even if the prog passed is utils\myprog.exe.

What did you do?

Read through the source code of LookPath.

What did you expect to see?

Something like filepath.Separator used instead of /.

What did you see instead?

A /

[qmuntal]

Thanks for reporting this @martinrode.

Could you link to the offending code using just /? I can't find it myself on the windows implementation at

go/src/os/exec/lp_windows.go

Line 65 in b9baf44

func LookPath(file string) (string, error) {

Take into account that the unix implementation is not the same as the windows one. Could it be that you are looking at the former (located here) instead of the later?

[martinrode]

Right, I am sorry for the confusion. The OS suffix is something I keep forgetting about ;-)