Bug #16303
tails-persistence-setup crashes due to missing electrum in feature/buster
0%
Description
electrum
was disabled in feature/buster
(see Bug #16204) which leads to a crash of tails-persistence-setup
after the persistence partition has been set up: when the window with the application-related choices is being prepared, the missing electrum icon in the Adwaita theme triggers a crash. Visiting e.g. the /usr/share/icons/Adwaita/scalable/apps
directory and copying a random SVG file into electrum.svg
prevents the crash.
Affected version: tails-persistence-setup
2.0.2-1.
This was caught by the test suite with persistence.feature
(waiting happens for long minutes while the window/app is gone); and reproducing it outside the test suite, starting the application from a terminal, made it possible to figure out what was happening.
Various thoughts:
- This (in
lib/Tails/Persistence/Configuration/Setting.pm
) should likely try and catch errors, maybe falling back to a “broken image” icon when the requested one isn’t available?
method _build_icon () {
Gtk3::Image->new_from_pixbuf($self->icon_theme->load_icon(
$self->icon_name, 48, 'use-builtin'
));
}
- Methods mentioned in https://developer.gnome.org/gtk3/stable/GtkImage.html seem to handle the “broken image” part but not those from https://developer.gnome.org/gtk3/stable/GtkIconTheme.html (and we seem to be using those from the latter).
- In the meanwhile, we could patch out the electrum part of the presets through a local patch in tails.git
- Or we could introduce a temporary electrum icon in tails.git; this would solve the crash but I’m not sure what would happen with the other widgets, so I think I’d rather patch out electrum entirely…
Subtasks
Related issues
Related to Tails - |
Resolved | 2018-12-07 |
History
#1 Updated by intrigeri 2019-01-06 07:05:31
- Status changed from New to Confirmed
> * In the meanwhile, we could patch out the electrum part of the presets through a local patch in tails.git
This looks like the cheapest short-term workaround. Once we have a clearer idea of the future of Electrum in Tails (or lack thereof) we can decide what our long-term course of action should be.
#2 Updated by intrigeri 2019-01-06 10:27:53
- related to
Bug #16204: Electrum is missing in feature/buster added
#3 Updated by CyrilBrulebois 2019-01-07 10:38:19
- Status changed from Confirmed to In Progress
Applied in changeset commit:tails|282920f0e2cdf6f7483e90c8b4ce2ce8dda80cf6.
#4 Updated by intrigeri 2019-01-07 13:34:12
- Status changed from In Progress to Resolved