Skip to content
This repository has been archived by the owner on Aug 29, 2020. It is now read-only.

Add snap installation instructions to the README #121

Conversation

brlin-tw
Copy link
Contributor

@brlin-tw brlin-tw commented Mar 3, 2019

Currently, the gotop-brlin snap name is used, should be replaced after the snap name issue is settled.

Signed-off-by: 林博仁(Buo-ren Lin) Buo.Ren.Lin@gmail.com

@brlin-tw brlin-tw force-pushed the patch/implement-snap-packaging-installation branch 2 times, most recently from ac6080a to 65a7466 Compare March 6, 2019 12:25
Signed-off-by: 林博仁(Buo-ren Lin) <Buo.Ren.Lin@gmail.com>
@brlin-tw brlin-tw force-pushed the patch/implement-snap-packaging-installation branch from 65a7466 to 76b2417 Compare March 6, 2019 14:16
@cjbassi
Copy link
Owner

cjbassi commented Apr 25, 2019

One question: I installed the package and it works for me, but I didn't have to run any of the snap connect commands (running snapd on Arch Linux). Were those commands necessary for you?

@brlin-tw
Copy link
Contributor Author

brlin-tw commented Apr 25, 2019

The following is the screenshot without connecting anything with the original Ubuntu kernel:

Screenshot_20190426_023759

Note the missing data in the disk usage and temperature widgets, these are due to the missing connection to the mount-observe and the hardware-observe interfaces. As the Arch linux kernel doesn't support full Snapd sandboxing features it is possible that some data is accessible even when the interface isn't connected, you can check out the sandboxing feature support status by running the snap debug sandbox-features command in a terminal.

Comparison after connecting to the 2 aforementioned interfaces:

Screenshot_20190426_024819

I'm not sure about the effectiveness of connecting the system-observe interface though, it might be unnecessary at all, I'll check if it can be dropped.

Most of the interface connecting instructions may be dropped after requesting auto-connection for those interfaces at the Snap Store's end, refer Process for aliases, auto-connections and tracks - doc - snapcraft.io for more details.

@cjbassi
Copy link
Owner

cjbassi commented Apr 26, 2019

I went ahead and updated the readme with the snap instructions. If you find any information about the permissions, then feel free to post it at #119. Thanks again!

@cjbassi cjbassi closed this Apr 26, 2019
@brlin-tw brlin-tw deleted the patch/implement-snap-packaging-installation branch April 27, 2019 15:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants