← Back to issue list

Update snap install command to include --classic

View original Github issue

Metadata

Project
charmcraft
Number
#805
Type
pull request
State
closed
Author
NucciTheBoss
Labels
—
Created
Updated
Closed

Current evaluation

Closed unmerged by author over a CLA check; superseded by PR #813, which added --classic to the README snap install command (commit 378f2271, merge f3005378).

Resolution: closed superseded

Reason: The author closed #805 himself to reauthor the commit with a CLA-compliant email; the reworked PR #813 was merged, adding --classic to the README install command in commit 378f2271 (merge f3005378).

Evaluated by: mr-cal-bot

No scores available.

Issue body

Very basic pull request focused on updating the `snap install ...` command for charmcraft in the main README of this repository. ### The problem The snap install command is originally written as `sudo snap install charmcraft`, however, if you look in the Snap Store, you see that the charmcraft snap has `--classic` confinement. This means that the `--classic` flag needs to included when you install charmcraft using snap. The absence of the `--classic` flag is made evident when you go to install charmcraft as it throws an error if you use the original command: ``` $ sudo snap install charmcraft error: This revision of snap "charmcraft" was published using classic confinement and thus may perform arbitrary system changes outside of the security sandbox that snaps are usually confined to, which may put your system at risk. If you understand and want to proceed repeat the command including --classic. ``` ### The fix Update the snap install command to include `--classic` so that the install is successful and does not throw an error about confinement. This will hopefully be beneficial to new charmers who are not familiar with snap confinement, and are working off of this repository to install charmcraft on their system.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded Closed unmerged by author over a CLA check; superseded by PR #813, which added --classic to the README snap install command (commit 378f2271, merge f3005378).
stealth/space-bunny-alpha — closed superseded Closed unmerged after CLA-related reauthoring; PR #813 superseded it, adding --classic to the README's snap install command in commit 378f2271 (merge f3005378).
stealth/space-bunny-alpha — closed stale Author closed unmerged PR #813 after its CLA check failed; the author intended to reauthor the commit. The --classic README change had been committed separately as 378f2271.
qwen/qwen3.6-35b-a3b — — Closed and abandoned by author to reauthor commit for CLA compliance. The PR updates the snap install command to include the required --classic flag and was approved. Not merged.
qwen3.6-35b-a3b-mtp-q6 — — Closed by author to reauthor commit and resolve CLA check failure. The --classic flag update was not merged and requires resubmission.

Update history

No update history recorded yet.

Related work

  • Superseded By: canonical/charmcraft#813 (confidence 96%)

    Reauthored CLA-compliant version of this PR; merged with commit 378f2271 (merge f3005378).

  • Related To: canonical/charmcraft#288 (confidence 70%)

    Earlier PR that moved the charmcraft snap to classic confinement, the reason --classic is required.

Related issues

Issue Project State Summary Similarity
#813 Update snap install to use --classic charmcraft merged Merged PR #813 (CLA-compliant reauthor of #805) added --classic to the README snap install command; commit 378f2271, merge f3005378. #805 was closed unmerged.
94%
#949 project: support building classic snaps snapcraft merged Merged PR #949 in commit 058f5626d, adding classic-snap build support and core-snap path configuration.
73%