Skip to content

Add ubuntu-desktop-session snap to the list#73

Open
sergio-costas wants to merge 1 commit intocanonical:mainfrom
sergio-costas:add-ubuntu-desktop-init
Open

Add ubuntu-desktop-session snap to the list#73
sergio-costas wants to merge 1 commit intocanonical:mainfrom
sergio-costas:add-ubuntu-desktop-init

Conversation

@sergio-costas
Copy link
Copy Markdown
Contributor

It is required to be able to automagically install it for future versions.

It is required to be able to automagically install it for future
versions.
@sergio-costas
Copy link
Copy Markdown
Contributor Author

@jhenstridge When you have some spare time, can you review this, merge it, and ask IS to sign it, please? I'm replacing the custom-modified ubuntu-core-desktop-init installer with the upstream ubuntu-desktop-init one.

@sergio-costas
Copy link
Copy Markdown
Contributor Author

I'm doing this in two stages: first adding the ubuntu-desktop-init snap to the list, and when everything is in place and migrated, remove the old ubuntu-core-desktop-init (it requires more PRs both here and in core-base-desktop, like canonical/core-base-desktop#61 ).

@jhenstridge
Copy link
Copy Markdown
Contributor

I would think you'd be able to test this change without updating the model. Using the makefile from this repo, you could do:

make pc-dangerous.img EXTRA_SNAPS="ubuntu-desktop-init=candidate"

Or if you're testing with a custom build of the base snap, include that in EXTRA_SNAPS too (separated by a space). If everything works, we can add remove ubuntu-core-desktop-init completely and only get IS to sign one new copy of the model.

@sergio-costas
Copy link
Copy Markdown
Contributor Author

I already tested it, and works fine. That's why I want to add a PR for this.

@sergio-costas
Copy link
Copy Markdown
Contributor Author

I even tested if signing the .model files with my own key, and worked too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants