remove kf5 snap if it is unused
ClosedPublic

Authored by sitter on Feb 6 2019, 9:24 AM.

Details

Summary

previously we'd seed the snap to work around a defect in snapd where it
would not properly install it when installing a snap, since that is
now working we have no reason to preseed the snap anymore and furthermore
that particular snap is now deprecated in favor of the core18 (18.04-based)
snap. to facilitate the cleanup we'll force remove the snap iff it is
not connected to (i.e. unused)

Test Plan

gets removed when unused

Diff Detail

Branch
snap-cleanup
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 7944
Build 7962: arc lint + arc unit
sitter requested review of this revision.Feb 6 2019, 9:24 AM
sitter created this revision.
sitter retitled this revision from remove out kf5 snap if it is unused to remove kf5 snap if it is unused.
jriddell accepted this revision.Feb 11 2019, 4:29 PM

The neon-settings package should depend on ruby since it uses it a lot.

Running it I get
Feb 11 16:20:38 librodelgadro systemd[844]: neon-update-kf5-snap.service: Failed to execute command: Permission denied
Feb 11 16:20:38 librodelgadro systemd[844]: neon-update-kf5-snap.service: Failed at step EXEC spawning /usr/lib/neon_update/kf5_snap.rb: Permission denied
which may just be patch not setting exec permission so just double check that before pushing

works great otherwise both to remove the snap and after it is removed.

Ruby code looks good.

This revision is now accepted and ready to land.Feb 11 2019, 4:29 PM