From unknown Thu Mar 28 12:23:22 2024 MIME-Version: 1.0 X-Mailer: MIME-tools 5.502 (Entity 5.502) X-Loop: owner@bugs.x2go.org From: owner@bugs.x2go.org (X2Go Bug Tracking System) Subject: Bug#911 closed by X2Go Release Manager (X2Go issue (in src:x2goclient) has been marked as closed) Message-ID: References: <20150728025008.7CC505DA8D@ymir.das-netzwerkteam.de> X-X2go-PR-Keywords: pending X-X2go-PR-Message: they-closed 911 X-X2go-PR-Package: x2goclient X-X2go-PR-Source: x2goclient Date: Tue, 28 Jul 2015 02:55:15 +0000 Content-Type: multipart/mixed; boundary="----------=_1438052115-11296-0" This is a multi-part message in MIME format... ------------=_1438052115-11296-0 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=utf-8 This is an automatic notification regarding your Bug report which was filed against the x2goclient package: #911: Linux desktop shortcuts fail due to missing space between "x2goclient= " and "--sessionid" It has been closed by X2Go Release Manager . Their explanation is attached below along with your original report. If this explanation is unsatisfactory and you have not received a better one in a separate message then please contact X2Go Release Manager <= git-admin@x2go.org> by replying to this email. --=20 X2Go Bug Tracking System Contact owner@bugs.x2go.org with problems ------------=_1438052115-11296-0 Content-Type: message/rfc822 Content-Disposition: inline Content-Transfer-Encoding: 7bit Received: (at control) by bugs.x2go.org; 28 Jul 2015 02:50:37 +0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on ymir.das-netzwerkteam.de X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,NO_RELAYS, URIBL_BLOCKED autolearn=ham version=3.3.2 Received: by ymir.das-netzwerkteam.de (Postfix, from userid 1005) id 7CC505DA8D; Tue, 28 Jul 2015 04:50:06 +0200 (CEST) From: X2Go Release Manager To: 911-submitter@bugs.x2go.org Cc: control@bugs.x2go.org, 911@bugs.x2go.org Subject: X2Go issue (in src:x2goclient) has been marked as closed Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit Message-Id: <20150728025008.7CC505DA8D@ymir.das-netzwerkteam.de> Date: Tue, 28 Jul 2015 04:50:06 +0200 (CEST) close #911 thanks Hello, we are very hopeful that X2Go issue #911 reported by you has been resolved in the new release (4.0.5.0) of the X2Go source project »src:x2goclient«. You can view the complete changelog entry of src:x2goclient (4.0.5.0) below, and you can use the following link to view all the code changes between this and the last release of src:x2goclient. http://code.x2go.org/gitweb?p=x2goclient.git;a=commitdiff;h=4caef96a30c62c24a2077fd003e0a5f238885b17;hp=301b58ced7f1591476af7a50d2c759deea9895d1 If you feel that the issue has not been resolved satisfyingly, feel free to reopen this bug report or submit a follow-up report with further observations described based on the new released version of src:x2goclient. Thanks a lot for contributing to X2Go!!! light+love X2Go Git Admin (on behalf of the sender of this mail) --- X2Go Component: src:x2goclient Version: 4.0.5.0-0x2go1 Status: RELEASE Date: Tue, 28 Jul 2015 04:48:20 +0200 Fixes: 696 882 901 909 911 Changes: x2goclient (4.0.5.0-0x2go1) RELEASED; urgency=low . [ Mihai Moldovan ] * New upstream release (4.0.5.0): - INSTALL: add more verbose instructions on how to build X2Go Client and friends. - onmainwindow.{cpp,h}: rename slotScDaemonOut() and slotScDaemonError() to slotScDaemonStdOut() and slotScDaemonStdErr(). I will need slotScDaemonError() later and the previous names were a bit of a misnomer. - x2goutils.{cpp,h}: add new show_RichText_Generic_MsgBox() main function designed to replace the show_RichText_WarningMsgBox() function. - x2goutils.{cpp,h}: make show_RichText_WarningMsgBox() use show_RichText_Generic_MsgBox (). - x2goutils.{cpp,h}: new show_RichText_ErrorMsgBox() function as a wrapper for show_RichText_Generic_MsgBox() with a critical icon. - onmainwindow.{cpp,h}: properly handle scdaemon errors with a message box and close the client. - onmainwindow.cpp: manipulate PATH and append /usr/lib/gnupg2/ to it when starting scdaemon. Fixes: #882. - x2goutils.h: add QMessageBox include, fixed compile error. - onmainwindow.cpp: fix typos leading to compile errors. - onmainwindow.cpp: add blocks to switch cases to fix a compile error. Also correctly append to informative_text on one occasion, instead of overwriting it. - x2goutils.cpp: correct typo to fix compile errors. - onmainwindow.cpp: drop a redundant variable and fix empty env check. - onmainwindow.cpp: add more newlines to scdaemon error output. Use trayQuit() method to really quit the client. - onmainwindow.cpp: add debugging output for new scdaemon PATH value. - onmainwindow.cpp: split up PATH value as printed in the error message box by a newline every 100 characters. Makes it look less ugly and fixes truncation issues. - sshprocess.cpp: export PATH variable when running commands. Otherwise, only the first one will respect the new PATH value. - sshprocess.cpp: wrap SSH command prelude and conclusion in double quotes. - sshprocess.cpp: wrap all remote commands with "bash" instead of the unspecific "sh". - onmainwindow.cpp: try to escape shell quotes correctly when using Kerberos/GSSApi when running the xmodmap keyboard sync command on OS X. - onmainwindow.cpp: also escape other shell quotes when using Kerberos/GSSApi. - onmainwindow.cpp: silence compiler warning by casting an int value to std::size_t. - onmainwindow.cpp: silence another compiler warning by not converting from string to int and back to string needlessly. Just use the string. No need for an OS X-specific variable anymore. - onmainwindow.cpp: fix compile error - use QString.toAscii () to get a valid C string representation for tr (). - onmainwindow.cpp: don't terminate if scdaemon exited with non-zero exit code. - sshprocess.cpp: don't use QProcess::start (QString). Qt is trying to be too smart and causes big trouble. Instead, use QProcess::start (QString, QStringList) and pass the arguments as a list. On Windows, Qt will automatically double quote the arguments and duplicate escaped double quotes or escape non-escaped double quotes. On UNIX-like platforms, each element of the list is passed as a unique argv element, so there's no need for quoting them (that's only a shell-internal thing to group arguments.) - sshprocess.cpp: add a bit more debugging - also print out the unmodified raw output of SSH commands. - onmainwindow.cpp: remove now-bogus double quote escaping. - sshmasterconnection.cpp: port QProcess::start () change. - sshprocess.cpp: whitespace/prettify only and a compile fix. - x2goclient.pro: reinstate old lrelease "search" behavior if qtPrepareTool() is not available. This should only happen on EPEL 6, which ships a broken Qt4 version... - onmainwindow.cpp: only set multidisplay mode for non-rootless sessions. - {ui/xsettingsui.ui,xsettingswidget.cpp}: add new "whole display" option for external X server configuration. Part of #883. - onmainwindow.cpp: rename "optionsmd" setting to "optionswholedisplay", which is more clear. - xsettingswidget.cpp: fix compile error: settings -> setting. - sessionmanagedialog.cpp: enable some debugging and make selected sessions configureable or deletable. Fixes: #909. . [ Mike Gabriel ] * debian/control: + Add alternative D for nxproxy: qvd-nxproxy. Allows parallel installation of X2Go Client and the TheQVD client (perl-qvd-client). . [ Mike DePaulo ] - Windows: Update bundled Win32 OpenSSL from 1.0.1m to 1.0.1p, which fixes the multiple CVEs announced on 2015-06-11 and CVE-2015-1793 (announced on 2015-07-09). - Windows: Upgrade the MSVC 2008 DLLs (9.0.21022.8) to the MSVC 2013 DLLs (12.0.21005.1), which Win32 OpenSSL now requires. - Windows: Only bundle msvcr120.dll; The other 2 MSVC DLLs aren't needed by Win32 OpenSSL. - Windows: Update bundled Cygwin OpenSSL from 1.0.2a-1 to 1.0.2d-1, which which fixes the multiple CVEs announced on 2015-06-11 and CVE-2015-1793 (announced on 2015-07-09). - Windows: Update bundled VcXsrv from 1.17.0.0-1 (X2Go/Arctica Build) to 1.17.0.0-3 (X2Go/Arctica Build) The differences relevant to X2Go are: + VcXsrv's bundled version of openssl was updated from 1.0.2a to 1.0.2d (fixes the multiple CVEs announced on 2015-06-11 and CVE-2015-1793 (announced on 2015-07-09). + Drop winmultiwindow.patch (Part of #883) + hw/xwin/glx: Don't create fbConfigs for un-accelerated pixelFormats This has the effect of disabling StaticColor under MultiWindow mode with -wgl (default) and -swrastwgl, and thus making X2Go sessions started from the Windows client able to be resumed on the client on a different OS or the client runing on Windows with different display settings. (Fixes: #696) - Linux: Fix desktop session icons (.desktop files) failing due to missing space between "x2goclient" and its 1st argument (e.g. "--session") (Fixes: #911) . [ Henning Heinold ] * New upstream release (4.0.5.0): - x2goclient.pro: simplify lrelease path preparing for different OS and cross compiling. Fixes: #901. . [ Oleksandr Shneyder ] * New upstream release (4.0.5.0): - onmainwindow.{cpp,h}: fix multimonitor mode on Windows. Run VcXsrv on selected screen without decorations. ------------=_1438052115-11296-0 Content-Type: message/rfc822 Content-Disposition: inline Content-Transfer-Encoding: 7bit Received: (at submit) by bugs.x2go.org; 28 Jul 2015 00:37:04 +0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on ymir.das-netzwerkteam.de X-Spam-Level: X-Spam-Status: No, score=0.8 required=5.0 tests=BAYES_50,FREEMAIL_FROM, T_DKIM_INVALID,URIBL_BLOCKED autolearn=ham version=3.3.2 Received: from mail-yk0-f178.google.com (mail-yk0-f178.google.com [209.85.160.178]) by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id 892F95DA83 for ; Tue, 28 Jul 2015 02:37:02 +0200 (CEST) Received: by ykay190 with SMTP id y190so83176539yka.3 for ; Mon, 27 Jul 2015 17:37:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:date:message-id:subject:from:to:content-type; bh=7WBGYRm7Xn8rcovx+AVDUNxLCGlZsUvx5vnU+CDyHPA=; b=MsyF7H6Hy2RHBX/kCch4Z1TAQSuaQoL0KLDDrioa0y3yIvZWgauXJIrq3nQwUD2ASr UcPO9pxMA+gxjGgLYZBSk353LLmxWkl1t89UNCDyPUcBVLMcFP7F6uNGf5T9FC+2IeGf D2YnA53enq/XN8jjmAF6SsimVHdDS+aaLD+5QmjtC/c2deG8c3OKh3oViEiNFBwvn+vl LXoqqre2QGhKPSL4Yit9rUBh5xlIBYmHQQSOkWBmS0bWQOxqUCKhmo7ANNnsjb/ycpSX ayNASvDrcuJS/bFi997IRVTV1UoPnxcoILKyIS8Q3IQCAGmaMOOhDcJWxiwUX6vvlfE9 giNA== MIME-Version: 1.0 X-Received: by 10.13.246.70 with SMTP id g67mr31838604ywf.75.1438043821326; Mon, 27 Jul 2015 17:37:01 -0700 (PDT) Received: by 10.37.215.147 with HTTP; Mon, 27 Jul 2015 17:37:01 -0700 (PDT) Date: Mon, 27 Jul 2015 20:37:01 -0400 Message-ID: Subject: Linux desktop shortcuts fail due to missing space between "x2goclient" and "--sessionid" From: Michael DePaulo To: "submit@bugs.x2go.org" Content-Type: text/plain; charset=UTF-8 Package: x2goclient Version: 4.0.4.0 As reported here: https://www.reddit.com/r/LinuxActionShow/comments/3dwa6j/get_going_with_x2go_las_374/ctbkfcq On Linux, but not on Windows, .desktop shortcuts are being created with Exec lines like: x2goclient--sessionid=1234567890123 Instead of: x2goclient --sessionid=1234567890123 Similarly, shortcuts like this are created: x2goclient--hide --sessionid=1234567890123 This appears to be a regression introduced by the fix for #871. I will commit a fix momentarily. ------------=_1438052115-11296-0--