GitLab

When the Python object is eventually destructed, the code forces a
disconnect.

However, if it is not actually connected, that would cause an exception.

To avoid that issue, simply force a disconnect only if we're actually
connected in the first place.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

If we add a delay, there is a possibility that the timeout may still
trigger during that additional delay, so better cancel the timeout.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Typically, gnome-shell will notify of the new app before its window is
actually placed which can cause the pointer to be misplaced if the first
generated event is a pointer move.

Add a half-second delay before returning from `waitFor` to give mutter
and gnome-shell some more time to place the place the window on screen.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Mention the new API addition in the README.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Add the waitFor() API

See merge request !1

Instead of the sleep().
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Add a new `waitFor()` API which takes care of waiting for a window to be
mapped and focused on screen.

After launching a new program, the caller can use this to wait for the
corresponding window to show up on screen and receive input focus,
rather than using `sleep()` and the like that are not guaranteed to work
depending on the time the client takes.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

There is no point in sending keyboard events to a window if it doesn't
have keyboard focus, because the actual key events will end up in
another window instead.

Add an optional `want_focus` parameter to `getWindowId()` which, if set
to `True`i, will check that the window also has focus.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Having the "introspect" API in gnome-shell gives us access to the
application id, make use of it.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Use 'app-id', 'title' and 'wm-class' as search criteria in the window
list.

The 'title' and 'wm-class' are optional properties (not all windows have
them) but the "app-id" is always there.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

gsettings key "app-introspection" renamed as just "introspect" in
gnome-shell branch for introspect.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

 * Introspect D-BUS API has moved from mutter to gnome-shell.
 * The patches for mutter have landed.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

The Introspect API is moved to gnome-shell instead of mutter, update the
python module to reflect that change.

Fix provided by: Vitezslav Humpa <vhumpa@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Adjust to the latest changes in mutter, and mention the new "Ponytail"
Python module.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

WindowList is now renamed "Introspect" and "GetList" method becomes
"GetWindows".
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

To provide the macro "%{python3_sitelib}" that we use in the spec file.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Those are in Python and rely on the "ponytail" Python module.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Implements the D-BUS interactions with gnome-ponytail-daemon interfaces
in python.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

That will start a gnome-calculator, do some operations and close the
window.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

We don't use it, no need to keep it around.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

If we use the same D-BUS name for the application and the activation
name, i.e. “org.gnome.Ponytail”, the EventController interface, i.e.
“org.gnome.Ponytail.EventController” will not be found at startup,
defeating the D-BUS activation mechanism.

Use a different name for the application (“org.gnome.PonytailDaemon”)
and explicitly own the “org.gnome.Ponytail” only once we export the
interface “org.gnome.Ponytail.EventController”.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

When started from D-BUS service, we need to wait for the screen-cast
session proxy acquisition before processing any Ponytail requests,
otherwise the first connection request will fail.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

The session might be started but the screen-cast proxy might take longer
to acquire.

During that time, we cannot precess ConnectWindow and ConnectMonitor()
requests.

Add a new signal "acquired" to notify when the screen-cast proxy is
actually acquired.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Install a D-BUS sevice and its associated systemd user service file to
start gnome-ponytail-daemon when requested.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Keep the code consistent, somehow.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Users may terminate the remote desktop session using the gnome-shell
icon.

If that occurs, restart the session so we can connect again (instead of
keeping a NULL proxy around).
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Add building steps, dependencies, credit and more importantly examples
on how to test.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

XKB adds a fixed offset of 8 to the evdev keycodes, remove that value
from the given keycode for evdev so that we take XKB keycodes directly.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

We do not actually need pipewire as we don't create nor connect to the
pipewire stream, so drop the unneeded dependency on pipewire.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>