godeps plugin: support for bases
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Revamp the godeps plugins installation candidates to prefer the go snap
from the Snap Store first and provide a mechanism to be able to
install a build-packages golang-go instead on the supported bases.
LP: #1794784
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. The godeps plugin now prioritizes the Go snap from the Snap Store and supports installing golang-go as a build package on supported bases. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged updates to the godeps plugin that prioritize the Go snap from the Snap Store and enable golang-go package installation on supported bases. Changes passed coverage checks and were integrated into the main branch. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2323 go plugin: support for bases | snapcraft | merged | Merged. Introduces base support to the Go plugin, prioritizing the Go snap from the Snap Store and enabling golang-go installation on supported bases. Approved by reviewer and passed all CI checks. | |
| #1057 godeps plugin: support for go-packages. | snapcraft | merged | Merged into master, adding go-packages support to the godeps plugin to match the go plugin. Approved by a reviewer, passed CI checks, and increased overall code coverage. | |
| #1179 godeps plugin: add git to build-packages | snapcraft | merged | Merged a patch adding git to the godeps plugin build-packages. The change resolves a missing dependency for go get commands. Approved by one reviewer and merged directly without CI checks. | |
| #691 Add godeps plugin. | snapcraft | merged | Merged the godeps plugin for Snapcraft, resolving Launchpad bug #1595981. The change simplifies snapcraft.yaml for Go projects like Juju and adds tag support. Approved and verified by multiple contributors prior to integration. | |
| #1046 godeps plugin: work when GOBIN is set | snapcraft | merged | Merged into master. Resolves LP: #1656118 by fixing the godeps plugin to function correctly when GOBIN is set. Approved by one reviewer, passed CI, and modified two files (+15/-1 lines). | |
| #3431 godeps spread test: use latest/stable go snap | snapcraft | merged | Merged into master. Updates spread tests to use the latest/stable go snap instead of Xenial's go 1.6, resolving go get failures. Approved by reviewer, passed CI checks, and maintained code coverage. | |
| #2194 plugins: do not install go debs if go snap is used | snapcraft | merged | 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. | |
| #1794784 Update the godeps plugin to be base aware | snapcraft (launchpad) | closed | Closed without comments or resolution. The request to update the godeps plugin for base awareness appears abandoned or superseded by subsequent build system changes. |