summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* Fix warning when compiling without spice-gtk supportChristophe Fergeau2013-01-241-0/+1
| | | | | | | | | | | When remote-viewer is compiled without spice-gtk support, spice-session.h will not get included in remote-viewer.c, causing these warnings: remote-viewer.c: In function 'remote_viewer_start': remote-viewer.c:693:9: warning: implicit declaration of function 'virt_viewer_session_set_file' [-Wimplicit-function-declaration] remote-viewer.c:693:9: warning: nested extern declaration of 'virt_viewer_session_set_file' [-Wnested-externs]
* Add missing intltool BuildRequiresChristophe Fergeau2013-01-241-0/+1
| | | | The configure script fails if intltool is not installed.
* mingw: Add packages for virt-viewer NSIS installerChristophe Fergeau2013-01-241-1/+51
| | | | | | This makes it much easier to build an updated Windows installer binary as this can now be done using mock/koji/... by using this .spec and a virt-viewer tarball.
* mingw: Build a gtk2 version of virt-viewerChristophe Fergeau2013-01-241-5/+5
| | | | | | | The nsis file we ship is generating an installer for a GTK+2 build of virt-viewer, so it's inconsistent for the mingw-virt-viewer spec file to generate a GTK+3 build. Switch to building a GTK+2 version of virt-viewer in mingw-virt-viewer.spec
* mingw: Package debug-helper.exeChristophe Fergeau2013-01-241-0/+2
|
* mingw: Package debug filesChristophe Fergeau2013-01-241-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When building on f18, the build fails because of unpackaged debug files. Use the appropriate mingw macro to generate the mingw debug packages. The build failure is: RPM build errors: error: Installed (but unpackaged) file(s) found: /usr/i686-w64-mingw32/sys-root/mingw/bin/debug-helper.exe.debug /usr/i686-w64-mingw32/sys-root/mingw/bin/remote-viewer.exe.debug /usr/i686-w64-mingw32/sys-root/mingw/bin/virt-viewer.exe.debug /usr/i686-w64-mingw32/sys-root/mingw/bin/windows-cmdline-wrapper.exe.debug /usr/x86_64-w64-mingw32/sys-root/mingw/bin/debug-helper.exe.debug /usr/x86_64-w64-mingw32/sys-root/mingw/bin/remote-viewer.exe.debug /usr/x86_64-w64-mingw32/sys-root/mingw/bin/virt-viewer.exe.debug /usr/x86_64-w64-mingw32/sys-root/mingw/bin/windows-cmdline-wrapper.exe.debug Installed (but unpackaged) file(s) found: /usr/i686-w64-mingw32/sys-root/mingw/bin/debug-helper.exe.debug /usr/i686-w64-mingw32/sys-root/mingw/bin/remote-viewer.exe.debug /usr/i686-w64-mingw32/sys-root/mingw/bin/virt-viewer.exe.debug /usr/i686-w64-mingw32/sys-root/mingw/bin/windows-cmdline-wrapper.exe.debug /usr/x86_64-w64-mingw32/sys-root/mingw/bin/debug-helper.exe.debug /usr/x86_64-w64-mingw32/sys-root/mingw/bin/remote-viewer.exe.debug /usr/x86_64-w64-mingw32/sys-root/mingw/bin/virt-viewer.exe.debug /usr/x86_64-w64-mingw32/sys-root/mingw/bin/windows-cmdline-wrapper.exe.debug
* mingw: Fix typo in mingw macro name (hedder -> header)Christophe Fergeau2013-01-241-1/+1
|
* nsis: Adjust file listChristophe Fergeau2013-01-241-4/+4
| | | | Adjust file names for mingw-usbredir dlls and readline dlls
* virt-viewer-display-spice: Get monitor under our windowHans de Goede2013-01-221-1/+2
| | | | | | | | | | | When getting monitor info for going fullscreen, Get the monitor under *our* window rather then under the root-window. Noticed this not working properly when testing the monitor coordinates stuff, but this should also help people seeing problems when using non equally sized monitors. Signed-off-by: Hans de Goede <hdegoede@redhat.com>
* vnc: add connection by fileMarc-André Lureau2013-01-185-22/+52
| | | | | | | | | | | | | | | Learn to connect to a VNC server with the connection details file, ex: [virt-viewer] type=vnc host=localhost port=2356 password=foobar v2: - add username/password support https://bugzilla.redhat.com/show_bug.cgi?id=843410
* file: add username supportMarc-André Lureau2013-01-182-0/+27
|
* Make hotkey configuration functionality available from the cmdline (v2)Hans de Goede2012-12-214-0/+36
| | | | | | | Changes in v2: -Add --hotkeys documentation to the man-pages Signed-off-by: Hans de Goede <hdegoede@redhat.com>
* Add a virt_viewer_app_set_hotkeys() helper functionHans de Goede2012-12-213-40/+55
| | | | Signed-off-by: Hans de Goede <hdegoede@redhat.com>
* Improve check for overridden grab key combinationChristophe Fergeau2012-12-202-4/+5
| | | | | | | | | remote-viewer can either use the default grab/ungrab handled by spice-gtk, or override it and use the standard gtk+ accelerator mechanism. However, the code currently assumes that if any accelerator is set in remote-viewer, then the grab key has been overridden. This commit makes sure the grab key is actually overridden before assuming so.
* Don't leak SpiceGrabSequence in enable_accel_changedChristophe Fergeau2012-12-201-2/+3
|
* Improve hotkeys controller behaviourChristophe Fergeau2012-12-201-0/+5
| | | | | | | Disable default accelerators when setting bindings from the controller in case the controller does not override them all. This ensures we don't inherit from the bindings set in VirtViewerApp::constructor if the controller doesn't set any bindings for a given action.
* Use monitor geometry, not screen sizeMarc-André Lureau2012-12-071-2/+6
| | | | | | | | | | | | | In a recent commit, 3bb6f5ec805ecfe78eba6d4d98e3ffcab195273a, I introduced a regression: going fullscreen would no longer match client and guest resolution correctly. A GdkScreen is not necessarily the physical screen monitor size. Lookup the physical monitor size using gdk_screen_get_monitor_geometry(). Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=881020
* session-spice: Cast CA string to gunit8 pointerMichal Privoznik2012-12-051-1/+1
| | | | | | | SpiceSession has 'ca' property which is type of GByteArray*. However, when we read the property from file, we read it as string. For conversion g_byte_array_new_take() is used which takes given pointer as guint8* so we need to do the cast.
* Sanitize syntax-checkMichal Privoznik2012-12-052-1/+1
| | | | | | | | make syntax-check is producing some errors about empty line at EOF and missing #include <config.h> in src/virt-viewer-file.c * src/virt-viewer-file.c: add #include <config.h> * data/virt-viewer-debug.nsis.in: remove empty line at EOF
* Make .desktop file comply with specDoug Goldstein2012-12-052-2/+3
| | | | | | | The .desktop file did not comply with the Desktop Entry spec as checked with desktop-file-validate. Boolean keys are defined as taking only 'true' or 'false', the entry Terminal had False. MimeType is a string list and therefore must be terminated with a ;
* remote-viewer: learn to connect from fileMarc-André Lureau2012-11-271-2/+22
| | | | | v2: - move some variables to inner-block as requested by reviewer
* spice: learn to connect from fileMarc-André Lureau2012-11-271-1/+90
|
* Add VirtViewerSession:file propertyMarc-André Lureau2012-11-273-0/+43
| | | | | | | | | | If VirtViewerSession:file is set, it should be used to define the connection parameters. Also correct the mime type used in this case. The mime type is needed to identify the kind of resources we are adding to the recent list. The recent list can then be filtered and various application handling that type may attempt to access that resource.
* Add VirtViewerFileMarc-André Lureau2012-11-273-0/+900
| | | | | | | v2: - use !! for boolean values setter - switch from bytearray/base64 to plain string for CA (PEM) - add file format comment
* Move spice_hotkey_to_gtk_accelerator() to utilMarc-André Lureau2012-11-273-114/+116
| | | | | To be compatible with RHEVM, VirtViewerFile hotkey format will use the Spice Controller format.
* recent: get mime type from sessionMarc-André Lureau2012-11-271-5/+4
| | | | | Do not use spice mime type for all kind of sessions, but depending on what is actually being used.
* session: add virt_viewer_session_mime_type()Marc-André Lureau2012-11-274-0/+27
|
* debug.nsis: remove superflous pathMarc-André Lureau2012-11-271-1/+1
|
* display: only un-constrain display size once it is mappedMarc-André Lureau2012-11-271-1/+2
| | | | | | Wait until the widget is actually on screen before removing its size constrain. This solves 50x50 window secondary window size when connecting to a multi-monitor spice guest.
* Add virt-viewer-debug.exe NSIS installerMarc-André Lureau2012-11-233-0/+52
| | | | | | This installer will provide with the tools and configuration needed to debug virt-viewer & remote-viewer. It will install itself by default in virt-viewer directory.
* debug-helper: include <config.h>Michal Privoznik2012-11-211-0/+1
|
* AUTHORS: add Dave AllanMichal Privoznik2012-11-211-0/+1
| | | | as he has supplied patch (commit 74b1b62510d9390).
* cfg.mk: Don't require bindtextdomain directlyMichal Privoznik2012-11-211-2/+2
| | | | | | | | | With current implementation, all binaries that need it call bindtextdomain but not directly. They call a helper function instead. This makes, however, syntax-check fail as it cannot recognize it. Original patch proposed by Eric Blake <eblake@redhat.com>
* spice: the session is connected when main channel openedMarc-André Lureau2012-11-211-2/+1
| | | | | | | The previous change in 399aae55aa384bf91dff0fc770497c0d5f935fa9 rely on correct session-connected signal. However, the spice backend is emiting it too early, when the main channel is created, where it should wait until it is connected instead.
* "Unable to connect to the graphic server" error on guest shutdownMarc-André Lureau2012-11-201-4/+6
| | | | | | | | | | In virt_viewer_app_activate(), priv->connected is set to FALSE when the connect/active is successfull. However, we rely on it to know whether the virt_viewer_app_disconnected() is an error, so only set it to FALSE when connection failed. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=875697
* build-sys: add min libvirt version requirement checkMarc-André Lureau2012-11-161-1/+1
|
* Make sure png screenshots have a .png extensionsChristophe Fergeau2012-11-151-2/+10
| | | | | | When falling back to saving to .png, the filename might not end with .png. This commit appends the .png extension to the screenshot filename if it's missing.
* Allow to save to other formats than pngChristophe Fergeau2012-11-151-2/+53
| | | | | | Currently, the screenshots can only be saved to png. This commit checks if the file extension is a known one, and will save to this format if it is. Otherwise it will fallback to saving to png.
* Set a default dir/name for screenshotsChristophe Fergeau2012-11-151-3/+5
| | | | | | | It makes sense for the screenshots to be saved in ~/Images, especially as otherwise the filechooser will display 'recent documents' to which we cannot save. This commit also sets the default screenshot name to 'Screenshot'.
* Fix spice-gtk check in configure.acChristophe Fergeau2012-11-151-1/+1
| | | | | | | | The check that at least one of spice-gtk and gtk-vnc is present uses a wrong variable name to check for spice-gtk presence, which causes the check to think it's never present. This would make gtk-vnc presence mandatory. This commit fixes the name of the spice-gtk variable ($have_gtk_spice -> $have_spice_gtk).
* Don't SIGSEGV if no transport is used.Michal Privoznik2012-11-141-1/+1
| | | | | | | | One of previous commits (74b1b62510d939) allowed us to connect to localhost directly if ssh transport was used. However, if there's not transport, we SIGSEGV'ed as g_str_equal doesn't like NULL as one of arguments. Change this to g_strcmp0 which does the same service but is more friendly to NULL arguments.
* Reconnect to libvirtd after connection breaksMichal Privoznik2012-11-134-9/+57
| | | | | | | | Currently, if user wants to reconnect to a domain he can use '-r' cmd line argument. This makes virt-viewer listen to domain events. However, if connection to libvirtd breaks somehow, we will receive no longer any event. Hence we must reconnect to the libvirt.
* Connect to localhost for display when transport is sshDave Allan2012-11-081-3/+11
| | | | | | | | | | | When connecting to a VM that does not have a 'listen' tag in its graphcs element, we have to guess where to try to connect to the VM's display. The current default is the host specified in the connection URI which is correct for most transports, however, the SSH transport makes the display connection from the remote end, so in that case, attempt to connect to localhost. Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
* Fail if neither vnc nor spice is detectedGuido Günther2012-10-251-0/+4
| | | | Since the viewer makes little sense otherwise.
* Fix jenkins build failureMarc-André Lureau2012-10-231-0/+1
| | | | | | | | | virt-viewer-util.c: In function 'virt_viewer_util_init': virt-viewer-util.c:289: warning: implicit declaration of function 'setlocale' virt-viewer-util.c:289: warning: nested extern declaration of 'setlocale' virt-viewer-util.c:289: error: 'LC_ALL' undeclared (first use in this function) virt-viewer-util.c:289: error: (Each undeclared identifier is reported only once virt-viewer-util.c:289: error: for each function it appears in.)
* Use a mutex to check if VirtViewer runningMarc-André Lureau2012-10-232-0/+21
| | | | | | | We need to warn user that installer can't proceed if there is already a running instance of VirtViewer or of the installer. https://bugzilla.redhat.com/show_bug.cgi?id=864033
* Use a common early init() functionMarc-André Lureau2012-10-234-32/+36
| | | | | There is a number of things both virt-viewer and remote-viewer need to do early during execution. Do it only in one place.
* windows: add debug-helperMarc-André Lureau2012-10-232-0/+75
| | | | | | | | | | | | This is a simple program that will set some debug variable, and run gdb and wait until it finished. This makes it possible to debug "remote-viewer --spice-controller" easily, by setting the necessary variables and keeping the parent process running (the activex whatches its death) To use it, replace the HKCU "Software\spice-space.org\spicex\client" value "$INSTDIR\bin\remote-viewer.exe --spice-controller" with "$INSTDIR\bin\debug-helper.exe remote-viewer.exe --spice-controller".
* nsis: add deps.txt file in installerMarc-André Lureau2012-10-233-0/+24
| | | | | | This helps track package version that were used during the build of Windows installer. It's not ideal, but make up the lack of package management on windows
* nsis: update required files and librariesMarc-André Lureau2012-10-231-17/+35
|