| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
|
|
|
|
|
|
| |
This is a simple way to share a GrlConfig between multiple
applications. The environment is set by the user and it should not
conflict with applications cache if any.
|
|
|
|
|
|
|
|
|
|
|
|
| |
As pointed out by Carlos in the issue below, we are currently calling
va_arg (args, gint) to the user parameter, that translates to 0 which
we would happily take in Grilo as correct.
For numeric types, user can simply use G_MAXINT and friends to limit
lower/upper limits. Grilo will take care of clamping this if value if
it is outside boundaries of a specific metadata-key.
Resolves: https://gitlab.gnome.org/GNOME/grilo/-/issues/140
|
|
|
|
|
|
| |
At this point, only a simple unit test related to the previous commit
that fix GrlOperationOptions storing upper and lower limits that are
outside the boundaries of numeric types metadata-keys.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
GrlOperationOptions needs to validate user input against the maximum
and minimum values of a metadata-key of numeric type. For numeric
types, the follow should always be true:
@min_value_registered <= @min_value_user
@min_value_user <= @max_value_user
@max_value_user <= @max_value_registered.
So, we do implement some sort generic CLAMP for numeric types. Sadly,
glib does not provide as of yet a g_value_cmp() [0] so we need to
write a bit more code to make this checks in Grilo today.
The followup commit adds a unit test.
Related: https://gitlab.gnome.org/GNOME/grilo/-/issues/140
[0] https://gitlab.gnome.org/GNOME/glib/-/issues/129
|
| |
|
| |
|
|
|
|
|
|
|
| |
GNOME CI runners have been updated to the latest libseccomp so the
problem should no longer happen.
This reverts commit 5dfaa6cf0751f66fb3053d2705e9e03d18175e29.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Found with -Wformat :
In file included from ../src/grilo.h:31,
from ../libs/net/grl-net-wc.c:52:
../libs/net/grl-net-wc.c: In function 'get_url':
../libs/net/grl-net-wc.c:770:16: warning: format '%lu' expects argument
of type 'long unsigned int', but argument 5 has type 'gint64' {aka
'long long int'} [-Wformat=]
770 | GRL_DEBUG ("delaying web request by %lu seconds",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
771 | priv->last_request - now);
| ~~~~~~~~~~~~~~~~~~~~~~~~
| |
| gint64 {aka long long int}
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
That would allow us to add more early exits when debugging, without
making the code more complicated. Also tidy up to not have a "source"
variable that's actually a GList element.
|
| |
|
|
|
|
|
|
|
|
| |
Since commit adc11e54537aff804c6dcd8d26ade0114632157b, it's
impossible to use grilo-test-ui in the Flatpak for another
application, as it tries to request a D-Bus name that it cannot own.
Instead of using our own app-id/D-Bus name, try to own a variant of the
real app's name.
|
|
|
|
|
|
|
|
| |
If the application couldn't own a name on the session bus, then it would
never setup its UI, and view would be a NULL pointer which we'd
dereference, and crash.
Exit early from cleanup function if the view was never setup.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Rawhide is breaking the introspecting build with a permission to read
error.
ldd: error: you do not have read permission for `/builds/victortoso/grilo/_build/tmp-introspectba6itfp6/Grl-0.3'
That is a temporary folder created by meson (0.55.0-2) but I think the
issue is in the new python 3.9. I could not reproduce locally but for
now, let's fix CI for grilo and grilo-plugins (due subprojects)
|
|
|
|
|
|
|
|
|
| |
grl_registry_get_plugins() is supposed to return a GList of GrlPlugin
but sent back a GList of strings when one asked for all the plugins,
as it returned the keys instead of the values from the plugins
hashtable.
Closes: #141
|
|
|
|
|
|
|
|
|
|
|
|
| |
Latest meson is creating a .so.p temporary folder to lua-factory plugin
and current code in grl-registry would try to load all .so* files. This
patch makes sure that it only tries to load files with .so suffix
(actually, G_MODULE_SUFFIX that could be different in other platforms)
Fixes:
Bail out! Grilo-FATAL-WARNING: [registry]
../subprojects/grilo/src/grl-registry.c:1210: Failed to open module:
/builds/victortoso/grilo-plugins/_build/src/lua-factory/libgrlluafactory.so.p: cannot read file data: Is a directory
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It is the counterpart of grl_data_set_for_id() for GrlRelatedKeys (see
commit b30608abacd37a4e39d5c5465efc82c566756631).
grl_related_keys_set_for_id() allows keys that aren't registered,
to be registered and set it's value. It simply sets the value
for registered keys.
grl_registry_register_metadata_key_for_type() and
grl_registry_register_or_lookup_metadata_key() need to be updated to
add a new bind_key parameter. For related keys, bind_key parameters is
used to create a relation between the key and bind_key.
|
|
|
|
|
|
|
|
|
|
|
| |
grl_data_set_for_id() and grl_data_new_for_id() share some code to
register or lookup a key by its name.
This shared code is put in a new private function named
grl_registry_register_or_lookup_metadata_key().
grl_registry_register_or_lookup_metadata_key() will be used in the
commit by the new grl_related_keys_set_for_id() function.
|
|
|
|
|
|
|
| |
On 3.22.0, gtk_show_uri has been deprecated in favor of
gtk_show_uri_on_window.
Add a GTK_CHECK_VERSION to prevent depending on a GTK version too
recent. It can be removed after bumping GTK to >= 3.22.0
|
|
|
|
|
| |
It has been deprecated since GTK 3.16. Use gdk_cursor_new_for_display
instead.
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
This patch makes usage of g_date_time_new_from_iso8601() from GLib
2.56 that returns GDateTime* from ISO 8601 formatted string.
Note that this API supported a little bit more than GLib's as a helper
while developing new plugins thus it makes sense to keep it around.
|
|
|
|
|
|
|
|
| |
Removes the following warning:
> ../libs/pls/grl-pls.c:50:27: warning: extra tokens at end of #ifndef
> directive
> 50 | #ifndef TOTEM_PL_IS_PARSER(x)
> | ^
|
|
|
|
|
|
| |
On 2.62.0 we can use new API and avoid GTimeVal usage in grl-pls.c. As
2.62 is a bit new, wrap the old code to avoid warnings on new builds.
It can be removed after bumping glib to >= 2.62.0
|
|
|
|
|
| |
We can easily replace it with timestamp in seconds for the throttling
feature.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Bumping to make it easier in keeping compatibility with recent APIs
without breaking stable distro releases.
Fedora 31 (last stable): 2.62
Debian 10 (last stable): 2.58
Considering that we just released Grilo 0.3.12, next release can still
be kept under current last stable distros releases.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
This fixes a reference counting bug in the bindings.
Closes: #107
|
| |
|
| |
|
| |
|
|
|
|
| |
Close #138
|
| |
|
| |
|
|
|
|
|
|
|
| |
This enables the library to be used when Grilo is embedded as a Meson
subproject.
See: https://gitlab.gnome.org/GNOME/grilo-plugins/merge_requests/68
|
|
|
|
|
|
|
|
| |
The value of meson.source_root() can't be trusted because if we are
embedded via subproject() into a larger project, the source root will
be that of the parent project.
Instead, use a variable to keep track of the toplevel source directory.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Recently, we started prefixing the values of all variables defined in pc files
with `prefix` meson variable. While this works even when `datadir` option is
absolute (supported by some meson forks), it leads to inflexible pc files even
when prefixing is not strictly necessary.
In this commit, we are constructing variables for pkg-config files separately
from paths used for installation, obtaining nice .pc files in the common case,
while still supporting unusual platforms like Nix.
|
| |
|
|
|
|
|
|
|
| |
The generated .pc have been kept essentially identical
to the original ones with the same variables.
To squash warnings the meson_version was also bumped
|