There's a pretty good chance that there were some changes in
evolution-data-server since `1.13`, so let's bump it to a reasonable
minimum by setting it to the one in Debian stable (which is 3.30)

We started requiring at least version 2.52 when we started using
`Uuid.string_random()` (thanks @ricotz for pointing this out).

Let's just immediately bump it to the currently supported version of
Debian stable, which is 2.58.

@ricotz helpfully pointed out that we don't need the
`--disable-since-check`, since we don't even compile on valac 0.38.10.

Let's remove the check for the option, and instead have a check at the
beginning that people aren't trying to compile Contcts on a too old
Vala.

* Remove `using` for all namespaces (except Folks)
* Rename some variables
* Add `this.` to instance variables

Whoopsydaisy

Just use `dialog.run()`

This gives us a consistent style, which allows us to install an
automatic review step for style consistency using a formatter.

Only elevate Folks to the primary namespace.

The current code that created a new window was a mess, and didn't really
say what or why we were doing things. This commit cleans up some of this
by renaming some variables and functions and making sure we always show
a window to the user, so that we can provide useful feedback.

... and fix the minimal height obscuring most of the content on
non-small screens.

Fixes https://gitlab.gnome.org/GNOME/gnome-contacts/-/issues/119

* Properly catch an error when unlinking
* Removed unused `using Gtk` statement
* Prefix instance variables with `this.`
* Don't use `error()` which will abort the application

* Fix warning
* Remove unnecessary `using` statements
* Add `this.` prefix for instance variables

We've had compilation errors thanks to that before, when libhandy
introduced a similar name as that of Gtk (e.g. HdyHeaderBar vs
GtkHeaderBar). It also makes grepping a bit easier for maintainers :-)

This shouldn't have been merged in, and is now giving warnings. Let's
remove it for now, and if somebody still needs it, they can use the Git
history for that.

We've had compilation errors thanks to that before, when libhandy
introduced a similar name as that of Gtk (e.g. HdyHeaderBar vs
GtkHeaderBar). It also makes grepping a bit easier for maintainers :-)

Fixes https://gitlab.gnome.org/GNOME/gnome-contacts/-/issues/170

For certain accelerators (like `<Primary>`), GTK needs to be able to
check the `GdkDisplay` under which it is running. Since this first needs
`gtk_init()` to be called, we need to wait until the
`GLib.Application.activate()` signal, where this call is done for us.

Fixes https://gitlab.gnome.org/GNOME/gnome-contacts/-/issues/169

`set` is also a keyword in Vala, which means that in certain cases this
will fail to compile. Let's just make sure we don't use it.

Vala 0.48 got a bit more strict on dynamic casts, so let's fix those
warnings.

Vala 0.48 got a bit more strict on dynamic casts, so let's fix those
warnings.

Vala 0.48 got a bit more strict on dynamic casts, so let's fix those
warnings.

(cherry picked from commit 24a1660e)

The dependencies got renamed, so fix that.

libcanberra-gtk got removed in the latest Flatpak runtime (3.38), so we
can't build cheese as it hard depends on it. Luckily, we don't
hard-depend on cheese, so let's get rid of it (for now).

As an extra, this commit also makes Cheese an optional dependency,
making it a bit easier to build it at the first go (without installing
cheese-devel) and a shorter CI time.

canberra-gtk got dropped out of the flatpak 3.38 runtime, so this is
making our CI fail. We don't really need it anyway, so let's drop it.