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

Move to classic unconfined snap package #893

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

Conversation

LyzardKing
Copy link
Contributor

As discussed this PR adds the unconfined snap package.
The package will be larger since it must include the qt5 libraries needed by the ui.
It should work with most (if not all) external packages, since it has full access to the system (as if it were installed via apt)

@LyzardKing
Copy link
Contributor Author

@JanWielemaker This PR makes the install larger, since it is adding the qt libs needed for the UI interface. We can then see if it is possible to reduce the size after it has been tested. If you want to add it to the edge channel it could be useful for testing purposes.

@JanWielemaker
Copy link
Member

I'm still in doubt about this. I found recently that the snap has about 1,000 weekly users. That is not a lot, but enough to justify it. Thanks! The main problem with the snap is linking foreign extensions against it. I think we had one complaint on Discourse, right? Does this justify a bigger snap and (as I understand correctly) a not-preferred unconfined snap?

@LyzardKing
Copy link
Contributor Author

I think (but I'd need a better way to test it) that the unconfined snap should have less problems in linking foreign extensions.
If there's a good way to test this it might be good to give it a go.
It would also be interesting to figure out what is needed for the integration in a confined snap, but I'm less confident in the feasability of that last part..

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