summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* Update geometry when enabling/disabling displaysFabiano Fidêncio2015-03-161-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | _update_displays_geometry() must be called every time a display is enabled/disabled, avoiding gaps (when a display is disabled) or overlaps (when a display is enabled) between the monitors. This is what happens when we have 3 displays enabled (each one represented by: width x height + x position + y position) ... Display #0 Display #1 Display #2 +---------+ +----------+ +---------+ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | +---------+ +----------+ +---------+ (680x804+0+0) (504x804+680+0) (408x804+1184+0) Whether the Display #1 is disable, a message will be sent down to the vdagent, representing the new arrangement of the monitors: Display #0 Display #2 +---------+ +---------+ | | | | | | | | | | | | | | | | | | | | | | | | +---------+ +---------+ (680x804+0+0) (408x804+1184+0) However, taking a look on the x position, a gap can be identified as Display #0 starts at position (0,0) and has 680 pixels of width. But Display #1 only starts at position (1184, 0), leaving 504 pixels as a gap. The proper message, however, should represent the following arrangement ... Display #0 Display #2 +---------+ +---------+ | | | | | | | | | | | | | | | | | | | | | | | | +---------+ +---------+ (680x804+0+0) (408x804+680+0) ... avoiding then gaps and overlaps. Resolves: rhbz#1111425 https://bugzilla.redhat.com/show_bug.cgi?id=1111425
* Deal with NULL gport in virt_viewer_app_set_connect_info()Christophe Fergeau2015-03-131-1/+1
| | | | | | | | virt_viewer_app_set_connect_info() has a debug statement printing gport/gtlsport. It checks that gtlsport is not NULL before printing it, but makes no similar check for gport. Since it's possible to get a NULL gport when using ovirt:// after the previous commit, it's better to check it too.
* ovirt: Don't try to use invalid port numbersChristophe Fergeau2015-03-131-2/+6
| | | | | | | | If a remote oVirt VM don't specify a port/secure port number, we'd still try to pass it down to spice-gtk, which would then complain that 0 (the default value) isn't a valid port number. This commit make sure we filter out the default port/secure-port value and pass NULL to spice-gtk instead when we get these values.
* foreign-menu: Don't show empty foreign menu on secondary displaysChristophe Fergeau2015-03-132-1/+9
| | | | | | | | | | When using ovirt://, the foreign menu will only be shown in the primary window after getting notified about OvirtForeignMenu::files (ie when it managed to fetch some ISO files to show in the foreign menu). However, for secondary windows, the foreign menu will be added to the window even if there are no files to show. This commit makes sure we destroy the window foreign menu whenever it would be empty.
* POTFILES: Add virt-viewer-preferences.xmlFabiano Fidêncio2015-03-121-0/+1
| | | | | The file was introduced in commit 73b80ba99fb80140cadd07bbbf09a412bb9a0098
* spec: Add virt-viewer-preferences.xmlFabiano Fidêncio2015-03-122-0/+3
| | | | | The file was introduced in commit 73b80ba99fb80140cadd07bbbf09a412bb9a0098
* ovirt: Fail gracefully when hostname is missingChristophe Fergeau2015-03-121-0/+7
| | | | | | | When parsing info returned by oVirt REST API, the hostname should be present. However, I recently run remote-viewer against a buggy oVirt instance where the hostname was missing. This commit handles better this situation by displaying an error message and exiting.
* ovirt: Take into account SPICE proxyChristophe Fergeau2015-03-121-0/+4
| | | | | | | | | | | VMs managed by oVirt can be hidden behind a proxy. This commit allows remote-viewer to make use of this information when it's available A recent oVirt instance is needed so that it's available through the REST API, as well as libgovirt 0.3.3 or newer. With older oVirt/libgovirt versions, the worst that can happen is a runtime warning in the console, and an impossibility to connect to VMs behind a proxy, so this commit is not raising the minimum libgovirt requirement.
* Take --direct into consideration when checking if a guest is reachableFabiano Fidêncio2015-03-121-6/+8
| | | | | | | | | | When connecting to a remote host (using qemu+ssh://...) that has a virtual machine listening to "127.0.0.1", virt_viewer_is_reachable() must take --direct into account, otherwise it can end up connecting to a local virtual machine listening to "0.0.0.0" instead of returning that the guest is not reachable. Resolves: rhbz#1085216
* Add G_SOURCE_REMOVE to vir-glib-compatFabiano Fidêncio2015-03-111-0/+5
| | | | | G_SOURCE_REMOVE was introduced in GLib 2.32 and has its value defined as FALSE.
* Avoid 'Dereference of a null pointer'Fabiano Fidêncio2015-03-111-1/+1
| | | | Caught by Covscan.
* Enable share folder widgets if supported by sessionMarc-André Lureau2015-03-051-22/+43
|
* Add virt_viewer_session_can_share_folder()Marc-André Lureau2015-03-053-0/+22
| | | | | Functions name says it all, it is only implement for Spice, checking for webdav channel presence.
* Sync preferences widgets with session propertiesMarc-André Lureau2015-03-051-0/+41
|
* Show preferences dialogMarc-André Lureau2015-03-054-1/+46
| | | | Add a menu item Preferences under File and show the preferences dialog
* Add preferences dialog UI fileMarc-André Lureau2015-03-052-0/+135
|
* spice: enable/disable share folderMarc-André Lureau2015-03-051-0/+28
| | | | Connect/disconnect webdav channel to enable or disable sharing folder
* spice: sync share folder preferences with sessionMarc-André Lureau2015-03-041-0/+7
|
* Bump spice-gtk to 0.28Marc-André Lureau2015-03-041-1/+1
| | | | The following patches will only work with spice-gtk >= 0.28.
* Add a few session properties for share folderMarc-André Lureau2015-03-041-0/+59
| | | | See properties comments for details.
* misc: add a missing "static" for functionMarc-André Lureau2015-03-041-1/+1
|
* Switch over to use zanata for managing translationsDaniel P. Berrange2015-02-23115-14146/+5990
| | | | | | | | | | | | | | | Push new pot with cd po make virt-viewer.pot zanata push Pull new translations with cd po zanata pull Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
* Do not add "https://" and "/api" to oVirt URIPavel Grunt2015-02-181-3/+2
| | | | | | | | | It is deprecated since govirt 0.3.1 (and virt-viewer already depends on govirt 0.3.2). Silences: (remote-viewer:19420): libgovirt-WARNING **: Passing a full http:// or https:// URI to ovirt_proxy_new() is deprecated (remote-viewer:19420): libgovirt-WARNING **: Passing an URI ending in /api to ovirt_proxy_new() is deprecated
* Bump release to 3.0 for next dev cycleDaniel P. Berrange2015-01-121-1/+1
|
* Update NEWS for 2.0 releaseDaniel P. Berrange2015-01-121-0/+36
|
* wixl: use a versionized installation directoryMarc-André Lureau2015-01-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | It turned out that not only the current MSI broke the "component rule", but also that our files are not versionized correctly. Windows Installer applies some file versioning rules before replacing a file http://msdn.microsoft.com/en-us/library/aa368599%28v=vs.85%29.aspx Since msitools doesn't extract version from files and populate the Version field of the File table, it "usually" keep the current file installed. It's practically impossible to rely on version information from files (from a quick look, only 5% of the files are versionized and even less correctly, libgcrypt seems to do non-monotonic buildid for example) So the rule that applies when files are not versionized is to check the file hash, and the modified date. File hash was added recently in msitools, but doesn't apply when the installed file itself has a version. In order to solve the above problems, it's simpler to just have a different installation prefix. Windows Installer will see files with different component guid, and won't be checking any file update rule. I have verified the upgrade is working, not leaving any file behind and updating registry correctly with this solution. Until the files are correctly versionized, it looks like the only sensible thing to do. Furthermore, this make it simpler to have several versions installed in parallel later on (when we change productid)
* Update LINGUAS file from translation refreshDaniel P. Berrange2015-01-121-11/+2
|
* Tweak autobuild.sh to facilitate release buildsDaniel P. Berrange2015-01-121-4/+9
|
* Refresh translations from transifexDaniel P. Berrange2015-01-12117-15847/+19733
|
* Add deps on mingw glib-networking RPMs needed by libsoupDaniel P. Berrange2015-01-091-0/+2
|
* Update mingw RPM deps to refer to gstreamer1Daniel P. Berrange2015-01-091-4/+4
|
* Fix typo s/SPICE_GTK3_REQUIRED/SPICE_GTK_REQUIRED/Daniel P. Berrange2015-01-081-2/+2
| | | | There is no separate version constant for SPICE GTK2 vs GTK3
* virt-viewer: set transient parent for connection's error dialogFabiano Fidêncio2015-01-081-1/+3
| | | | | Commit c3d24f8b sets transient parent for the most part of the GtkDialogs, but seems like this one was forgotten.
* virt-viewer: Don't connect to localhost displays with qemu+tcp://Christophe Fergeau2015-01-061-0/+57
| | | | | | | | | | | When connecting to a remote libvirt instance, a VM may only be listening on localhost for SPICE/VNC connections. In such a situation, virt-viewer then tries to connect to localhost, which is not correct as this 'localhost' referred to the remote libvirt host it connected to. This commit adds a couple of tests on the libvirt URI used and the <graphics> listen address to error out in this situation. Resolves: rhbz#1108523
* Remove 'map' handler for VirtViewerDisplayJonathon Jongsma2015-01-061-19/+4
| | | | | | | | | | | | | | | | | | | | In order to solve several problems with sizing and resizing displays, a 'map' handler was added to VirtViewerDisplay. The first time the map handler runs, its queues a resize to attempt to ensure that the window gets created at its desired size. Subsequent map events generate a call to _make_resizable(), which was an attempt to ensure that the window was always 'shrinkable' on the Microsoft Windows platform. Recent testing suggests that this _make_resizable() is not actually necessary on Windows anymore, since it is possible to shrink the display even when this call is removed. In addition, the call to _queue_resize() is a bit of an indirect solution to the problem of ensuring the proper size at startup. What we really want is to guarantee that the very first size request negotiation returns the desired size rather than the minimum size. In order to do this, we've added a flag to determine whether we've ever received a size request, and if not, we return our desired size, even if 'dirty' is not set.
* spec: Update mingw in order to use gstreamer1Fabiano Fidêncio2015-01-051-1/+1
|
* nsis: use gstreamer-1.0 instead of gstreamer-0.10Victor Toso2015-01-051-62/+52
| | | | | Spice-gtk now is able to use gstreamer-1.0 for playback and record audio. This patch updates the installer to use the latest version.
* win: Use correct format string for intptr_tChristophe Fergeau2015-01-021-1/+2
| | | | | | | | | Using %d as a format-specifier for intptr_t causes a warning with mingw64: virt-viewer-events.c: In function 'virt_viewer_events_add_handle': virt-viewer-events.c:103:5: warning: format '%d' expects argument of type 'int', but argument 5 has type 'intptr_t' [-Wformat=] g_debug("Converted fd %d to handle %d", fd, _get_osfhandle(fd));
* spec: Update mingw in order to use ovirtFabiano Fidêncio2014-12-231-0/+4
|
* spec: Update mingw in order to use gtk3Fabiano Fidêncio2014-12-231-9/+10
|
* nsis: use gtk-vnc conditionallyFabiano Fidêncio2014-12-232-15/+21
| | | | Only include gtk-vnc as dep when it's explicitly done in the configure.
* nsis: use spice-gtk conditionallyFabiano Fidêncio2014-12-232-90/+107
| | | | Only include spice-gtk as dep when it's explicitly done in the configure.
* nsis: use libgovirt conditionallyFabiano Fidêncio2014-12-232-17/+26
| | | | Only include libgovirt as dep when it's explicitly done in the configure.
* nsis: add missing file needed to use virsh.exeFabiano Fidêncio2014-12-231-0/+2
|
* nsis: use libvirt conditionallyFabiano Fidêncio2014-12-232-27/+44
| | | | Only include libvirt as dep when it's explicitly done in the configure.
* msi: use gtk-vnc conditionallyFabiano Fidêncio2014-12-212-2/+17
| | | | Only include gtk-vnc as dep when it's explicitly done in the configure.
* msi: use spice-gtk conditionallyFabiano Fidêncio2014-12-212-2/+17
| | | | Only include spice-gtk as dep when it's explicitly done in the configure.
* msi: use libgovirt conditionallyFabiano Fidêncio2014-12-212-2/+13
| | | | Only include libgovirt as dep when it's explicitly done in the configure.
* msi: use libvirt conditionallyFabiano Fidêncio2014-12-212-2/+13
| | | | Only include libvirt as dep when it's explicitly done in the configure.
* cosmetic: use $(VAR) instead of ${VAR}Fabiano Fidêncio2014-12-211-1/+1
|