| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
| |
Tumbler could get stuck if a previous request was cancelled because of
this. An obsolete assert is also removed which may prevent an error
signal from being sent.
Fixes #56.
|
|
|
|
| |
Completes previous commit, related to !21.
|
|
|
|
|
|
|
|
|
| |
This is primarily for Flatpak packaging, where a "private service" is
desirable, although one could imagine using it in other contexts. The
prefix must be specified via the environment variable
`TUMBLER_SERVICE_NAME_PREFIX`.
Related to xfconf#27, see !21 for more details.
|
|
|
|
|
| |
In particular, $EGREP was introduced in
d82e509551db98ff70e47d1990f759948ee7fecd whithout check.
|
|
|
|
| |
An oversight in 49ffba98d7cf9626a1470bc49a53596300fe3550.
|
| |
|
| |
|
|
|
|
|
|
|
| |
The "X-Tumbler Settings" group is searched in each desktop file with the
same keys as in `tumbler.rc`, where the default configuration is now.
Closes #52.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The request thumbnailer array no longer contains one thumbnailer per
URI, but the list of available thumbnailers for this URI (after applying
the filters of the configuration file), sorted by priority.
For a given URI, we browse the thumbnailer list until we succeed in
generating a thumbnail. An error signal is only issued if the last
thumbnailer has failed. Intermediate errors are only displayed if debug
logging is enabled.
Closes #50.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When running under a systemd user session, this results in tumblerd
being placed in in its own cgroup, whereas previously it would be a
child of dbus-daemon in the dbus.service cgroup.
It also avoids races between up to three separate instances of the
same executable being started for its three names, which could result
in spurious activation failures if they are used in quick succession,
similar to <https://bugs.freedesktop.org/show_bug.cgi?id=53220> in
telepathy-mission-control.
Closes #14.
|
|
|
|
| |
Closes #23.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A parameter is added to Tumbler's internal error signals to account for
the error domain, and the error code `TUMBLER_ERROR_OTHER_ERROR_DOMAIN`
is added to indicate that the error originated from a third-party API.
When sending the error signal via D-Bus, cancelling errors are filtered
out, and when the error originates from a third-party API, the error
message is prefixed with the domain and code of that error, in a
possibly parsable format.
Fixes #49.
|
| |
|
| |
|
|
|
|
|
|
|
| |
This is more convenient for both the developer and the user, especially
when it comes to providing logs in a bug report. As for the loss in
terms of performance, it is minimal, and this is normally not an issue
where these functions are used.
|
|
|
|
|
| |
This simplifies the code, fixes leaks in `g_debug()` calls, and
GLib 2.56 will be required when moving to Xfce 4.18.
|
|
|
|
|
|
|
| |
If their priority is equal to that of another plugin, there is no way to
know in advance which one will be prioritized in case of a mime type
match (more precisely, it depends on the order in which
`g_dir_read_name()` goes through the plugin directory).
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 8f0234a4a14bfec9f0a1ef5c1c8b94fb523f5d41.
There was no particular justification for this change, and since then
there have been several interventions in bug reports or on the forum
from users wondering why Tumbler does not generate thumbnails for
certain files, especially video.
There should not be a default size limitation, or only for a particular
plugin, with a justification in the comments, and only if the problem
cannot be fixed in the code.
|
|
|
|
| |
Fixes #22.
|
|
|
|
| |
Completes d6aeb7d65acf6e41738c78b9d137fab3502d1c58.
|
| |
|
|
|
|
|
|
|
|
|
| |
* Preserve the directory order in $XDG_DATA_DIRS and the priority of
$XDG_DATA_HOME over $XDG_DATA_DIRS for desktop thumbnailers.
* Fix some logical errors in `tumbler_registry_compare()`.
* Keep the priority of the pixbuf thumbnailer plugin higher than that of
the desktop thumbnailer plugin by default in `tumbler.rc` (it was
lowered in 6935184665902097511db1708eaaf78015ff61ac).
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This allows to fully use the capabilities of the specific pixbuf loader
for the considered mime type according to the installed optional
libraries. This is especially necessary to load some SVG or HEIF
thumbnails.
On the other hand, if this specific loader doesn't work properly, it may
be even more noticeable. This is the case for libopenraw, for which a
dedicated plugin using another API has fortunately been implemented. So
we just have to prioritize it in `tumbler.rc`.
Fixes #20, #32, #36, see !18 for more details.
Co-authored-by: @Krifa75 (Gitlab ID)
|
| |
|
| |
|
|
|
|
|
|
|
| |
Related:
https://gitlab.xfce.org/xfce/libxfce4util/-/merge_requests/18
https://gitlab.xfce.org/xfce/thunar/-/merge_requests/126
https://gitlab.xfce.org/xfce/thunar/-/issues/262
|
|
|
|
|
|
| |
Convert the callback into a real GSourceFunc.
Signed-off-by: Olaf Hering <olaf@aepfle.de>
|
|
|
| |
Current glib minimum is 2.50.0
|
|\ |
|
| | |
|
|/ |
|
|\ |
|
| | |
|
| | |
|
|/ |
|
|
|
|
|
|
|
| |
As the D-Bus data type "u" indicates 32 bit unsigned integer, explicily
use guint32 instead of guint.
See Bug #16833 for more details.
|
| |
|
| |
|
|
|
|
|
| |
Any path found in a plugin Excludes will be ignored,
code contributed by Markus Kolb xfce@tower-net.de Bug #16130.
|
| |
|
| |
|
|
|
|
| |
2 Gb for video files and 200 Mb for photo/pdf files.
|
|
|
|
| |
Move the sparse video files check to ffmpeg and gstreamer plugins.
|
|
|
|
| |
This commit fixes #14693 to allow thumbnailing of folders.
|
|
|
|
| |
Signed-off-by: Ali Abdallah <ali@xfce.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Sparse files, especially sparse video files, cause high cpu and memory
load when trying to get a thumbnail image out of them, using gst or ffmpeg
plugins.
For non sparse files, usually ((double)S_BLKSIZE * sb.st_blocks / sb.st_size) = 1.0,
but this is not always true, for example on file systems that support
compression, the ratio can be lower (ex. on ZFS), but not by too much for
already compressed video and image files.
We assume video files with ratio lower than 0.8 to be sparse and we avoid
them. Video files with ratio higher than 0.8 usually have enough valid
video frames, so getting a thumbnail out of them always succeeds. This
commit should fix bug #12259.
|
| |
|
|
|
|
| |
be consumed by the gdbus signal emission.
|