← Back to issue list

plugins: do not install go debs if go snap is used

View original Github issue

Metadata

Project
snapcraft
Number
#2194
Type
pull request
State
merged
Author
mvo5
Labels
Created
Updated
Closed

Current evaluation

Merged a fix to the go plugin that skips installing the golang-go deb package when a go build-snap is defined, resolving PATH conflicts. Approved by reviewers, passed CI with full coverage, and integrated into master.

Suggested action:

No scores available.

Issue body

Right now the go plugin will always pull in the golang-go build-package. This is problematic when the project also has a "go" build-snap. The deb package comes before in PATH and it seems like there is no easy way to override this. This PR fixes it by simply not installing the golang-go deb if there is a build-snap already defined.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a fix to the go plugin that skips installing the golang-go deb package when a go build-snap is defined, resolving PATH conflicts. Approved by reviewers, passed CI with full coverage, and integrated into master.
qwen3.6-35b-a3b-mtp-q6 Merged to master. The Go plugin now skips installing the golang-go deb when a go build-snap is defined, resolving PATH priority conflicts. Maintainers accepted the fix, noting future logic may be needed for mixed snap and deb part configurations.
qwen3.6-35b-a3b-mtp-q6 Merged into master. The go plugin now skips installing the golang-go deb when a go build-snap is specified, resolving PATH conflicts. Approved as a functional fix, with reviewers noting multi-part build scenarios may require future logic updates.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1616985 the go plugin doesn't use go build-snaps snapcraft (launchpad) closed Resolved by adding build-snap support to snapcraft and publishing a dedicated go snap. The go plugin was updated to use the go build-snap instead of the legacy deb package, enabling support for newer Go versions.
76%
#78 plugins: add go plugin (CRAFT-48) craft-parts merged Merged a Go plugin for go.mod-based projects, adapted from Snapcraft V2. Approved by two reviewers, passed CI, and introduced 425 additions across seven files. Co-authored by Sergio Schvezov.
72%
#2061 go: only use Go build package if not using the snap snapcraft closed Abandoned without merging. The author withdrew the Go plugin changes after offline discussions determined the implementation required further design review before proceeding.
72%
#3046 plugins: introduce v2.GoPlugin snapcraft merged Merged after approval and CI checks. Introduces v2.GoPlugin for Go plugin support requiring go.mod and adds .get_build_snaps as an abstract method to PluginV2. Changes span 26 files with 366 additions and 49 deletions.
71%
#2328 godeps plugin: support for bases snapcraft merged Merged. Updated the godeps plugin to prioritize the Go snap and added support for installing golang-go via build-packages on supported bases. Approved, passed CI, and integrated.
70%
#1858785 go plugin doesn't strip binaries snapcraft (launchpad) open Feature request to add an option to strip binaries built by the go plugin to reduce snap size. 6.6 years old with zero comments and no labels; the go plugin has since been removed from snapcraft, making the request moot.
70%
#1612741 Snapcraft go plugin install issues snapcraft (launchpad) open Snapcraft's go plugin fails to build godoc because go install writes to $GOROOT/bin (permission denied), a known golang bug; reporter also asks snapcraft to prevent installs into $GOROOT or force install path inside the snap. No labels, no comments, no maintainer activity in ~10 years.
70%