Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 653786 - [gnome-overlay] gnome-base/gnome-desktop-3.26.2 won't generate thumbnails without bubblewrap installed
Summary: [gnome-overlay] gnome-base/gnome-desktop-3.26.2 won't generate thumbnails wit...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Overlays (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL: https://bugzilla.gnome.org/show_bug.c...
Whiteboard:
Keywords:
Depends on:
Blocks: gnome-3.26
  Show dependency tree
 
Reported: 2018-04-22 14:57 UTC by Mike Auty
Modified: 2018-12-14 19:03 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
bubblewrap-0.2.1 ebuild (bubblewrap-0.2.1.ebuild,809 bytes, text/plain)
2018-04-22 14:58 UTC, Mike Auty
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Auty gentoo-dev 2018-04-22 14:57:06 UTC
Having just installed gnome-desktop-3.26.2 I noticed that thumbnails weren't being generated.  A file was being generated in .cache/thumbnails/fail/ but never under normal or large.

As best I could find, this is a gnome-desktop issue [1] which is resolved by installing a tool called bubblewrap (it's not clear why this isn't part of the configure script)?

Either way, installing the attached (to follow) ebuild (originally for an old EAPI from foo-overlay) resolves the issue and thumbnails are correctly generated and displayed in nautilus.
Comment 1 Mike Auty gentoo-dev 2018-04-22 14:58:05 UTC
Created attachment 528228 [details]
bubblewrap-0.2.1 ebuild

Ebuild for bubblewrap, as required by gnome-desktop in order to generate thumbnails.
Comment 2 Mike Auty gentoo-dev 2018-04-22 14:59:37 UTC
Sorry, [1] is the address in the URL field:

https://bugzilla.gnome.org/show_bug.cgi?id=785963#c19
Comment 3 Mart Raudsepp gentoo-dev 2018-04-22 15:42:49 UTC
yeah, bubblewrap is something on top of libseccomp iirc; there was some upstream churn about it being hard requiring in some package for all linux, which was wrong as libseccomp isn't available for all linux supported architectures, and I believe that patch got accepted. So it's a bit confusing - I understood that without bwrap thumbnailing is just not sandboxed (but the build system would enforce sandboxing by default for linux on architectures that support seccomp) - but here you are seeing silent failures to thumbnail :(
Maybe this was in newer gnome-desktop versions than 3.26, or gdk-pixbuf bits of it.
Comment 4 Larry the Git Cow gentoo-dev 2018-12-14 19:03:40 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ccace77bf2e557b738f017d1139806cb2ed2a5d6

commit ccace77bf2e557b738f017d1139806cb2ed2a5d6
Author:     Mart Raudsepp <leio@gentoo.org>
AuthorDate: 2018-12-14 18:22:42 +0000
Commit:     Mart Raudsepp <leio@gentoo.org>
CommitDate: 2018-12-14 19:01:44 +0000

    gnome-base/gnome-desktop: bump to 3.26.2
    
    Also force seccomp when available, or configure fails. This is not
    patched to be optional on purpose - this kind of security is NOT
    optional as far as I'm concerned.
    
    Compared to gnome overlay version, this one:
    
    * adds a big patchset for more sandboxed thumbnailing fixes,
      memory leak fixes and other misc changes that don't break ABI
    * patch to hopefully fix thumbnailing on non-seccomp architectures
      (bwrap was still enabled, leading to broken runtime as bwrap
      requires seccomp)
    * RDEPENDs on sys-apps/bubblewrap with seccomp as required
    * removes USE=dbus dep on glib
    * revbumps over overlay revision to ensure upgrades
    
    Closes: https://bugs.gentoo.org/653786
    Signed-off-by: Mart Raudsepp <leio@gentoo.org>
    Package-Manager: Portage-2.3.52, Repoman-2.3.11

 gnome-base/gnome-desktop/Manifest                  |  2 +
 .../gnome-desktop/gnome-desktop-3.26.2-r2.ebuild   | 67 ++++++++++++++++++++++
 profiles/base/package.use.force                    |  4 ++
 3 files changed, 73 insertions(+)