[PR #360] [MERGED] Fix snapcraft yaml #1610

New issue

Closed

opened 2026-05-05 07:57:39 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 07:57:39 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/debauchee/barrier/pull/360
Author: @maxiberta
Created: 7/10/2019
Status: ✅ Merged
Merged: 7/10/2019
Merged by: @AdrianKoshka

Base: master ← Head: fix-snapcraft-yaml

📝 Commits (8)

6bab29a Use version from git tag.
916138b Fix interface plugs.
ddc5fbd Add commands: barrier, barrierc, barriers.
3a4626b Drop custom qt5, use distro provided one.
05ccc75 Update build-packages and stage-packages.
93f4de2 Add desktop entry with an app icon.
9052135 Set snap icon and license.
cc15f0c Use appstream metadata from flathub.

📊 Changes

1 file changed (+53 additions, -74 deletions)

View changed files

📝 snap/snapcraft.yaml (+53 -74)

📄 Description

Hi! This should fix issue #301. Includes lots of fixes and improvements, such as:

Fix snap permissions - including networking and OpenGL.
Streamline build to use distro-provided Qt - saves build time (and space).
Drop -dev packages from final snap - reduces snap size from 289MB to 51MB.
Set snap icon and license, use flathub appstream metadata for summary and description.
Add desktop entry with an icon.
Expose commands: barrier, barrierc, barriers (with a barrier-kvm. prefix).
Use version from git tag - suitable for CI.

Btw, is the barrier-kvm snap official? It's currently in very bad shape. And, it should really be named barrier, so that the main command will be exposed to users as barrier with no prefix. I've checked and the name barrier is already reserved at the Snap Store, although with no published snaps. Is it reserved by you? If not, you can claim it here.

Finally, are there plans for a CI pipeline to publish e.g. daily builds of snaps? If not, you could start with https://build.snapcraft.io. For example, I've already set up my fork to be automatically built and published to the edge channel (hopefully temporarily; will drop this fork once there's an official up-to-date snap).

I can happily help if needed :)
Thanks!

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/debauchee/barrier/pull/360 **Author:** [@maxiberta](https://github.com/maxiberta) **Created:** 7/10/2019 **Status:** ✅ Merged **Merged:** 7/10/2019 **Merged by:** [@AdrianKoshka](https://github.com/AdrianKoshka) **Base:** `master` ← **Head:** `fix-snapcraft-yaml` --- ### 📝 Commits (8) - [`6bab29a`](https://github.com/debauchee/barrier/commit/6bab29ad2b30ff545d6166ebd590d99c3561cfe9) Use version from git tag. - [`916138b`](https://github.com/debauchee/barrier/commit/916138b62aa9ccbd09cf38a9c86f4fa995e539a0) Fix interface plugs. - [`ddc5fbd`](https://github.com/debauchee/barrier/commit/ddc5fbd884b6ee1533aef5f230df158a6c279bb1) Add commands: `barrier`, `barrierc`, `barriers`. - [`3a4626b`](https://github.com/debauchee/barrier/commit/3a4626b909fbb982f499595d26e963b56d979e0e) Drop custom qt5, use distro provided one. - [`05ccc75`](https://github.com/debauchee/barrier/commit/05ccc7589d4f3c2d9f1e4ccff795e23d57acd38c) Update build-packages and stage-packages. - [`93f4de2`](https://github.com/debauchee/barrier/commit/93f4de2f0123006783163bd1f25b650ea90e88f4) Add desktop entry with an app icon. - [`9052135`](https://github.com/debauchee/barrier/commit/905213530ee24a766584232a60fd3e409c8fb690) Set snap icon and license. - [`cc15f0c`](https://github.com/debauchee/barrier/commit/cc15f0c13d5089871a53f751ad918df2b9f3846d) Use appstream metadata from flathub. ### 📊 Changes **1 file changed** (+53 additions, -74 deletions) <details> <summary>View changed files</summary> 📝 `snap/snapcraft.yaml` (+53 -74) </details> ### 📄 Description Hi! This should fix issue #301. Includes lots of fixes and improvements, such as: - Fix snap permissions - including networking and OpenGL. - Streamline build to use distro-provided Qt - saves build time (and space). - Drop `-dev` packages from final snap - reduces snap size from 289MB to 51MB. - Set snap icon and license, use [flathub appstream metadata](https://github.com/flathub/com.github.debauchee.barrier/blob/master/com.github.debauchee.barrier.appdata.xml) for summary and description. - Add desktop entry with an icon. - Expose commands: `barrier`, `barrierc`, `barriers` (with a `barrier-kvm.` prefix). - Use version from git tag - suitable for CI. Btw, is the [barrier-kvm](https://snapcraft.io/barrier-kvm) snap official? It's currently in very bad shape. And, it should really be named `barrier`, so that the main command will be exposed to users as `barrier` with no prefix. I've checked and the name `barrier` is already reserved at the Snap Store, although with no published snaps. Is it reserved by you? If not, you can claim it [here](https://dashboard.snapcraft.io/register-snap/?name=barrier). Finally, are there plans for a CI pipeline to publish e.g. daily builds of snaps? If not, you could start with https://build.snapcraft.io. For example, I've already set up [my fork](https://snapcraft.io/barrier-maxiberta) to be [automatically built and published to the edge channel](https://build.snapcraft.io/user/maxiberta/barrier) (hopefully temporarily; will drop this fork once there's an official up-to-date snap). I can happily help if needed :) Thanks! --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>