Binaries installed via `go install` are not accessible on path about setup-go

[kjk]

I agree this should work by default but for now you can add to each step:

export PATH=${PATH}:`go env GOPATH`/bin

See https://presstige.io/p/Using-GitHub-Actions-with-Go-2ca9744b531f4f21bdae9976d1ccbb58#e86c1c15-6f43-4f43-935f-37895bd14208 for a fuller example (I use that for running staticcheck)

(just a tip, I'm not the maintainer).

[davidgraca]

Hi everyone, ressurecting the dead here...
Let's start by the beggining, you:

• put that on your gh action

- name: Setup Go environment uses: actions/[email protected]

• go check go env

GOPATH="/home/runner/go"

• go check $PATH

$PATH: /home/runner/.local/bin:/opt/pipx_bin:/home/runner/.cargo/bin:/home/runner/.config/composer/vendor/bin:/usr/local/.ghcup/bin:/home/runner/.dotnet/tools:/snap/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin

• and see that all your go installs are on GOPATH /bin

I added to my action:
export PATH=${PATH}:go env GOPATH/bin

and now it works fine.
Would it be possible to refix this?
Thanks

[bryanmacfarlane]

actually after further investigation it looks like the env and path are already getting set, but to the wrong values.

Agreed. Will fix.

[JeremyLoy]

In addition to that @devigned I think the ‘GOPATH’ statement two lines above that needs an ‘else’

[ARolek]

I just hit this problem and upgrading to v2-beta fixed this problem for me.

[bryanmacfarlane]

Awesome to hear. I’ll probably release v2 Monday

[JeremyLoy]

actually after further investigation it looks like the env and path are already getting set, but to the wrong values.

Below are some screenshots of the MacOS setup. go env is correctly saying that GOPATH is User/runner/go. This means that GOBIN will default to User/runner/go/bin since it is unset.

If we look at PATH, it seems that /usr/local/go/bin is getting set. By my understanding, this is not the correct value for any operating system, definitely not macOS.

[devigned]

setup-go/src/installer.ts

Lines 119 to 121 in 75259a5

	if (goBin) {
	core.exportVariable('GOBIN', goBin);
	}

Should have an else and export the default GOBIN into the path and set the PATH to include the GOBIN.

[devigned]

Agreed. If nothing is overridden, Golang defaults should be honored.

[lmittmann]

Duplicate of #14

[bryanmacfarlane]

This should be fixed in @v2-beta version of this action. See #14

[ARolek]

Nice! Thanks for putting this action together. I have been using it with a few projects.