- Oct 01, 2018
-
-
Yuri Benditovich authored
SpiceUsbDevice now presents SpiceUsbBackendDevice. Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
-
Yuri Benditovich authored
Use SpiceUsbBackendDevice instead. Note that usb device manager uses the same context that win-usb-dev (handling events must be using the same libusb context that used for getting usb devices list. Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
-
- Sep 29, 2018
-
-
Yuri Benditovich authored
Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
-
Yuri Benditovich authored
This layer communicates with libusb and libusbredir and provides the API for USB redirection procedures. All the modules of spice-gtk communicate only with usb backend instead of calling libusb and usbredirhost directly. This is prerequisite of further implementation of cd-sharing via USB redirection. Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
-
Yuri Benditovich authored
Currently mingw build of qmp-helper fails under mingw due to undefined memmem function. Disabling it for mingw till better solution. Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
-
- Sep 28, 2018
-
-
Frediano Ziglio authored
SPICE version 2 was introduced more than 8 years ago. RHEL 6 already removed support for version 1 in the server. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
- Sep 27, 2018
-
-
Eduardo Lima (Etrunko) authored
Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
-
Eduardo Lima (Etrunko) authored
Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
-
Eduardo Lima (Etrunko) authored
Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
Eduardo Lima (Etrunko) authored
This new version ships the fix for the issue where 'check' keyword could not be used in project definition, so we had to run that command again only to check if it succeeded. https://github.com/mesonbuild/meson/issues/3944 Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
Marc-André Lureau authored
g_hash_table_remove() will call the free handler and dispose the pending tasks. Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
Marc-André Lureau authored
Do not leak the strong channel ref. Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
Marc-André Lureau authored
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
Marc-André Lureau authored
There is no size argument for SpicePortChannel::port-event signal. Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
Frediano Ziglio authored
Unused since commit 85ed2684. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Marc-André Lureau <marcandre.lureau@gmail.com>
-
Frediano Ziglio authored
Unused since commit ab33cf5f. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Marc-André Lureau <marcandre.lureau@gmail.com>
-
- Sep 26, 2018
-
-
Frediano Ziglio authored
Not much sense to reimplement a list using Ring type. This patch maintains the stability of the iterators and the order of the items in the container. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Marc-André Lureau <marcandre.lureau@gmail.com>
-
Victor Toso authored
Introduced in the last iteration (v4) by mistake, commit bd195d3f usb-device-manager.c: In function ‘channel_event’: usb-device-manager.c:900:19: error: format ‘%d’ expects argument of type ‘int’, but argument 4 has type ‘unsigned int’ [-Werror=format=] g_warning("Unhandled SpiceChannelEvent %d, disconnecting usbredir %p", event, channel); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~ /usr/include/glib-2.0/glib/gmessages.h:339:32: note: in definition of macro ‘g_warning’ __VA_ARGS__) ^~~~~~~~~~~ Also removes extra space. Signed-off-by: Victor Toso <victortoso@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
-
Victor Toso authored
Otherwise spice-channel is left with a broken state. This code moves parent's call to channel_reset() into _channel_reset_finish() - Note that spice-channel's channel_reset() can be called from GMainContext. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1625550 Signed-off-by: Victor Toso <victortoso@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
-
Victor Toso authored
This should not change code behavior, only add some comments. This is a preparatory patch for bug below, introduced with: commit 9fbf6794 Author: Kirill Moizik <kmoizik@redhat.com> Date: Tue Mar 8 16:05:57 2016 +0200 usbredir: Disconnect USB device asynchronously Related: https://bugzilla.redhat.com/show_bug.cgi?id=1625550 Signed-off-by: Victor Toso <victortoso@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
-
Victor Toso authored
As we are not able to redirect anything in case that usbredir channel is not connected. Related: https://bugzilla.redhat.com/show_bug.cgi?id=1625550 Signed-off-by: Victor Toso <victortoso@gnome.org> Acked-by: Frediano Ziglio <fziglio@redhat.com>
-
Victor Toso authored
The channel_connect() function could fail leading to a spice-channel existing as zombie (its coroutine return soon after). Check if channel_connect() fails and give a proper error signal to user when that happens. Related: https://bugzilla.redhat.com/show_bug.cgi?id=1625550 Signed-off-by: Victor Toso <victortoso@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
-
- Sep 25, 2018
-
-
Christophe de Dinechin authored
Acked-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Lukáš Hrázký <lhrazky@redhat.com>
-
I broke gitlab CI in commit f61534dd by introducing json-glib dependency. Instead of depending on dnf builddep, list the build dependencies explicitly. Remove copr usage, that fails from time to time. Install spice-protocol from git. Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
- Sep 24, 2018
-
-
Marc-André Lureau authored
Add a few helper functions to deal with a QMP port channel, in order to ease json handling, and wrapping a few commands. (by convention, the port should have the name "org.qemu.monitor.qmp.0", but it's not strictly required) This helper is put into use in the virt-viewer "Add QEMU-like UI: VT console & basic VM status" series. Note: this adds a strong dependency on json-glib for spice-client-glib, a widely available and fairly small library. QMP specification is: https://git.qemu.org/?p=qemu.git;a=blob;f=docs/interop/qmp-spec.txt Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
- Sep 18, 2018
-
-
The scroll wheel on the mouse will trigger a smooth scroll event with a delta of +/-1.0. The code added in 2212f051 triggers a scroll when the delta is strictly greater than 1.0. This means that right after connecting a client, we won't be triggering a scroll with the first mouse wheel 'click'. This commit adjusts the check so that we try to scroll when the delta is equal or greater than 1.0. https://bugzilla.redhat.com/show_bug.cgi?id=1627823 Signed-off-by: Christophe Fergeau <cfergeau@redhat.com> Acked-by: Marc-André Lureau <marcandre.lureau@gmail.com>
-
- Sep 05, 2018
-
-
Frediano Ziglio authored
Meson waits for pipes (stdout and stderr) to be not readable but our mocking helper does not close these handles causing Meson to exit with a stack trace. Note that the fact that the helper is not closing the handles is a wanted behaviour. Already reported at https://github.com/mesonbuild/meson/issues/3967 . Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Eduardo Lima (Etrunko) <etrunko@redhat.com>
-
- Sep 04, 2018
-
-
No need to use it when we can actually specify the parameters in the actual gnome.generate_gir() function calls. We still keep it in the case of gnome.gtkdoc(), as there is no way to specify the libraries to link with. Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
-
Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
-
This function does accept a list of arguments, so there is no need to iterate over it. Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
-
- Sep 03, 2018
-
-
Frediano Ziglio authored
If gir files are not installed generate_vapi is not able to find generated gir files, so look also in src generated files. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Eduardo Lima (Etrunko) <etrunko@redhat.com>
-
Frediano Ziglio authored
File generated should be depends on currently files built, not from installed files, so use target objects for spice-gtk dependency on spice-lib. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Eduardo Lima (Etrunko) <etrunko@redhat.com>
-
- Aug 30, 2018
-
-
Eduardo Lima (Etrunko) authored
Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
Eduardo Lima (Etrunko) authored
In a comparison with current autotools build system, meson/ninja provides a huge improvement in build speed, while keeping the same functionalities currently available and being considered more user friendly. The new system coexists within the same repository with the current one, so we can do more extensive testing of its functionality before deciding if the old system can be removed, or for some reason, has to stay for good. - Meson: https://mesonbuild.com This is the equivalent of autogen/configure step in autotools. It generates the files that will be used by ninja to actually build the source code. The project has received lots of traction recently, with many GNOME projects willing to move to this new build system. The following wiki page has more details of the status of the many projects being ported: https://wiki.gnome.org/Initiatives/GnomeGoals/MesonPorting Meson has a python-like syntax, easy to read, and the documentation on the project is very complete, with a dedicated page on how to port from autotools, explaining how most common use cases can be implemented using meson. http://mesonbuild.com/Porting-from-autotools.html Other important sources of information: http://mesonbuild.com/howtox.html http://mesonbuild.com/Syntax.html http://mesonbuild.com/Reference-manual.html - Ninja: https://ninja-build.org Ninja is the equivalent of make in an autotools setup, which actually builds the source code. It has being used by large and complex projects such as Google Chrome, Android and LLVM. There is not much to say about ninja (other than it is much faster than make) because we won't interact directly with it as much, as meson does the middle man job here. The reasoning for creating ninja in the first place is explained on the following post: http://neugierig.org/software/chromium/notes/2011/02/ninja.html Also its manual provides more in-depth information about the design principles: https://ninja-build.org/manual.html - Basic workflow: Meson package is available for most if not all distros, so, taking Fedora as an example, we only need to run: # dnf -y install meson ninja-build. With Meson, building in-tree is not possible at all, so we need to pass a directory as argument to meson where we want the build to be done. This has the advantage of creating builds with different options under the same parent directory, e.g.: $ meson ./build --prefix=/usr $ meson ./build-extra -Dextra-checks=true -Dalignment-checks=true After configuration is done, we call ninja to actually do the build. $ ninja -C ./build $ ninja -C ./build install Ninja defaults to parallel builds, and this can be changed with the -j flag. $ ninja -j 10 -C ./build - Hacking: * meson.build: Mandatory for the project root and usually found under each directory you want something to be built. * meson_options.txt: Options that can interfere with the result of the build. Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
- Aug 28, 2018
-
-
Eduardo Lima (Etrunko) authored
Ninja parallel build fails in certain environments, so it is required specify those files as a dependency so that vncdisplaykeymap.c will only be built after all files it depends on are generated. The problem is that, when using meson declare_dependency() function, only header files should be used as the "sources" keyword, as documented in http://mesonbuild.com/Wrap-best-practices-and-tips.html#declare-generated-headers-explicitly and http://mesonbuild.com/Wrap-best-practices-and-tips.html#avoid-exposing-compilable-source-files-in-declare_dependency Signed-off-by: Eduardo Lima (Etrunko) <etrunko@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com>
-
- Aug 27, 2018
-
-
Frediano Ziglio authored
Instead of keeping 2 fields for the conversion monotonic time -> multimedia time compute a single offset. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Uri Lublin <uril@redhat.com>
-
- Aug 22, 2018
-
-
Jorge Olmos authored
When building spice-gtk for windows: - libusb uses __stdcall calling convention when compiled for win32. It does not include an option to be compiled with __cdecl calling convention. Directly calling libusb functions works fine. But it is a problem when its functions are passed as callbacks to a function that expects other calling convention. - glib uses __cdecl calling convention and expects the functions it receives as parameters to follow __cdecl convention. So the lines included in spice-gtk like: g_clear_pointer(&priv->device, libusb_unref_device); cause libusb_unref_device (compiled with _stdcall convention) to be called with __cdecl convention. This causes stack corruption, and hence crashes. Related: https://gitlab.gnome.org/GNOME/glib/issues/1494 Acked-by: Victor Toso <victortoso@redhat.com>
-
- Aug 16, 2018
-
-
Frediano Ziglio authored
This brings in the following changes: Eduardo Lima (Etrunko) (5): ci: Fix typo: celt501 -> celt051 meson: Make options accessible through parent project meson: Fix checking for python Meson: Make use of dictionary type introduced in version 0.47 Meson: Make use of 'feature' option type introduced in version 0.47 Frediano Ziglio (2): ci: Fix Meson feature option Fix flexible array buffer overflow Lukáš Hrázký (1): spice*.proto: Replace tabs with the appropriate number of spaces Acked-by: Eduardo Lima (Etrunko) <etrunko@redhat.com>
-
- Aug 14, 2018
-
-
This patch configures libusb in the context used in win-usb-dev, with the same backend as the context initialized in usb-device-manager. Otherwise spice_usb_device_manager_libdev_match returns false for all devices, and no usb device is offered to end user for redirection. Acked-by: Christophe Fergeau <cfergeau@redhat.com>
-
- Aug 09, 2018
-
-
wangfan authored
In libusb 1.0.21, UsbDk is enabled at compile time with --enable-usbdk option. In libusb 1.0.22, the backend is selected at runtime. This patch avoids a crash when spice client is compiled against libusb 1.0.22 and UsbDk is not installed. * Enabling dynamic backend selection in Windows https://github.com/libusb/libusb/commit/54884e84d024e761450287ab56aca spice/spice-gtk#74 Signed-off-by: Victor Toso <victortoso@redhat.com> Acked-by: Christophe Fergeau <cfergeau@redhat.com>
-