Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

many: modify snap run to understand component hooks #13976

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

andrewphelpsj
Copy link
Collaborator

@andrewphelpsj andrewphelpsj commented May 17, 2024

This PR enables the "snap run" command to run component hooks. It modifies the meaning of the "--hook" parameter a bit to allow snapd to run component hooks like so:

snap run --hook install snapname+compname

@github-actions github-actions bot added the Needs Documentation -auto- Label automatically added which indicates the change needs documentation label May 17, 2024
@andrewphelpsj andrewphelpsj force-pushed the component-snap-run branch 3 times, most recently from b06f281 to 0523065 Compare May 23, 2024 16:24
… interfaces can build a complete label expression, including component hooks
…urrent revision of a component for a snap revision
…resent snap hooks, component hooks, and apps

This commit doesn't need to be here, and things will work without it.
But things were getting a bit complicated in runSnapConfine with
arguments that represented different things based on what we were
running.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Documentation -auto- Label automatically added which indicates the change needs documentation
Projects
None yet
1 participant