fix: discover correct path to the SSL certificates
Metadata
Current evaluation
Merged as 40e4317e, fixing pygit2 SSL certificate discovery on non-Ubuntu systems by deriving the path from meta/snap.yaml; closes #679.
Resolution: closed resolved
Reason: PR #729 was approved, passed all CI checks, and merged as commit 40e4317e. Its change resolves #679 by using the snap base to locate SSL certificates.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint && make test`?
- [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
Use `meta/snap.yaml` to determine the base from which SSL certs should be downloaded during `pygit2` import on non-Ubuntu OS.
Closes: #679
(CRAFT-4321)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged as 40e4317e, fixing pygit2 SSL certificate discovery on non-Ubuntu systems by deriving the path from meta/snap.yaml; closes #679. | |
| stealth/space-bunny-alpha | — | closed resolved | PR #729 merged as commit 40e4317e: craft_application/git/_utils.py now derives the SSL cert dir from the base in $SNAP/meta/snap.yaml (falling back to core22), fixing pygit2 cert lookup on non-Ubuntu bases. Closes #679 (CRAFT-4321). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged craft-application#729 as 40e4317e, fixing SSL certificate discovery for pygit2 on non-Ubuntu systems by deriving the path from meta/snap.yaml; closes #679. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #729 as commit 40e4317e, fixing SSL certificate discovery for pygit2 on non-Ubuntu systems by deriving the path from meta/snap.yaml. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged fix discovering correct SSL certificate paths for pygit2 on non-Ubuntu OS by parsing meta/snap.yaml. Approved by two reviewers, passed all CI checks, and closed issue #679. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-application#679
(confidence 100%)
PR #729 explicitly closes this issue with the corrected certificate-path discovery.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #211 fix(remote-build): set a known path to cacerts | craft-application | merged | Merged PR #211 as commit 86e1c2b5, setting a known CA-certificates path so pygit2 remote-build operations work under snap confinement. |