xeyownt / xseticon

xseticon from Paul Evans -- http://www.leonerd.org.uk/
GNU General Public License v2.0
34 stars 13 forks source link

Implement snap packaging, snaps are universal Linux packages #4

Closed brlin-tw closed 5 years ago

brlin-tw commented 5 years ago

This patch implements the necessary details to package xseticon as a snap that can be install and run on a wide range of GNU/Linux distribution with ease. Refer https://snapcraft.io for more details.

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

xeyownt commented 5 years ago

Thanks for the patch.

jglick commented 5 years ago

/snap/bin/xseticon fails for me:

Aborted at function fopen()

If I use /snap/xseticon/current/bin/xseticon it works fine.

brlin-tw commented 5 years ago

@jglick Snap apps have limited regular file access in nature, please ensure the icon is under your home directory, and /mnt and /media after you've connected the snap to the removable-media security confinement interfaces.

jglick commented 5 years ago

That makes the snap less than useful, unless I can reinstall with --classic. At least the tool ought to report a meaningful error message in this case.