From unknown Thu Mar 28 23:47:22 2024 X-Loop: owner@bugs.x2go.org Subject: Bug#1118: Share folder is broken for x2goclient-4.0.5.2-2016.09.20-setup.exe Reply-To: Michael , 1118@bugs.x2go.org Resent-From: Michael Resent-To: x2go-dev@lists.x2go.org Resent-CC: owner@bugs.x2go.org X-Loop: owner@bugs.x2go.org Resent-Date: Tue, 29 Nov 2016 05:00:02 +0000 Resent-Message-ID: Resent-Sender: owner@bugs.x2go.org X-X2Go-PR-Message: report 1118 X-X2Go-PR-Package: x2goclient-4.0.5.2-2016.09.20-setup X-X2Go-PR-Keywords: Received: via spool by submit@bugs.x2go.org id=B.14803953518800 (code B); Tue, 29 Nov 2016 05:00:02 +0000 Received: (at submit) by bugs.x2go.org; 29 Nov 2016 04:55:51 +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.1 required=3.0 tests=BAYES_40,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,HTML_MESSAGE autolearn=ham version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 1FD065DA85 for ; Tue, 29 Nov 2016 05:55:50 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id S5-4YLrYia5f for ; Tue, 29 Nov 2016 05:55:32 +0100 (CET) Received: from mail-ua0-f174.google.com (mail-ua0-f174.google.com [209.85.217.174]) by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id CDA7F5DA82 for ; Tue, 29 Nov 2016 05:55:31 +0100 (CET) Received: by mail-ua0-f174.google.com with SMTP id 20so166853825uak.0 for ; Mon, 28 Nov 2016 20:55:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:from:date:message-id:subject:to; bh=w0gaWwV/DGc149H/oSdibbqRQOepp70H4aHSMif9zzQ=; b=jmOukeY7rIE25ge8NPQfgo//qQJw+tdGEpk3gSZOeQkwtRGVhITrr9Th5I2bcA8QF6 2gleBpNUOq24rTVkZBVhc6z/SYCkrFKSCM3DCTSA0WjKLdbmIKDHdMKFZ7xvrd6iL0KK KzN0AyzZlGIBBux2JvxQQWY49oeWWj37O8qwaKb89KB3Ra9pivJsdBX3QhgpEtfjgJj2 qhZKom4WcPDOhRnnzp4pALlPT3fBrYzfzWe4KnArMZidSeq4121er0gIIBqORdsl/gO/ +/wmaOCH2aWqWNLGARrYJZ+Ke956Tytj1R1NZQDJ565y5jC+vvwmjMCXB48qiA4Dutb8 gOEg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:from:date:message-id:subject:to; bh=w0gaWwV/DGc149H/oSdibbqRQOepp70H4aHSMif9zzQ=; b=LLpJ2KzvEMSTPPJ9e5cv73YHfJlDQSHbGsG/mit0bkO1wJnaPl31nvZOauc0JR2kWL V0crUryDJBOgm9qDqMNKrsxpRTG2pwKaUrQt6S61GPa4plQ34Zu7r19OH0YndoZhstSD 4b+TFh1CXK9T9sjX1fQRWGSVEbF3h8rSxbXHQYIg6uscbrrSpPqn32LkBjTVgOddjoK1 FcYi6AykbT1GCjK0vcs36ZqiOu01v3jullsm/uxN42EjGdrrqn5IWHvsxboea/GK5P3a 13LWu8j+ROrjIAco9tvxWYMQrMHVogDeEuJ+CwTerQbLDxeV/YGoFpx1XbZdP8AjZNvv L+Ag== X-Gm-Message-State: AKaTC02JX2XKC0mRYenBJW48NNszC+GAAKt9Wf4svlSDrrrLiK6zeExHOBuY/nKdMGJ4byXIYLrfs5ajVXSXeA== X-Received: by 10.176.17.18 with SMTP id e18mr19455640uab.19.1480395330546; Mon, 28 Nov 2016 20:55:30 -0800 (PST) MIME-Version: 1.0 Received: by 10.176.83.156 with HTTP; Mon, 28 Nov 2016 20:55:10 -0800 (PST) From: Michael Date: Tue, 29 Nov 2016 15:55:10 +1100 Message-ID: To: submit@bugs.x2go.org Content-Type: multipart/alternative; boundary=f403045e3b6ecdf1860542696537 --f403045e3b6ecdf1860542696537 Content-Type: text/plain; charset=UTF-8 Package: x2goclient-4.0.5.2-2016.09.20-setup Version: x2goclient-4.0.5.2-2016.09.20-setup Steps to reproduce: 1. Install x2goclient-4.0.5.2-2016.09.20-setup.exe 2. Make sure C:\Users\XXXXXXX\.x2go\etc does not exist, or delete it if existed. 3. Create a new session to a x2go server. 4. Enable share folder and watch the bug report window. >From the source code: 9931 void ONMainWindow::startWinServers() 9932 { 9952 generateHostKey(RSA_KEY_TYPE); For Windows, we only generate RSA_KEY, but in 8013 QString ONMainWindow::createRSAKey() 8014 { 8043 #ifdef Q_OS_WIN 8044 rsa.setFileName ( 8045 wapiShortFileName ( 8046 homeDir+"\\.x2go\\etc\\ssh_host_dsa_key.pub" ) ); 8056 if ( !rsa.open ( QIODevice::ReadOnly | QIODevice::Text ) ) 8057 { We check DSA_KEY instead, which of course would not exist. To fix this bug: 8046 - homeDir+"\\.x2go\\etc\\ssh_host_dsa_key.pub" ) ); 8046 + homeDir+"\\.x2go\\etc\\ssh_host_rsa_key.pub" ) ); --f403045e3b6ecdf1860542696537 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
Package: =
x2goclient-4.0.5.2-2016.09.20-setup
Version: x2goclient-4.0.5.2-2016.09.20-setup

Steps t=
o reproduce:
1. Install x2goclient-4.0.5.2-2016.09.20-se=
tup.exe

2. Make sure C:\Users\XXXXXXX\.x2go\etc does not exist, or d= elete it if existed.
3. Create a new session to a x2go server=
.
4. Enable share folder and watch the bug report window.
=
From the source code:

9931 void ONMainWindow::startWi= nServers()
9932 {
9952 generateHostKey(RSA_KEY_TYPE);
For Windows, we only generate RSA_KEY, but in

8013 QString ONMainWindow::createRSAKey= ()
8014 {
8043 #ifdef Q_OS_WIN
8044 rsa.setFileName (
8045 = wapiShortFileName (
8046 homeDir+"\\.x2go\\etc\= \ssh_host_dsa_key.pub" = ) );
8056 if ( !rsa.open ( QIODevice::ReadOnly | QIODevice::Text ) )=
8057 {


We check DSA_KEY instead, which of course would n= ot exist.
To fix this bug:

8046=C2=A0=C2=A0 - =C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0 homeDir+"\\.x2go\\etc\\ssh_host_dsa_key.pub&q= uot; ) );
8046=C2=A0=C2=A0 + =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = homeDir+"\\.x2go\\etc\\ssh_host_rsa_key.pub" ) );
--f403045e3b6ecdf1860542696537-- From unknown Thu Mar 28 23:47:22 2024 X-Loop: owner@bugs.x2go.org Subject: Bug#1118: [X2Go-Dev] Share folder is broken for x2goclient-4.0.5.2-2016.09.20-setup.exe Reply-To: Mihai Moldovan , 1118@bugs.x2go.org Resent-From: Mihai Moldovan Resent-To: x2go-dev@lists.x2go.org Resent-CC: owner@bugs.x2go.org X-Loop: owner@bugs.x2go.org Resent-Date: Tue, 29 Nov 2016 17:50:01 +0000 Resent-Message-ID: Resent-Sender: owner@bugs.x2go.org X-X2Go-PR-Message: followup 1118 X-X2Go-PR-Package: x2goclient-4.0.5.2-2016.09.20-setup X-X2Go-PR-Keywords: Received: via spool by 1118-submit@bugs.x2go.org id=B1118.148044167428308 (code B ref 1118); Tue, 29 Nov 2016 17:50:01 +0000 Received: (at 1118) by bugs.x2go.org; 29 Nov 2016 17:47:54 +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=-2.0 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,SPF_HELO_PASS autolearn=ham version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 7DCDA3BC8A for <1118@bugs.x2go.org>; Tue, 29 Nov 2016 18:47:51 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id KJNACi15P-vl for <1118@bugs.x2go.org>; Tue, 29 Nov 2016 18:47:44 +0100 (CET) Received: from Root24.de (powered.by.root24.eu [5.135.3.88]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 8C4623BC7D for <1118@bugs.x2go.org>; Tue, 29 Nov 2016 18:47:44 +0100 (CET) Received: from nopileos.local (178.162.222.41.adsl.inet-telecom.org [178.162.222.41]) by mail.ionic.de (Postfix) with ESMTPSA id DA0954F00066; Tue, 29 Nov 2016 18:47:43 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=ionic.de; s=default; t=1480441664; bh=JnHK7Y0WwqDIWU8XAeM/RU+RmYImRnopnjZ1NUMHzuA=; h=Subject:To:References:From:Date:In-Reply-To:From; b=HJ51cUJ7cDt8tm+BAEIReUzUXF2zA+Eyr1PiMQqxycvkIniA2qOt+5TKwRKBzmCL6 Y5ehTYj7NFrs3BC5PIMvI/jGJrcd1rtgx1NIEG+OVkV7xLCwLh52ygi4WJskfwTsbs NxOalvnzi3CD7eWiRVdlG1z0PlFeh3C3deS25Z7w= To: Michael , 1118@bugs.x2go.org References: From: Mihai Moldovan Message-ID: <2a11eb7c-be89-f040-dba6-46d6bca48bf0@ionic.de> Date: Tue, 29 Nov 2016 18:47:41 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:45.0) Gecko/20100101 Thunderbird/45.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="D09c906F7ai3o6jS6JOUXjgm5NL42CW9O" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --D09c906F7ai3o6jS6JOUXjgm5NL42CW9O Content-Type: multipart/mixed; boundary="p2rrEdXNUnw5sU4TvBiwxWKdoob6llUXV"; protected-headers="v1" From: Mihai Moldovan To: Michael , 1118@bugs.x2go.org Message-ID: <2a11eb7c-be89-f040-dba6-46d6bca48bf0@ionic.de> Subject: Re: [X2Go-Dev] Share folder is broken for x2goclient-4.0.5.2-2016.09.20-setup.exe References: In-Reply-To: --p2rrEdXNUnw5sU4TvBiwxWKdoob6llUXV Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable Control: reassign -1 x2goclient 4.0.5.2 Control: tag -1 win32 On 29.11.2016 05:55 AM, Michael wrote: > Package: x2goclient-4.0.5.2-2016.09.20-setup > Version: x2goclient-4.0.5.2-2016.09.20-setup >=20 > Steps to reproduce: >=20 > 1. Install x2goclient-4.0.5.2-2016.09.20-setup.exe >=20 > 2. Make sure C:\Users\XXXXXXX\.x2go\etc does not exist, or delete it if= existed. >=20 > 3. Create a new session to a x2go server. >=20 > 4. Enable share folder and watch the bug report window. >=20 > From the source code: >=20 > 9931 void ONMainWindow::startWinServers() > 9932 { > 9952 generateHostKey(RSA_KEY_TYPE); >=20 > For Windows, we only generate RSA_KEY, but in >=20 > 8013 QString ONMainWindow::createRSAKey() > 8014 { > 8043 #ifdef Q_OS_WIN > 8044 rsa.setFileName ( > 8045 wapiShortFileName ( > 8046 homeDir+"\\.x2go\\etc\\ssh_host_dsa_key.pub" ) ); > 8056 if ( !rsa.open ( QIODevice::ReadOnly | QIODevice::Text ) ) > 8057 { >=20 >=20 > We check DSA_KEY instead, which of course would not exist. >=20 > To fix this bug: >=20 > 8046 - homeDir+"\\.x2go\\etc\\ssh_host_dsa_key.pub" ) ); > 8046 + homeDir+"\\.x2go\\etc\\ssh_host_rsa_key.pub" ) ); I'm aware of that bug, but probably won't fix it directly. That thing sho= uldn't come up with the new code in my bugfix/osx branch, since I've rewritten a= lmost all of the host key, client key (as used for the local [*not* remote] ssh= daemon) and ssh daemon handling. Changing portions there on the current m= aster branch would only lead to biggish conflicts while rebasing - that's not r= eally worth the effort. I'm rather concentrating on getting the other functiona= lity done and release-worthy instead, which will automatically fix this bug. Mihai --p2rrEdXNUnw5sU4TvBiwxWKdoob6llUXV-- --D09c906F7ai3o6jS6JOUXjgm5NL42CW9O Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBCgAGBQJYPb89AAoJEB/WLtluJTqHXV8P/3SD+rwaK4wYR9EBicavgJfH hUSiaYBPOYY5P8erLTKdJtxCNVR9NcnOCCwotTFM8qsYegiO7Y9dZGdF7FUAg+ul GwH3zf8MvygKuLt8mE9h6p/jWaqGjTFeY5cuNkQRhlmWcDf9gMGXCPp83PIN7V45 YNJiLZ5agg76BWsQDilXLzFgnqt5nEiGFfxKusvDoA6BUo0Rp+cKKOIzzfv6otq/ kpRRuP+HyvUkD9x+L5YQhMADIdXrCFI02ZSPdqtvmXw7VU8gh/UfB6dbjkhoOthe uTXxXDQGZFdGKCSI/q4zsExd8ksRUW3gtKsm6XuwC7lpE8hpShoDNKTwcoVsd46A nPbJN0nQ9HKCu422vhCvauV1CFXaMH+8/Y2PeA7C1RTGI7OBFaDnKmHPMftXPOKA KzJgJxlfbaQALP6Fo0T7aqqGx3QbdAdHDK+COJcmwNhrDUvcPm8huOK5p4y8Jnjo vid5ZONet+TnmAHaTxOSqhgleR3pBY/v4rBnBGAjtEsOWvoB6Wh3egnfpXp+AG6Q ddXjzmjncREr9sMNRSJSHDUqonBUlVRYsBCqX/6bpqzPj7UhX80rOS33NvhxxM4+ l3yxbawHbfJ4rqhlnFlOuBtNmJMTyiZfrAgOmMOANtAKsZidUN8F4vYqI2u7FTTe Ublw7EcSgMka2JM+6vMa =fTwc -----END PGP SIGNATURE----- --D09c906F7ai3o6jS6JOUXjgm5NL42CW9O-- From ionic@ionic.de Tue Nov 29 18:55:08 2016 Received: (at control) by bugs.x2go.org; 29 Nov 2016 17:55:08 +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.7 required=3.0 tests=BAYES_20,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MISSING_SUBJECT,SPF_HELO_PASS autolearn=no version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id DC14E3BC8A for ; Tue, 29 Nov 2016 18:55:07 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id hJ7ICEtxP3lI for ; Tue, 29 Nov 2016 18:55:01 +0100 (CET) Received: from Root24.de (powered.by.root24.eu [5.135.3.88]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 7C6DD3BC7D for ; Tue, 29 Nov 2016 18:55:00 +0100 (CET) Received: from nopileos.local (178.162.222.41.adsl.inet-telecom.org [178.162.222.41]) by mail.ionic.de (Postfix) with ESMTPSA id 0E11E4F00120 for ; Tue, 29 Nov 2016 18:54:59 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=ionic.de; s=default; t=1480442100; bh=W6KkGGx4EWgTc9qhB56sQUbCh/+7gK4tGLzTVkOAxwc=; h=To:From:Date:From; b=GIYs4lzziQHeC3oQSWoIE4uLHjWww+taglW6hVlO+FjsRG9njhvRGtKm0UactA7OW qchtCvrYXsNkQ5ika/WMHX6rYXu9EOvSfOrVz45RbH6Qql4Qnc/Wm98KF2bnV4Q02h s4N1dsXV3ufUEsjkAwq0ALRrL9hzTFk4mz7PEbXQ= To: control@bugs.x2go.org From: Mihai Moldovan Message-ID: <698997fc-abbd-60ff-bfb7-60f6abcfa6be@ionic.de> Date: Tue, 29 Nov 2016 18:54:58 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:45.0) Gecko/20100101 Thunderbird/45.5.0 MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="6pnUrHGTxJOC09PtQp58gUVvPKmtdt099" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --6pnUrHGTxJOC09PtQp58gUVvPKmtdt099 Content-Type: multipart/mixed; boundary="WHx00s47GOeQOwrvmKmfd7FD3LeN75umL"; protected-headers="v1" From: Mihai Moldovan To: control@bugs.x2go.org Message-ID: <698997fc-abbd-60ff-bfb7-60f6abcfa6be@ionic.de> --WHx00s47GOeQOwrvmKmfd7FD3LeN75umL Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable tag 1118 build-win32 --WHx00s47GOeQOwrvmKmfd7FD3LeN75umL-- --6pnUrHGTxJOC09PtQp58gUVvPKmtdt099 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBCgAGBQJYPcDyAAoJEB/WLtluJTqH9agP/RRfdFe3WSra6zMJxrH5DTDq eIUGO2EAk6IRk447iJJRxyc+rt5+FDn4Y3HpfGg09DQf72IIiQRGZmS4HPOXNDVT ySFcyJx3DxP1LJPYm5Pto8IPXIvvVBkmXXf0YiPeslsHrBZuR+TJck5mGRyz6JvD sYq9UhFrZmvT6xW+hBBCvGn/Vb5uBn3vFyUpzKj7AA37mtIR2ZRRV1mxIHbwcdel u52NA+ipufxy1ZvJPSiaNcGu05G2eFtTIkki/I+/sfYKRYzyVOLdCTwd9EVi7JF6 oloFnLD+O+Ljka9nkGNvbksk7NDej03nDi5nk7908zfhD3RxmKMI6z5CqP4Zi8Bs usV5UCONzDA+tr7TYxZKNhW1OWJRsJvRObRhqVzHyCknWdZasnMq+WyxcTNxVb+W mPQmNnA87zaAQ3/0O0XU/ugjFSTv/n6OW9YUTaAgvdqXLfEgzIAbekaJclbGcJui BJ272EPH2afbky085RX7tql1UtKsEE5rjmYsOeUAFa+rdblnROKBwQSKksiBmyEA eTu3pl9o5ZZYFCpmc1OLweBhdggp8IL2lYY8aHFfJCNgW1AEoqtD5KM/ueOj8RJQ f3LUTOm0ap5VeCeP4kA4gnz8iozJ53DOv+TRb5pGVdD3/+vqXAkMyD9NlDae4HPg Y35b4XlanAEEyOLpO8DA =qK41 -----END PGP SIGNATURE----- --6pnUrHGTxJOC09PtQp58gUVvPKmtdt099-- From ionic@ionic.de Fri Dec 9 20:15:33 2016 Received: (at control) by bugs.x2go.org; 9 Dec 2016 19:15:33 +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.7 required=3.0 tests=BAYES_20,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MISSING_SUBJECT,SPF_HELO_PASS,URIBL_BLOCKED autolearn=no version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 005D25DAA3 for ; Fri, 9 Dec 2016 20:15:33 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id R-d4S5ilIapk for ; Fri, 9 Dec 2016 20:15:26 +0100 (CET) Received: from Root24.de (powered.by.root24.eu [5.135.3.88]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 51DA05DA86 for ; Fri, 9 Dec 2016 20:15:26 +0100 (CET) Received: from nopileos.local (178.162.222.41.adsl.inet-telecom.org [178.162.222.41]) by mail.ionic.de (Postfix) with ESMTPSA id DC33E4F00120 for ; Fri, 9 Dec 2016 20:15:25 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=ionic.de; s=default; t=1481310926; bh=RrjcXYkxqxGilVjRQ9JkWOfWCePdp+kilmFYxoCqdS4=; h=To:From:Date:From; b=dOwVKcHBU+QPz2SeP2RGIC/tZs/1iAbgnaPEMmQ76zV9KXnZZ0LUNCjY8H1iWsKg+ hlrCXdn1FGZKnNh34z4kMC4lA0pNxiHRnDvyRUHX80hYXNg80iyZJGXBOSiCotGkef HwQ28sEHMCCFt8w1FTBkTIV80tFCgpvzISnKEzVo= To: control@bugs.x2go.org From: Mihai Moldovan Message-ID: <850faa98-1ce3-5c03-e8ef-05cba7eec922@ionic.de> Date: Fri, 9 Dec 2016 20:15:24 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:45.0) Gecko/20100101 Thunderbird/45.5.0 MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="7shv5rDXabdN8afPC89Swti90O7cnHxTe" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --7shv5rDXabdN8afPC89Swti90O7cnHxTe Content-Type: multipart/mixed; boundary="B6ggrml1w0TWhuS2s6Ol8El9TJuIQ8kHq"; protected-headers="v1" From: Mihai Moldovan To: control@bugs.x2go.org Message-ID: <850faa98-1ce3-5c03-e8ef-05cba7eec922@ionic.de> --B6ggrml1w0TWhuS2s6Ol8El9TJuIQ8kHq Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable forcemerge 1118 1105 --B6ggrml1w0TWhuS2s6Ol8El9TJuIQ8kHq-- --7shv5rDXabdN8afPC89Swti90O7cnHxTe Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBCgAGBQJYSwLMAAoJEB/WLtluJTqHbnMP/1xiPISSg9KhcOeySK4T8uI5 KZsxhhHgvJviXi6U1ZStjIqiHI6Ykm+4938xwYKjdgn5AKkSVVi+HBaIHfmdzsCT rA57Jt3S5NDzHTGIqNXB1beNUSxtK2dncSr8oGBAY22obZepwOUDfn2NWhVWhQ7p UzxItcsFejFlWiX5oyjqfZzfOncoNtMoHlTSCyB9FWKUlct2sE98+U9dfh/Z63x4 Obt0Un9n9mCB1Hs7RHxvEEZDwpaVV9zIf5JnOxaRAOiL0TlxnKJv+5KuhWrZa46T IxOJ0DBqDK/HjHLd9wCBDqx2SKIZ/12jp85RZEtZkcwWDq51ibicLQAPeTva/o4j 2zJpLAXM0ULh7LwCQ7L/xnsBeJzeep4XC+wqr+cF6iDxzK1+ge1LFAg1CAG7oQ+S ddkiDKL6pz57SGyHai2CevtlIYr4aJHfZInuUgMaD9qJlTo+8nxG6whGbCSsDccB mWfCZ9FDFPeSxfpSHc7WyKnb43SKtu8Z0dcnosz2j5bKh9SJMLE3f/PredoyQdML avFMrOXk43rWWXiDBt8mE26KxAcq+ReXRP/5rjzA3Fwzyp19buhTxbX1xrzgb/51 wx9dH/b/rwzCt2XFrPSOWzdD7qJpAcpl5awx9yRgIsnhs394Jrl3kUQnkRRwC7tO f41E/YbqEgOHGUaCUta4 =qcxa -----END PGP SIGNATURE----- --7shv5rDXabdN8afPC89Swti90O7cnHxTe-- From x2go@ymir.das-netzwerkteam.de Fri Dec 9 20:21:27 2016 Received: (at control) by bugs.x2go.org; 9 Dec 2016 19:21:45 +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=-2.9 required=3.0 tests=ALL_TRUSTED,BAYES_00, URIBL_BLOCKED autolearn=ham version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id B4BDD3CC4C; Fri, 9 Dec 2016 20:21:27 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id q9wKNUUZvvWw; Fri, 9 Dec 2016 20:21:20 +0100 (CET) Received: by ymir.das-netzwerkteam.de (Postfix, from userid 1005) id 6F1585DAA3; Fri, 9 Dec 2016 20:21:20 +0100 (CET) From: Mihai Moldovan To: 1105-submitter@bugs.x2go.org Cc: control@bugs.x2go.org, 1105@bugs.x2go.org Subject: X2Go issue (in src:x2goclient) has been marked as pending for release Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit X-Mailer: http://snipr.com/post-receive-tag-pending Message-Id: <20161209192120.6F1585DAA3@ymir.das-netzwerkteam.de> Date: Fri, 9 Dec 2016 20:21:20 +0100 (CET) tag #1105 pending fixed #1105 4.0.5.3 thanks Hello, X2Go issue #1105 (src:x2goclient) reported by you has been fixed in X2Go Git. You can see the changelog below, and you can check the diff of the fix at: http://code.x2go.org/gitweb?p=x2goclient.git;a=commitdiff;h=08a070b The issue will most likely be fixed in src:x2goclient (4.0.5.3). light+love X2Go Git Admin (on behalf of the sender of this mail) --- commit 08a070bbb18da3d1a6c4578e3310067f68c1a628 Author: Mihai Moldovan Date: Fri Dec 9 20:18:04 2016 +0100 debian/changelog: fixup latest entry and add closure for #1105. diff --git a/debian/changelog b/debian/changelog index fbaaa36..4ad9a49 100644 --- a/debian/changelog +++ b/debian/changelog @@ -27,7 +27,8 @@ x2goclient (4.0.5.3-0x2go1) UNRELEASED; urgency=medium multiple autostart apps at the same time. Fixes: #1024. [ Oleksandr Shneyder ] - * Fix SSHFS on Windows client. + * New upstream version (4.0.5.3): + - src/onmainwindow.cpp: fix SSHFS on Windows client. Fixes: #1105. -- X2Go Release Manager Mon, 19 Sep 2016 09:07:07 +0200 From x2go@ymir.das-netzwerkteam.de Wed Feb 22 05:10:27 2017 Received: (at control) by bugs.x2go.org; 22 Feb 2017 04:11:09 +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=-2.9 required=3.0 tests=ALL_TRUSTED,BAYES_00, URIBL_BLOCKED autolearn=unavailable version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 196224809E; Wed, 22 Feb 2017 05:10:27 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id fwh6C+qmdqHl; Wed, 22 Feb 2017 05:10:18 +0100 (CET) Received: by ymir.das-netzwerkteam.de (Postfix, from userid 1005) id 95011480A0; Wed, 22 Feb 2017 05:10:08 +0100 (CET) From: X2Go Release Manager To: 1105-submitter@bugs.x2go.org Cc: control@bugs.x2go.org, 1105@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: <20170222041008.95011480A0@ymir.das-netzwerkteam.de> Date: Wed, 22 Feb 2017 05:10:08 +0100 (CET) close #1105 thanks Hello, we are very hopeful that X2Go issue #1105 reported by you has been resolved in the new release (4.1.0.0) of the X2Go source project »src:x2goclient«. You can view the complete changelog entry of src:x2goclient (4.1.0.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=a972b0cd4622a101b1ddb3d4d3b0cf1f71dbfd6d;hp=550ae93c0ea493e0c740d6c1c1e83b014d8265dd 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.1.0.0-0x2go1 Status: RELEASE Date: Wed, 22 Feb 2017 05:02:35 +0100 Fixes: 910 1024 1096 1105 1124 1139 Changes: x2goclient (4.1.0.0-0x2go1) RELEASED; urgency=medium . [ Mihai Moldovan ] * New upstream version (4.1.0.0): - src/sessionbutton.cpp: replace drop down menu button on session buttons with a more prominent, bigger settings gear icon. Requested for visibility. - src/onmainwindow.cpp: add debug statements to createRSAKey () regarding the host pub key locations on OS X. - src/onmainwindow.cpp: fix error message string in printSshDError_noHostPubKey (). - src/sshmasterconnection.cpp: add SecurID prompt to challenge auth code prompts. - src/sharewidget.cpp: fix compile on UNIX-like platforms by defaulting to ISO-8859-1 not just on Linux but also other platforms. Fixes: #1124. - src/x2goutils.cpp: don't try to wrap absolute paths as legacy resource URIs in wrap_legacy_resource_URIs (). Fixes: #910. - {res/img/svg/,res/resources.qrc}: add new file hamburger.svg, a custom-created hamburger icon. - src/sessionbutton.cpp: replace drop down menu button on session buttons again with the custom-created hamburger icon. Also fix positioning issues in mini mode. - src/onmainwindow.{cpp,h}: add new function images_resource_path (), similar to iconsPath () but without explicitly referring to icons. - src/sessionbutton.cpp: we cannot use ".." in resource paths to move up one level, so use images_resource_path () instead and hope that we finally get a yummy hamburger. - src/onmainwindow.{cpp,h}: let iconsPath () take a const-reference instead of an object and make it a const member function. - src/onmainwindow.{cpp,h}: let images_resource_path () take a new, optional "base" parameter, that might be empty and is put in-between the fixed resource path string and the filename parameter. - src/onmainwindow.cpp: use images_resource_path () in iconsPath (). - src/sessionbutton.cpp: instead of using the hamburger.svg file directly, render it to a raster image and use this Pixmap as the icon file - otherwise the icon will not show up. Also change the icon size back to 16x16 px, that's big enough. - src/onmainwindow.cpp: remove left-over debug test message. - {unixhelper.{cpp,h},x2goclient.pro{,.maemo}}: add unixhelper stub. - unixhelper.{cpp,h}: implement cleanup program for process group. - x2goclient.cpp: wrap X2Go Client main function and use that. - x2goclient.cpp: add fork_helper() function to start up the UNIX cleanup helper. - x2goclient.cpp: use setsid() on UNIX to become session and process group leader. If that fails, fork, terminate the parent and execute setsid() in the child process. Use fork_helper() to start the UNIX cleanup helper in a child process and continue with the main application in the parent. - x2goclient.cpp: add myself to copyright header. - x2goclient.cpp: add legacy header needed for old operating systems. - x2goclient.cpp: fix std::edit -> std::exit typo. - unixhelper.{h,cpp}: also include to have Q_OS_UNIX defined on UNIX-based platforms. Move guards around. - unixhelper.h: forgot to declare type of unix_cleanup() function. - {unixhelper.{cpp,h},x2goclient.cpp}: unixhelper should really be a (module) namespace, not a class. - unixhelper.{cpp,h}: add documentation/comments. - x2goclient.cpp: return return value of unixhelper::unix_cleanup(). - unixhelper.cpp: kill process group on error. - unixhelper.{cpp,h}: implement signal unblocking in unixhelper::unix_cleanup(). Update documentation. - unixhelper.cpp: port from std::signal() to sigaction(). Handle errors. - x2goclient.cpp: clean up. We don't care if setsid() fails or succeeds and we certainly do not need to fork to force it to succeed. Whether we are a process group leader already or become one doesn't matter, if the end result is that we are process group leader. - x2goclient.cpp: don't start the UNIX cleanup helper process right after forking, but introduce a new command line option --unixhelper and re-execute the main binary with this new option to indicate that the UNIX cleanup helper tool is requested. It is necessary to call exec() after fork on virtually all operating system, especially on OS X. - x2goclient.cpp: fix compile problems introduced with the last commit. - x2goclient.cpp: fix string comparison. - x2goclient.cpp: create new argv array on the heap instead of on the stack. Stack data will be invalid once the function goes out of scope (which is what execv is doing.) - unixhelper.{cpp,h}: raise sleeping time to two seconds and grace period to 10 seconds. - {unixhelper.{cpp,h},x2goclient.cpp}: switch main cleanup handling to parent PID polling. - unixhelper.{cpp,h}: ignore SIGINT, SIGTERM, SIGPIPE, SIGQUIT, SIGUSR1 and SIGUSR2. - x2goclient.cpp: fix failing exec() call for the UNIX helper utility: correctly terminate arguments vector. - {appdialog,configwidget,onmainwindow,unixhelper}.cpp: fix some compile warnings with GCC. Fix a few whitespace issues. - appdialog.cpp: initialize parent in default case. Another GCC compile warning fix. - onmainwindow.cpp: correctly use ~/.x2go/.ssh as ssh directory when starting sshd in user mode. Put the authorized_keys file in there. Check and set correct permissions for both the directory and authorized_keys file. Generalize some Windows-specific sections by using QDir and QFile. - compat.{cpp,h}: new files. Implements strndup on OS X 10.6 and below. Add to x2goclient.cpp, x2goclient.pro and x2goclient.pro.maemo. - compat.{cpp,h}: remove inline keyword, because function is not defined in header file. - compat.h: include QtCore/qglobal.h for Q_OS_... macros. - pulsemanager.{cpp,h}: new class for PulseAudio management. - x2goclient.pro{,.maemo}: reference new pulsemanager.{cpp,h} files. - macbuild.sh: add new MACPORTS_PREFIX detection/variable. - macbuild.sh: add new dependency_error() helper function. - macbuild.sh: add new lazy_canonical_path() helper function. - macbuild.sh: add new get_nesting_level() helper function. - macbuild.sh: add new repeat_str() helper function. - macbuild.sh: refactor nxproxy detection to use MACPORTS_PREFIX. - macbuild.sh: add EXE_DIR and FRAMEWORKS_DIR internal variables. Create directories based on that. Use them when bundling. - macbuild.sh: define PulseAudio libraries and binaries to be later copied. Discover them using MACPORTS_PREFIX. Error out if any component was not found. - macbuild.sh: copy PulseAudio libraries and binaries. - macbuild.sh: add phase output for bundling nxproxy. - macbuild.sh: add work-in-progress PulseAudio bundling. Needs to be ripped apart and be redone, because dylibbundler is not smart enough to handle our case. - macbuild.sh: add Linux library file name scheme regex as a precaution. - macbuild.sh: directories are "executable", so the first check must explicitly include a check for the argument being not a directory. - macbuild.sh: add some more debugging output. - macbuild.sh: remove libpulse and libpulsecore from libraries to be bundled, as something else will bundle them anyway. - macbuild.sh: use while-read-loop and find to actually recurse through directories. - macbuild.sh: check current file name against regex, not a (now) full path. - macbuild.sh: add the correct file path to the library bundling array. - macbuild.sh: when actually bundling libraries, use @executable_path instead of @loader_path. Will need testing. - macbuild.sh: fix echo call to also show the intermediate lib dir, not just the library file name. Helps debugging. - macbuild.sh: also install libraries into staging area with intermediate library path. Putting everything into a single place is a stupid idea. - deduplicate.sh: add new, more or less Proof of Concept, and Work in Progress file deduplication script. - deduplicate.sh: add duplicates-to-real mapping. - deduplicate.sh: add missed local variable declaration. - deduplicate.sh: new function for parsing otool's output and printing library dependencies if no error occurred. - deduplicate.sh: non-functional: change WARNING to ERROR in error messages. - deduplicate.sh: add first skeleton for library dependency rewriting of deleted duplicates. - deduplicate.sh: correctly unset an array element. - deduplicate.sh: change range-based for loops to "${!arr[@]}" to handle "sparse" arrays correctly. - deduplicate.sh: quote "${arr[@]}" correctly to not suddenly force word splitting. - deduplicate.sh: fix parse_otool_output: we want to return failure only iff any of the strings are present in otool's output and actually handle the failure string *ARRAY* correctly as that. - deduplicate.sh: change exit to return in parse_otool_output. - deduplicate.sh: record crafted "library path" values in duplicates replacement to_files array. - deduplicate.sh: debug parse_otool_output fiercely for the time being. - deduplicate.sh: prevent word splitting when parsing otool's output. We want the raw bits. - deduplicate.sh: switch to range-based for loop in fixup section for duplicate array walking as we need to access the same element index in the replacement array. - deduplicate.sh: add more useful information in fixup section's error message. - deduplicate.sh: unstub fixup section. - deduplicate.sh: parse_otool_output: jump over first matching entry, which is - hopefully - the id line. - deduplicate.sh: actually include file name in install_name_tool pseudo output. - deduplicate.sh: save dependency format base string as a readonly variable and use that instead of repeating a fixed string. - deduplicate.sh: copy lazy_canonical_path from macbuild.sh for now. deduplicate.sh will eventually be merged into macbuild.sh anyway. - deduplicate.sh: more temporary debugging output in fixup section. - deduplicate.sh: surprisingly, it turned out that only checking the duplicate's basename against the current dependency's basename is not good enough. Actually construct "the real dependency-format duplicate string" and check it against "the real dependency". Even this is still not good enough as it needs a bit cleanup - coming up next. - deduplicate.sh: cleanup as announced in the last commit - using lazy_canonical_path. - deduplicate.sh: remove a bit of noisy debug output, but turn on parse_otool_output debugging. - deduplicate.sh: fix wrong return value capture. We want to check the return value of the executed function/command, not that of "typeset". - deduplicate.sh: add WARNING to status message. - deduplicate.sh: remove some noisy debug output. - macbuild.sh: replace tabs with two spaces. No functional changes. - deduplicate.sh: replace tabs with two spaces. No functional changes. - macbuild.sh: merge deduplicate.sh content in. Also enable the functionality "for real", not just as a dry-run. - macbuild.sh: spelling fix. - macbuild.sh: more more debug messages and a whitespace change. - macbuild.sh: fix removal of base prefix in deduplication if base prefix ends in a slash. - macbuild.sh: copy "special" files, so that they can be removed later on. - macbuild.sh: rewrite ID line detection algorithm. Modules don't seem to be needing one, so skipping the first dependency line incidentally skips a "real" dependency for modules. - pulsemanager.{cpp,h}: use name and underscore for member variables, not underscore and name. - pulsemanager.cpp: minor non-behavior changing fixes. - pulsemanager.{cpp,h}: reformat only. - pulsemanager.cpp: use initializer list for constructor. - pulsemanager.{cpp,h}: add pulse_version_{major,minor}_ member variables. - pulsemanager.cpp: reformat only. - pulsemanager.{cpp,h}: switch port definitions to std::uint16_t. - pulsemanager.h: add std:: namespace selector for uint32_t version variables. - pulsemanager.{cpp,h}: add ESD support. - pulsemanager.{cpp,h}: rename relaunch () to restart (). - pulsemanager.cpp: reformat only. - pulsemanager.{cpp,h}: fixup preprocessor usage. - pulsemanager.{cpp,h}: add server_args_, server_binary_ and server_working_dir_ class variables. - pulsemanager.cpp: make start () wrap the "real" OS-specific start functions. - pulsemanager.{cpp,h}: new function create_client_dir (). - pulsemanager.cpp: add Windows support to shutdown (). - pulsemanager.cpp: add cleanup support to on_pulse_finished (). - pulsemanager.cpp: add Windows stuff to initial env in constructor. - pulsemanager.{cpp,h}: add new function start_generic () to split off common functionality. - pulsemanager.cpp: let start_osx () use start_generic (). - pulsemanager.{cpp,h}: implement start_win () as part of Windows functionality. - pulsemanager.cpp: also load module-esound-protocol-tcp module. - pulsemanager.cpp: make generate_server_config () Windows-compatible. - pulsemanager.cpp: make generate_client_config () Windows-compatible. - pulsemanager.cpp: only play startup sound if DEBUG macro is defined. - pulsemanager.cpp: make startup sound playing via slot_play_startup_sound () Windows-compatible. - pulsemanager.cpp: typo fix in variable name server_working_dir_. - pulsemanager.{cpp,h}: find_port () actually returns a value, fix declaration accordingly. - pulsemanager.cpp: fix compile errors in find_port () by renaming the ret or port variable to search_port. - pulsemanager.cpp: fix another compile error due to typo'd versions of generate_server_config () and generate_client_config (). - pulsemanager.cpp: another typo fix: findPort -> find_port. - pulsemanager.cpp: reorder member variables in initialization list. Compile warning fix. - pulsemanager.cpp: add "dummy" start_linux () function. - pulsemanager.cpp: generate platform-dependent values for member variables server_working_dir_ and server_binary_ in constructor. We need them right away. - pulsemanager.{cpp,h}: new member function fetch_pulseaudio_version () with first skeleton. Call it in the constructor. - pulsemanager.cpp: re-initialize buffer string on each run while getting CWD on Linux. - pulsemanager.cpp: fix some error messages by removing redundant newlines. - pulsemanager.{cpp,h}: some older PA versions used the major, minor and micro numbering scheme together with a descriptive string. Add new member variables for this and initialize them correctly. - pulsemanager.h: reformat a bit by adding newlines and moving lines in a (hopefully) logical fashion. - pulsemanager.cpp: add algorithm for actually extracting PA version number. - pulsemanager.{cpp,h}: only allow changing PA or ESD ports when server is not currently running. - pulsemanager.{cpp,h}: add record and playback private members, as well as getters and setters. - pulsemanager.cpp: reorder member variables in initializer list. - pulsemanager.cpp: fix compile error in startup sound function. - pulsemanager.h: more moving around. - pulsemanager.cpp: use record and playback member variables. - pulsemanager.cpp: cleanup, don't use copy assignment operator when we just need to construct a new object. - macbuild.sh: PulseAudio has been updated to version 8.0. Reflect this. - macbuild.sh: libpulsecore.dylib also resides in the "private" pulseaudio subdirectory now, adding it to "special files workaround". - pulsemanager.cpp: remove config.pa while doing post-exit cleanups. - src/pulsemanager.{cpp,h}: add get_pulse_dir () for fetching the (OS-dependent) PulseAudio config and cookie directory. - src/pulsemanager.cpp: replace hardcoded modules paths with dynamic ones depending on the detected PA version. - src/pulsemanager.cpp: add "FIXME" debug logging (and cleanup.) - src/pulsemanager.cpp: backport fix for 526. - src/pulsemanager.cpp: backport fix for 422. - src/configdialog.{cpp,h}: add "no record" support for OS X (through PulseManager.) - src/configdialog.cpp: whitespace only. - src/onmainwindow.{cpp,h}: hook PulseManager in - for both OS X and Windows. - src/mediawidget.cpp: reflect new capabilities through PulseManager. - src/pulsemanager.h: move x2gologdebug.h inclusion into implementation file. - src/pulsemanager.cpp: output PA stdout/stderr via x2goDebug, not plainly to stdout. - src/configdialog.cpp: fix compile error by removing obsolete (and now faulty) line for removed restart message. - src/onmainwindow.cpp: fix compile errors due to misspelled function names. - src/pulsemanager.cpp: let start () act as a wrapper and do nothing if the PA server is already running. - src/onmainwindow.cpp: fix compile error due to misuse of #ifdef. - src/onmainwindow.cpp: convert QDir object to QString for further processing. - src/onmainwindow.cpp: make use of new PulseManager::start () behavior and always use it, no matter whether server is up already or not. - src/pulsemanager.cpp: fix compile error due to typo. - src/pulsemanager.cpp: hide Windows-only code behind #ifdefs. Otherwise we'll get compile failures on non-Windows platforms. - src/pulsemanager.{cpp,h}: add debugging setter and private variable. - src/pulsemanager.{cpp,h}: port to new debugging feature. - src/onmainwindow.cpp: port to new debugging feature of PulseManager. - src/pulsemanager.h: fix compile error due to mismatching function declaration for set_debug (). - res/sound: add startup.wav sound to be played back when starting the PA daemon in debug mode. - src/pulsemanager.cpp: don't time out while trying to start PA or paplay. "Handle" errors by printing out debug messages. - macbuild.sh: add support for copying resource files. Currently only used for startup.wav. - res: rename "sound" to "audio". - macbuild.sh: fix copying of resource files. - macbuild.sh: also add PA binaries to deduplication fixup list. - macbuild.sh: don't error out while executing parse_otool_output (). - macbuild.sh: save return value of parse_otool_output (). Otherwise we'll fetch the return value of "set", which is not really what we need. - src/pulsemanager.cpp: fix typo in fetch_pulseaudio_version (). Read standard output from temporary process, not our "main" server process (which at this point is not even started yet.) - src/pulsemanager.cpp: don't overwrite the temporary string variable for the current line with what was supposed to hold new data in fetch_pulseaudio_version (). - src/pulsemanager.{cpp,h}: rename on_pulse_finished () to slot_on_pulse_finished (). Oops... - src/pulsemanager.cpp: actually only play startup sound if debugging is enabled. - src/pulsemanager.cpp: don't insert stray newline into config.pa. - src/pulsemanager.cpp: insert PULSE_SERVER and PULSE_COOKIE variables into environment as used by PulseManager. Otherwise we won't be able to start binaries like paplay. - src/pulsemanager.cpp: don't remove PA config and log file in ~/.x2go/pulse on PA shutdown if debugging has been requested. - src/x2goutils.{cpp,h}: add new function find_binary (). Searches for a specific binary in a colon-separated list of paths. Really just a compat function, as Qt 5 includes QStandardPaths::findExecutable (). - src/pulsemanager.cpp: auto-detect PA binary in $PATH and MacPorts default prefix as well on OS X. - src/pulsemanager.cpp: add comments to Qt 4 code part. - src/pulsemanager.cpp: add reminder for Linux-implementation in slot_play_startup_sound (). - src/x2goutils.cpp: fix compile error. - src/pulsemanager.h: add new system_pulse_ variable to indicate that a system PA binary shall be used. - src/pulsemanager.cpp: correctly initialize and set system_pulse_ when required. - src/x2goutils.{cpp,h}: add application modality parameter to show_RichText_Generic_MsgBox () and its wrappers. - src/pulsemanager.cpp: add error message boxes throughout PulseManager. Don't just abort () the program. - src/x2goutils.h: fix compile error due to typo. - src/pulsemanager.cpp: fix compile error on Windows. - src/pulsemanager.cpp: fix startup when using the system-PA version. - src/pulsemanager.cpp: fix startup sound playback when using the system-PA version. - src/pulsemanager.cpp: add warning message boxes where appropriate. - src/pulsemanager.cpp: remove extraneous newlines from debug output. - src/x2goutils.cpp: fix faulty logic in find_binary (): only reset the given path to CWD iff it's actually empty. Not the other way around. - src/x2goutils.cpp: add some debug logging to find_binary (). - nsis/x2goclient.nsi: add startup.wav to installer package. - src/: new file windows_stdint.h for Windows-compatibility when using std::(u)int*_t types. These are only available with GCC in C++11 mode on Windows for some reason. - src/pulsemanager.h: use windows_stdint.h header on Windows, cstdint otherwise. - src/onmainwindow.cpp: short out code parts unconditionally checking for the userSshd variable. We don't need it on Windows (and don't define it on purpose) because using an user-mode SSH daemon is implied there (for now.) - src/onmainwindow.cpp: drop obsolete pulseVersionTest variable initialization in Windows-only code. - src/onmainwindow.cpp: drop obsolete pulseVersionIsLegacy variable initialization in Windows-only code. - src/onmainwindow.cpp: add another pair of braces to in_addr-type variable initialization on Windows to silence compiler warning. - src/onmainwindow.cpp: fix compile error on Windows after moving some code around. - src/onmainwindow.cpp: use correct C++ universal initializer for initializing in_addr-type variable, the C-style universal zero initializer won't cut it here. - src/pulsemanager.h: include accidentally omitted wapi.h header file. - src/pulsemanager.cpp: make PA version parsing function more generic and don't expect that a number is always terminated with a period or dash. Especially on Windows, with a PA version number of "6.0", this lead to errors. OS X was fine with its "8.0.0-something" version number, but the optional part at the end or even the micro version number could be missing any time. - src/pulsemanager.cpp: make sure that wherever we interface with non-Qt functions, paths are in native form (i.e., containing empty separators, instead of the "generic" slash UNIX-style separator.) Should fix a PA startup issue on Windows. - src/pulsemanager.cpp: fix PA version fetching on OS X again. - src/unixhelper.{cpp,h}: make argument of kill_pgroup () const. - src/unixhelper.h: style and general fixes within comments. - src/unixhelper.{cpp,h}: split off core functionality of kill_pgroup () into a new function called real_kill_pgroup (). - src/unixhelper.{cpp,h}: rewrite kill_pgroup () to act as a wrapper around real_kill_pgroup (). Adjust comment. We need to do this so the cleanup process doesn't kill itself before all other processes in the process group are killed. - src/pulsemanager.cpp: remove unused variables. - src/unixhelper.cpp: code cleanup. - src/{unixhelper.cpp,x2goclient.cpp}: fix errno usage - save before use. - src/onmainwindow.cpp: document createRSAKey () because what it does is completely non-obvious. - src/onmainwindow.cpp: add support for explicitly setting the private key bit size based on type and use values (currently) regarded as secure to generateHostKey (). - src/onmainwindow.{cpp,h}: rename generateHostKey () to generateKey () and add a boolean parameter with a default of false to request a public key. Change old invocations of generateHostKey () accordingly. - src/onmainwindow.cpp: refactor generateKey () a little bit to compact it. - src/onmainwindow.{cpp,h}: rewrite generateKey () function to actually also generate user keys. Now returns the private key file path (public key file path is trivially determined by appending ".pub" to that return value.) Change all locations referencing generateKey () to use the return value (if necessary) and remove dead code that was used to generate a user key manually previously. - src/onmainwindow.cpp: add more error handling to generateKey () and fix up a broken error-handling section. - src/onmainwindow.cpp: fix up a few debug strings. - src/{onmainwindow.{cpp,h},configdialog.cpp}: remove userSshd variable and code related to non-user-mode-sshd setups. Simplifies code a bit and we want to always start a user-mode OpenSSH server anyway. Also get rid of the clientport variable that is now unnecessary. - src/onmainwindow.cpp: when starting sshd on Unix-based plattforms, raise maximum startup time to 5 seconds and break out early, if the process went into running state earlier. - src/onmainwindow.cpp: uppercase key type in debug output and comment string in generateKey (). - src/onmainwindow.cpp: terminate sshd more correctly. First via terminate (), then wait up to 5 seconds, then via kill (). - src/onmainwindow.{cpp,h}: add new function check_key_type (). Takes a key type and checks for validity/if it's known. - src/onmainwindow.{cpp,h}: add new function key_type_to_string (). Returns a stringified version of the selected key type, after checking for validity. - src/onmainwindow.{cpp,h}: add new function default_size_for_key_type (). Returns the default key size in bits for the selected key type, after checking for validity. - src/onmainwindow.cpp: use the new functions in generateKey (). - src/onmainwindow.h: move generateKey () declaration around. - src/onmainwindow.{cpp,h}: reformat generateKey () only. - src/onmainwindow.{cpp,h}: move createRSAKey () around. - src/onmainwindow.{cpp,h}: reformat createRSAKey () only. - src/onmainwindow.cpp: let startSshd () act as a wrapper and be called multiple times. Check if the OpenSSH Server is already running and return true, otherwise try to start it. - src/onmainwindow.cpp: add some comments only to createRSAKey (). - src/onmainwindow.{cpp,h}: rename createRSAKey () to the more-appropriate name createKeyBundle (). Also add a key-type parameter and use it to select the required key type (and maybe generate it if necessary.) Also use the new functions to get a stringified version of the key type etc. - macbuild.sh: switch to bundling PA 9.0. - src/onmainwindow.cpp: move QProcess startup check in startSshd () to non-Windows code section and add a listening socket check loop. Windows uses other means of starting the daemon. - src/onmainwindow.cpp: add comment to last all-in listening check in startSshd (). - src/onmainwindow.cpp: add listening-check loop to the Windows-specific code of startSshd (). - src/onmainwindow.{cpp,h}: make startSshd () private. There's no need to keep it public, as only class-internal functions are using it. - src/onmainwindow.{cpp,h}: add ssh_key_type_ member variable and getter and setter to WinServerStarter class. - src/onmainwindow.{cpp,h}: add SSH key type parameter to startSshd (). - src/onmainwindow.cpp: pass new ssh_key_type_ member to startSshd () call in WinServerStarter::run (). - src/onmainwindow.cpp: pass key_type parameter to startSshd () in createKeyBundle (). - src/onmainwindow.cpp: add and use key_types parameter to startWinServers (). - src/onmainwindow.cpp: actually use key_type parameter in startSshd () and make selection more generic. - src/onmainwindow.h: move WinServerStarter class definition around to hopefully get the definition of ONMainWindow::key_types. - src/onmainwindow.h: remove WWrapper definition, seems to be unused. - src/onmainwindow.{cpp,h}: don't try to initialize non-static ssh_key_type_ member variable directly, but in the constructor. Also, use the initializer list to initialize the other member variables directly. - src/onmainwindow.cpp: re-order initializer list of WinServerStarter constructor. - src/onmainwindow.h: actually, we do need startSshd () to be public. Move it back. - src/onmainwindow.cpp: only pass non-Cygwin paths to QFile::exists () in generateKey (). Refactor a bit to drop the now-unnecessary ret variable. - src/onmainwindow.cpp: use QFileInfo to get the key file's basename in exportDirs () instead of the old, potentially faulty string replace dance. - src/x2goutils.{cpp,h}: make find_binary () and add_to_path () available on all UNIX-based operating systems. We will need it for sftp-server discovery. - src/pulsemanager.cpp: refactor PA binary searching code. Less duplication now. - src/onmainwindow.cpp: fix sshd_config generation on UNIX by searching for the real sftp-server binary path. - src/pulsemanager.cpp: fix PA binary searching logic. We don't want to unconditionally set system_pulse_ to true if we found a binary... it might as well be the bundled one... - src/onmainwindow.cpp: move generateEtcFiles () call to startSshd (). We must make sure that the sshd_config file exists and is up-to-date, so let's always do it right before starting the sshd binary. - src/onmainwindow.cpp: stop duplicating the list of known sftp-server binary locations. - src/{onmainwindow,pulsemanager}.cpp: actually *use* what we got back from add_to_path ()... Fixes a few bugs here and there. - src/onmainwindow.cpp: remove trailing slash from Mageia/SUSE/Arch sftp-server binary search path. - src/x2goutils.cpp: add debugging output to add_to_path (). - src/x2goutils.cpp: fix add_to_path (). We want to remove trailing slashes from elements-to-add, not just fetch a trailing slash if encountered. Also, better don't compare a value from the original path value to itself, but rather to the things we want to add in order to remove duplicates, not remove most of the list of elements to add... - src/unixhelper.cpp: use cstdio instead of stdio.h. It's the C++ header and imports functions into the std namespace. - src/: create new file unix_stdint.h which imports the (u)int*_t types into the std namespace. - src/pulsemanager.h: use unix_stdint.h on non-Windows-platforms to get std::(u)int*_t types without having to use C++11. - src/pulsemanager.cpp: fix Linux-based code by fixing a really stupid typo ("erange" instead of "errno") and including the cerrno header. - src/pulsemanager.cpp: use std::ssize_t for the path length, because it could be -1 and std::size_t is not applicable. - src/pulsemanager.cpp: cast the return value of realloc () to char*, as GCC doesn't seem to like reassigning the pointer otherwise. - src/unix_stdint.h: use same hack to import ssize_t into std namespace. - src/onmainwindow.{cpp,h}: remove left-over parec references and code. - src/onmainwindow.cpp: convert piece of code to non-Windows and non-Darwin, so that not only Linux but other operating systems are covered as well. - src/mediawidget.cpp: unify some redundant code. - src/mediawidget.cpp: simplify expression. - src/mediawidget.cpp: add deprecation warning for ARTS. - src/mediawidget.cpp: add deprecation warning for ESounD. - macbuild.sh: make compatible with new llvm-based otool binary, which now uses proper return values. Be backwards-compatible with otool-classic. - res/i18n/: refresh translation files. - deduplicate.sh: remove file, now obsolete and merged into macbuild.sh. - res/i18n/: refresh translation files again after copyright date changes. - res/i18n/x2goclient_fr.ts: fixup French translation file. - res/i18n/x2goclient_sv.ts: fixup Swedish translation file a little bit and add comments/requests for re-translations for the next time. - macbuild.sh: switch to bundling PulseAudio 10.0 since it's now available in MacPorts. - src/x2goclient.cpp: no need to put new_argv_c_str std::vector onto the heap. Keep it on the stack. - src/x2goclient.cpp: use $PATH-exploration when re-executing x2goclient in UNIX cleanup helper mode. Fixes: #1139. - src/onmainwindow.cpp: correctly initialize pulseManager and pulseManagerThread member variables. Fixes crashes on OS X and Windows if X2Go Client terminates before these objects have been created at run time (for instance when running x2goclient --help.) - src/onmainwindow.cpp: correctly guard the new lines from the previous commit. Fixes compile issues on Linux and other systems. - res/i18n/x2goclient_es.ts: fixup Spanish translation file where necessary and add comments/requests for re-translations for the next time. - res/i18n/x2goclient_es.ts: remove obsolete comments and very minor whitespace fixup. - res/i18n/x2goclient_de.ts: fixup German translation file, including typo, whitespace, grammar fixes and other stuff. - res/i18n/x2goclient_sv.ts: remove obsolete comments and very minor whitespace fix. - macbuild.sh: sanitize find input, as BSD find and GNU find behave differently. - macbuild.sh: get rid of problems and use gfind directly. We need special GNU features. - src/appdialog.cpp: logic so that X2Go-Top applications are shown again. Used faulty logic when silencing compiler warnings. - res/i18n/x2goclient_fi.ts: fix up Finnish translation file, including typo, whitespace fixes and marking untranslated entries as such again, after removing the English "translation" text. . [ Bernard Cafarelli ] * New upstream version (4.1.0.0): - {res/,x2goclient.pro}: rename .rcc files to .qrc. Fixes: #1096. "rcc" is the extension for compiled resource files, whereas "qrc" is the mapping file. Qt5 stumbles upon that - i.e., it doesn't try to compile files with the "rcc" extension, while Qt4 did. - res/: fix up previous change, the renaming of changelog.qrc was not correctly committed. . [ Jason Alavaliant ] * New upstream version (4.1.0.0): - src/{onmainwindow.{cpp,h},help.cpp}: modify autostart option to support multiple autostart apps at the same time. Fixes: #1024. . [ Oleksandr Shneyder ] * New upstream version (4.1.0.0): - src/onmainwindow.cpp: fix SSHFS on Windows client. Fixes: #1105. . [ Sébastien Ducoulombier ] * New upstream version (4.1.0.0): - res/i18n/x2goclient_fr.ts: update French translation file. . [ Daniel Lindgren ] * New upstream version (4.1.0.0): - res/i18n/x2goclient_sv.ts: update Swedish translation file. - res/i18n/x2goclient_sv.ts: fixup Swedish translation file. . [ Ricardo Díaz Martín ] * New upstream release (4.1.0.0): - res/i18n/x2goclient_es.ts: update Spanish translation file. - res/i18n/x2goclient_es.ts: update Spanish translation file. . [ Stefan Baur ] * New upstream release (4.1.0.0): - res/i18n/x2goclient_de.ts: update German translation file. . [ Martti Pitkänen ] * New upstream version (4.1.0.0): - res/i18n/x2goclient_fi.ts: update Finnish translation file. From unknown Thu Mar 28 23:47: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#1118 closed by X2Go Release Manager (X2Go issue (in src:x2goclient) has been marked as closed) Message-ID: References: <20170222041008.95011480A0@ymir.das-netzwerkteam.de> X-X2go-PR-Keywords: build-win32 pending X-X2go-PR-Message: they-closed 1118 X-X2go-PR-Package: x2goclient X-X2go-PR-Source: x2goclient Date: Wed, 22 Feb 2017 04:15:11 +0000 Content-Type: multipart/mixed; boundary="----------=_1487736911-17124-3" This is a multi-part message in MIME format... ------------=_1487736911-17124-3 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: #1118: Share folder is broken for x2goclient-4.0.5.2-2016.09.20-setup.exe 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 ------------=_1487736911-17124-3 Content-Type: message/rfc822 Content-Disposition: inline Content-Transfer-Encoding: 7bit Received: (at control) by bugs.x2go.org; 22 Feb 2017 04:11:09 +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=-2.9 required=3.0 tests=ALL_TRUSTED,BAYES_00, URIBL_BLOCKED autolearn=unavailable version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 196224809E; Wed, 22 Feb 2017 05:10:27 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id fwh6C+qmdqHl; Wed, 22 Feb 2017 05:10:18 +0100 (CET) Received: by ymir.das-netzwerkteam.de (Postfix, from userid 1005) id 95011480A0; Wed, 22 Feb 2017 05:10:08 +0100 (CET) From: X2Go Release Manager To: 1105-submitter@bugs.x2go.org Cc: control@bugs.x2go.org, 1105@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: <20170222041008.95011480A0@ymir.das-netzwerkteam.de> Date: Wed, 22 Feb 2017 05:10:08 +0100 (CET) close #1105 thanks Hello, we are very hopeful that X2Go issue #1105 reported by you has been resolved in the new release (4.1.0.0) of the X2Go source project »src:x2goclient«. You can view the complete changelog entry of src:x2goclient (4.1.0.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=a972b0cd4622a101b1ddb3d4d3b0cf1f71dbfd6d;hp=550ae93c0ea493e0c740d6c1c1e83b014d8265dd 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.1.0.0-0x2go1 Status: RELEASE Date: Wed, 22 Feb 2017 05:02:35 +0100 Fixes: 910 1024 1096 1105 1124 1139 Changes: x2goclient (4.1.0.0-0x2go1) RELEASED; urgency=medium . [ Mihai Moldovan ] * New upstream version (4.1.0.0): - src/sessionbutton.cpp: replace drop down menu button on session buttons with a more prominent, bigger settings gear icon. Requested for visibility. - src/onmainwindow.cpp: add debug statements to createRSAKey () regarding the host pub key locations on OS X. - src/onmainwindow.cpp: fix error message string in printSshDError_noHostPubKey (). - src/sshmasterconnection.cpp: add SecurID prompt to challenge auth code prompts. - src/sharewidget.cpp: fix compile on UNIX-like platforms by defaulting to ISO-8859-1 not just on Linux but also other platforms. Fixes: #1124. - src/x2goutils.cpp: don't try to wrap absolute paths as legacy resource URIs in wrap_legacy_resource_URIs (). Fixes: #910. - {res/img/svg/,res/resources.qrc}: add new file hamburger.svg, a custom-created hamburger icon. - src/sessionbutton.cpp: replace drop down menu button on session buttons again with the custom-created hamburger icon. Also fix positioning issues in mini mode. - src/onmainwindow.{cpp,h}: add new function images_resource_path (), similar to iconsPath () but without explicitly referring to icons. - src/sessionbutton.cpp: we cannot use ".." in resource paths to move up one level, so use images_resource_path () instead and hope that we finally get a yummy hamburger. - src/onmainwindow.{cpp,h}: let iconsPath () take a const-reference instead of an object and make it a const member function. - src/onmainwindow.{cpp,h}: let images_resource_path () take a new, optional "base" parameter, that might be empty and is put in-between the fixed resource path string and the filename parameter. - src/onmainwindow.cpp: use images_resource_path () in iconsPath (). - src/sessionbutton.cpp: instead of using the hamburger.svg file directly, render it to a raster image and use this Pixmap as the icon file - otherwise the icon will not show up. Also change the icon size back to 16x16 px, that's big enough. - src/onmainwindow.cpp: remove left-over debug test message. - {unixhelper.{cpp,h},x2goclient.pro{,.maemo}}: add unixhelper stub. - unixhelper.{cpp,h}: implement cleanup program for process group. - x2goclient.cpp: wrap X2Go Client main function and use that. - x2goclient.cpp: add fork_helper() function to start up the UNIX cleanup helper. - x2goclient.cpp: use setsid() on UNIX to become session and process group leader. If that fails, fork, terminate the parent and execute setsid() in the child process. Use fork_helper() to start the UNIX cleanup helper in a child process and continue with the main application in the parent. - x2goclient.cpp: add myself to copyright header. - x2goclient.cpp: add legacy header needed for old operating systems. - x2goclient.cpp: fix std::edit -> std::exit typo. - unixhelper.{h,cpp}: also include to have Q_OS_UNIX defined on UNIX-based platforms. Move guards around. - unixhelper.h: forgot to declare type of unix_cleanup() function. - {unixhelper.{cpp,h},x2goclient.cpp}: unixhelper should really be a (module) namespace, not a class. - unixhelper.{cpp,h}: add documentation/comments. - x2goclient.cpp: return return value of unixhelper::unix_cleanup(). - unixhelper.cpp: kill process group on error. - unixhelper.{cpp,h}: implement signal unblocking in unixhelper::unix_cleanup(). Update documentation. - unixhelper.cpp: port from std::signal() to sigaction(). Handle errors. - x2goclient.cpp: clean up. We don't care if setsid() fails or succeeds and we certainly do not need to fork to force it to succeed. Whether we are a process group leader already or become one doesn't matter, if the end result is that we are process group leader. - x2goclient.cpp: don't start the UNIX cleanup helper process right after forking, but introduce a new command line option --unixhelper and re-execute the main binary with this new option to indicate that the UNIX cleanup helper tool is requested. It is necessary to call exec() after fork on virtually all operating system, especially on OS X. - x2goclient.cpp: fix compile problems introduced with the last commit. - x2goclient.cpp: fix string comparison. - x2goclient.cpp: create new argv array on the heap instead of on the stack. Stack data will be invalid once the function goes out of scope (which is what execv is doing.) - unixhelper.{cpp,h}: raise sleeping time to two seconds and grace period to 10 seconds. - {unixhelper.{cpp,h},x2goclient.cpp}: switch main cleanup handling to parent PID polling. - unixhelper.{cpp,h}: ignore SIGINT, SIGTERM, SIGPIPE, SIGQUIT, SIGUSR1 and SIGUSR2. - x2goclient.cpp: fix failing exec() call for the UNIX helper utility: correctly terminate arguments vector. - {appdialog,configwidget,onmainwindow,unixhelper}.cpp: fix some compile warnings with GCC. Fix a few whitespace issues. - appdialog.cpp: initialize parent in default case. Another GCC compile warning fix. - onmainwindow.cpp: correctly use ~/.x2go/.ssh as ssh directory when starting sshd in user mode. Put the authorized_keys file in there. Check and set correct permissions for both the directory and authorized_keys file. Generalize some Windows-specific sections by using QDir and QFile. - compat.{cpp,h}: new files. Implements strndup on OS X 10.6 and below. Add to x2goclient.cpp, x2goclient.pro and x2goclient.pro.maemo. - compat.{cpp,h}: remove inline keyword, because function is not defined in header file. - compat.h: include QtCore/qglobal.h for Q_OS_... macros. - pulsemanager.{cpp,h}: new class for PulseAudio management. - x2goclient.pro{,.maemo}: reference new pulsemanager.{cpp,h} files. - macbuild.sh: add new MACPORTS_PREFIX detection/variable. - macbuild.sh: add new dependency_error() helper function. - macbuild.sh: add new lazy_canonical_path() helper function. - macbuild.sh: add new get_nesting_level() helper function. - macbuild.sh: add new repeat_str() helper function. - macbuild.sh: refactor nxproxy detection to use MACPORTS_PREFIX. - macbuild.sh: add EXE_DIR and FRAMEWORKS_DIR internal variables. Create directories based on that. Use them when bundling. - macbuild.sh: define PulseAudio libraries and binaries to be later copied. Discover them using MACPORTS_PREFIX. Error out if any component was not found. - macbuild.sh: copy PulseAudio libraries and binaries. - macbuild.sh: add phase output for bundling nxproxy. - macbuild.sh: add work-in-progress PulseAudio bundling. Needs to be ripped apart and be redone, because dylibbundler is not smart enough to handle our case. - macbuild.sh: add Linux library file name scheme regex as a precaution. - macbuild.sh: directories are "executable", so the first check must explicitly include a check for the argument being not a directory. - macbuild.sh: add some more debugging output. - macbuild.sh: remove libpulse and libpulsecore from libraries to be bundled, as something else will bundle them anyway. - macbuild.sh: use while-read-loop and find to actually recurse through directories. - macbuild.sh: check current file name against regex, not a (now) full path. - macbuild.sh: add the correct file path to the library bundling array. - macbuild.sh: when actually bundling libraries, use @executable_path instead of @loader_path. Will need testing. - macbuild.sh: fix echo call to also show the intermediate lib dir, not just the library file name. Helps debugging. - macbuild.sh: also install libraries into staging area with intermediate library path. Putting everything into a single place is a stupid idea. - deduplicate.sh: add new, more or less Proof of Concept, and Work in Progress file deduplication script. - deduplicate.sh: add duplicates-to-real mapping. - deduplicate.sh: add missed local variable declaration. - deduplicate.sh: new function for parsing otool's output and printing library dependencies if no error occurred. - deduplicate.sh: non-functional: change WARNING to ERROR in error messages. - deduplicate.sh: add first skeleton for library dependency rewriting of deleted duplicates. - deduplicate.sh: correctly unset an array element. - deduplicate.sh: change range-based for loops to "${!arr[@]}" to handle "sparse" arrays correctly. - deduplicate.sh: quote "${arr[@]}" correctly to not suddenly force word splitting. - deduplicate.sh: fix parse_otool_output: we want to return failure only iff any of the strings are present in otool's output and actually handle the failure string *ARRAY* correctly as that. - deduplicate.sh: change exit to return in parse_otool_output. - deduplicate.sh: record crafted "library path" values in duplicates replacement to_files array. - deduplicate.sh: debug parse_otool_output fiercely for the time being. - deduplicate.sh: prevent word splitting when parsing otool's output. We want the raw bits. - deduplicate.sh: switch to range-based for loop in fixup section for duplicate array walking as we need to access the same element index in the replacement array. - deduplicate.sh: add more useful information in fixup section's error message. - deduplicate.sh: unstub fixup section. - deduplicate.sh: parse_otool_output: jump over first matching entry, which is - hopefully - the id line. - deduplicate.sh: actually include file name in install_name_tool pseudo output. - deduplicate.sh: save dependency format base string as a readonly variable and use that instead of repeating a fixed string. - deduplicate.sh: copy lazy_canonical_path from macbuild.sh for now. deduplicate.sh will eventually be merged into macbuild.sh anyway. - deduplicate.sh: more temporary debugging output in fixup section. - deduplicate.sh: surprisingly, it turned out that only checking the duplicate's basename against the current dependency's basename is not good enough. Actually construct "the real dependency-format duplicate string" and check it against "the real dependency". Even this is still not good enough as it needs a bit cleanup - coming up next. - deduplicate.sh: cleanup as announced in the last commit - using lazy_canonical_path. - deduplicate.sh: remove a bit of noisy debug output, but turn on parse_otool_output debugging. - deduplicate.sh: fix wrong return value capture. We want to check the return value of the executed function/command, not that of "typeset". - deduplicate.sh: add WARNING to status message. - deduplicate.sh: remove some noisy debug output. - macbuild.sh: replace tabs with two spaces. No functional changes. - deduplicate.sh: replace tabs with two spaces. No functional changes. - macbuild.sh: merge deduplicate.sh content in. Also enable the functionality "for real", not just as a dry-run. - macbuild.sh: spelling fix. - macbuild.sh: more more debug messages and a whitespace change. - macbuild.sh: fix removal of base prefix in deduplication if base prefix ends in a slash. - macbuild.sh: copy "special" files, so that they can be removed later on. - macbuild.sh: rewrite ID line detection algorithm. Modules don't seem to be needing one, so skipping the first dependency line incidentally skips a "real" dependency for modules. - pulsemanager.{cpp,h}: use name and underscore for member variables, not underscore and name. - pulsemanager.cpp: minor non-behavior changing fixes. - pulsemanager.{cpp,h}: reformat only. - pulsemanager.cpp: use initializer list for constructor. - pulsemanager.{cpp,h}: add pulse_version_{major,minor}_ member variables. - pulsemanager.cpp: reformat only. - pulsemanager.{cpp,h}: switch port definitions to std::uint16_t. - pulsemanager.h: add std:: namespace selector for uint32_t version variables. - pulsemanager.{cpp,h}: add ESD support. - pulsemanager.{cpp,h}: rename relaunch () to restart (). - pulsemanager.cpp: reformat only. - pulsemanager.{cpp,h}: fixup preprocessor usage. - pulsemanager.{cpp,h}: add server_args_, server_binary_ and server_working_dir_ class variables. - pulsemanager.cpp: make start () wrap the "real" OS-specific start functions. - pulsemanager.{cpp,h}: new function create_client_dir (). - pulsemanager.cpp: add Windows support to shutdown (). - pulsemanager.cpp: add cleanup support to on_pulse_finished (). - pulsemanager.cpp: add Windows stuff to initial env in constructor. - pulsemanager.{cpp,h}: add new function start_generic () to split off common functionality. - pulsemanager.cpp: let start_osx () use start_generic (). - pulsemanager.{cpp,h}: implement start_win () as part of Windows functionality. - pulsemanager.cpp: also load module-esound-protocol-tcp module. - pulsemanager.cpp: make generate_server_config () Windows-compatible. - pulsemanager.cpp: make generate_client_config () Windows-compatible. - pulsemanager.cpp: only play startup sound if DEBUG macro is defined. - pulsemanager.cpp: make startup sound playing via slot_play_startup_sound () Windows-compatible. - pulsemanager.cpp: typo fix in variable name server_working_dir_. - pulsemanager.{cpp,h}: find_port () actually returns a value, fix declaration accordingly. - pulsemanager.cpp: fix compile errors in find_port () by renaming the ret or port variable to search_port. - pulsemanager.cpp: fix another compile error due to typo'd versions of generate_server_config () and generate_client_config (). - pulsemanager.cpp: another typo fix: findPort -> find_port. - pulsemanager.cpp: reorder member variables in initialization list. Compile warning fix. - pulsemanager.cpp: add "dummy" start_linux () function. - pulsemanager.cpp: generate platform-dependent values for member variables server_working_dir_ and server_binary_ in constructor. We need them right away. - pulsemanager.{cpp,h}: new member function fetch_pulseaudio_version () with first skeleton. Call it in the constructor. - pulsemanager.cpp: re-initialize buffer string on each run while getting CWD on Linux. - pulsemanager.cpp: fix some error messages by removing redundant newlines. - pulsemanager.{cpp,h}: some older PA versions used the major, minor and micro numbering scheme together with a descriptive string. Add new member variables for this and initialize them correctly. - pulsemanager.h: reformat a bit by adding newlines and moving lines in a (hopefully) logical fashion. - pulsemanager.cpp: add algorithm for actually extracting PA version number. - pulsemanager.{cpp,h}: only allow changing PA or ESD ports when server is not currently running. - pulsemanager.{cpp,h}: add record and playback private members, as well as getters and setters. - pulsemanager.cpp: reorder member variables in initializer list. - pulsemanager.cpp: fix compile error in startup sound function. - pulsemanager.h: more moving around. - pulsemanager.cpp: use record and playback member variables. - pulsemanager.cpp: cleanup, don't use copy assignment operator when we just need to construct a new object. - macbuild.sh: PulseAudio has been updated to version 8.0. Reflect this. - macbuild.sh: libpulsecore.dylib also resides in the "private" pulseaudio subdirectory now, adding it to "special files workaround". - pulsemanager.cpp: remove config.pa while doing post-exit cleanups. - src/pulsemanager.{cpp,h}: add get_pulse_dir () for fetching the (OS-dependent) PulseAudio config and cookie directory. - src/pulsemanager.cpp: replace hardcoded modules paths with dynamic ones depending on the detected PA version. - src/pulsemanager.cpp: add "FIXME" debug logging (and cleanup.) - src/pulsemanager.cpp: backport fix for 526. - src/pulsemanager.cpp: backport fix for 422. - src/configdialog.{cpp,h}: add "no record" support for OS X (through PulseManager.) - src/configdialog.cpp: whitespace only. - src/onmainwindow.{cpp,h}: hook PulseManager in - for both OS X and Windows. - src/mediawidget.cpp: reflect new capabilities through PulseManager. - src/pulsemanager.h: move x2gologdebug.h inclusion into implementation file. - src/pulsemanager.cpp: output PA stdout/stderr via x2goDebug, not plainly to stdout. - src/configdialog.cpp: fix compile error by removing obsolete (and now faulty) line for removed restart message. - src/onmainwindow.cpp: fix compile errors due to misspelled function names. - src/pulsemanager.cpp: let start () act as a wrapper and do nothing if the PA server is already running. - src/onmainwindow.cpp: fix compile error due to misuse of #ifdef. - src/onmainwindow.cpp: convert QDir object to QString for further processing. - src/onmainwindow.cpp: make use of new PulseManager::start () behavior and always use it, no matter whether server is up already or not. - src/pulsemanager.cpp: fix compile error due to typo. - src/pulsemanager.cpp: hide Windows-only code behind #ifdefs. Otherwise we'll get compile failures on non-Windows platforms. - src/pulsemanager.{cpp,h}: add debugging setter and private variable. - src/pulsemanager.{cpp,h}: port to new debugging feature. - src/onmainwindow.cpp: port to new debugging feature of PulseManager. - src/pulsemanager.h: fix compile error due to mismatching function declaration for set_debug (). - res/sound: add startup.wav sound to be played back when starting the PA daemon in debug mode. - src/pulsemanager.cpp: don't time out while trying to start PA or paplay. "Handle" errors by printing out debug messages. - macbuild.sh: add support for copying resource files. Currently only used for startup.wav. - res: rename "sound" to "audio". - macbuild.sh: fix copying of resource files. - macbuild.sh: also add PA binaries to deduplication fixup list. - macbuild.sh: don't error out while executing parse_otool_output (). - macbuild.sh: save return value of parse_otool_output (). Otherwise we'll fetch the return value of "set", which is not really what we need. - src/pulsemanager.cpp: fix typo in fetch_pulseaudio_version (). Read standard output from temporary process, not our "main" server process (which at this point is not even started yet.) - src/pulsemanager.cpp: don't overwrite the temporary string variable for the current line with what was supposed to hold new data in fetch_pulseaudio_version (). - src/pulsemanager.{cpp,h}: rename on_pulse_finished () to slot_on_pulse_finished (). Oops... - src/pulsemanager.cpp: actually only play startup sound if debugging is enabled. - src/pulsemanager.cpp: don't insert stray newline into config.pa. - src/pulsemanager.cpp: insert PULSE_SERVER and PULSE_COOKIE variables into environment as used by PulseManager. Otherwise we won't be able to start binaries like paplay. - src/pulsemanager.cpp: don't remove PA config and log file in ~/.x2go/pulse on PA shutdown if debugging has been requested. - src/x2goutils.{cpp,h}: add new function find_binary (). Searches for a specific binary in a colon-separated list of paths. Really just a compat function, as Qt 5 includes QStandardPaths::findExecutable (). - src/pulsemanager.cpp: auto-detect PA binary in $PATH and MacPorts default prefix as well on OS X. - src/pulsemanager.cpp: add comments to Qt 4 code part. - src/pulsemanager.cpp: add reminder for Linux-implementation in slot_play_startup_sound (). - src/x2goutils.cpp: fix compile error. - src/pulsemanager.h: add new system_pulse_ variable to indicate that a system PA binary shall be used. - src/pulsemanager.cpp: correctly initialize and set system_pulse_ when required. - src/x2goutils.{cpp,h}: add application modality parameter to show_RichText_Generic_MsgBox () and its wrappers. - src/pulsemanager.cpp: add error message boxes throughout PulseManager. Don't just abort () the program. - src/x2goutils.h: fix compile error due to typo. - src/pulsemanager.cpp: fix compile error on Windows. - src/pulsemanager.cpp: fix startup when using the system-PA version. - src/pulsemanager.cpp: fix startup sound playback when using the system-PA version. - src/pulsemanager.cpp: add warning message boxes where appropriate. - src/pulsemanager.cpp: remove extraneous newlines from debug output. - src/x2goutils.cpp: fix faulty logic in find_binary (): only reset the given path to CWD iff it's actually empty. Not the other way around. - src/x2goutils.cpp: add some debug logging to find_binary (). - nsis/x2goclient.nsi: add startup.wav to installer package. - src/: new file windows_stdint.h for Windows-compatibility when using std::(u)int*_t types. These are only available with GCC in C++11 mode on Windows for some reason. - src/pulsemanager.h: use windows_stdint.h header on Windows, cstdint otherwise. - src/onmainwindow.cpp: short out code parts unconditionally checking for the userSshd variable. We don't need it on Windows (and don't define it on purpose) because using an user-mode SSH daemon is implied there (for now.) - src/onmainwindow.cpp: drop obsolete pulseVersionTest variable initialization in Windows-only code. - src/onmainwindow.cpp: drop obsolete pulseVersionIsLegacy variable initialization in Windows-only code. - src/onmainwindow.cpp: add another pair of braces to in_addr-type variable initialization on Windows to silence compiler warning. - src/onmainwindow.cpp: fix compile error on Windows after moving some code around. - src/onmainwindow.cpp: use correct C++ universal initializer for initializing in_addr-type variable, the C-style universal zero initializer won't cut it here. - src/pulsemanager.h: include accidentally omitted wapi.h header file. - src/pulsemanager.cpp: make PA version parsing function more generic and don't expect that a number is always terminated with a period or dash. Especially on Windows, with a PA version number of "6.0", this lead to errors. OS X was fine with its "8.0.0-something" version number, but the optional part at the end or even the micro version number could be missing any time. - src/pulsemanager.cpp: make sure that wherever we interface with non-Qt functions, paths are in native form (i.e., containing empty separators, instead of the "generic" slash UNIX-style separator.) Should fix a PA startup issue on Windows. - src/pulsemanager.cpp: fix PA version fetching on OS X again. - src/unixhelper.{cpp,h}: make argument of kill_pgroup () const. - src/unixhelper.h: style and general fixes within comments. - src/unixhelper.{cpp,h}: split off core functionality of kill_pgroup () into a new function called real_kill_pgroup (). - src/unixhelper.{cpp,h}: rewrite kill_pgroup () to act as a wrapper around real_kill_pgroup (). Adjust comment. We need to do this so the cleanup process doesn't kill itself before all other processes in the process group are killed. - src/pulsemanager.cpp: remove unused variables. - src/unixhelper.cpp: code cleanup. - src/{unixhelper.cpp,x2goclient.cpp}: fix errno usage - save before use. - src/onmainwindow.cpp: document createRSAKey () because what it does is completely non-obvious. - src/onmainwindow.cpp: add support for explicitly setting the private key bit size based on type and use values (currently) regarded as secure to generateHostKey (). - src/onmainwindow.{cpp,h}: rename generateHostKey () to generateKey () and add a boolean parameter with a default of false to request a public key. Change old invocations of generateHostKey () accordingly. - src/onmainwindow.cpp: refactor generateKey () a little bit to compact it. - src/onmainwindow.{cpp,h}: rewrite generateKey () function to actually also generate user keys. Now returns the private key file path (public key file path is trivially determined by appending ".pub" to that return value.) Change all locations referencing generateKey () to use the return value (if necessary) and remove dead code that was used to generate a user key manually previously. - src/onmainwindow.cpp: add more error handling to generateKey () and fix up a broken error-handling section. - src/onmainwindow.cpp: fix up a few debug strings. - src/{onmainwindow.{cpp,h},configdialog.cpp}: remove userSshd variable and code related to non-user-mode-sshd setups. Simplifies code a bit and we want to always start a user-mode OpenSSH server anyway. Also get rid of the clientport variable that is now unnecessary. - src/onmainwindow.cpp: when starting sshd on Unix-based plattforms, raise maximum startup time to 5 seconds and break out early, if the process went into running state earlier. - src/onmainwindow.cpp: uppercase key type in debug output and comment string in generateKey (). - src/onmainwindow.cpp: terminate sshd more correctly. First via terminate (), then wait up to 5 seconds, then via kill (). - src/onmainwindow.{cpp,h}: add new function check_key_type (). Takes a key type and checks for validity/if it's known. - src/onmainwindow.{cpp,h}: add new function key_type_to_string (). Returns a stringified version of the selected key type, after checking for validity. - src/onmainwindow.{cpp,h}: add new function default_size_for_key_type (). Returns the default key size in bits for the selected key type, after checking for validity. - src/onmainwindow.cpp: use the new functions in generateKey (). - src/onmainwindow.h: move generateKey () declaration around. - src/onmainwindow.{cpp,h}: reformat generateKey () only. - src/onmainwindow.{cpp,h}: move createRSAKey () around. - src/onmainwindow.{cpp,h}: reformat createRSAKey () only. - src/onmainwindow.cpp: let startSshd () act as a wrapper and be called multiple times. Check if the OpenSSH Server is already running and return true, otherwise try to start it. - src/onmainwindow.cpp: add some comments only to createRSAKey (). - src/onmainwindow.{cpp,h}: rename createRSAKey () to the more-appropriate name createKeyBundle (). Also add a key-type parameter and use it to select the required key type (and maybe generate it if necessary.) Also use the new functions to get a stringified version of the key type etc. - macbuild.sh: switch to bundling PA 9.0. - src/onmainwindow.cpp: move QProcess startup check in startSshd () to non-Windows code section and add a listening socket check loop. Windows uses other means of starting the daemon. - src/onmainwindow.cpp: add comment to last all-in listening check in startSshd (). - src/onmainwindow.cpp: add listening-check loop to the Windows-specific code of startSshd (). - src/onmainwindow.{cpp,h}: make startSshd () private. There's no need to keep it public, as only class-internal functions are using it. - src/onmainwindow.{cpp,h}: add ssh_key_type_ member variable and getter and setter to WinServerStarter class. - src/onmainwindow.{cpp,h}: add SSH key type parameter to startSshd (). - src/onmainwindow.cpp: pass new ssh_key_type_ member to startSshd () call in WinServerStarter::run (). - src/onmainwindow.cpp: pass key_type parameter to startSshd () in createKeyBundle (). - src/onmainwindow.cpp: add and use key_types parameter to startWinServers (). - src/onmainwindow.cpp: actually use key_type parameter in startSshd () and make selection more generic. - src/onmainwindow.h: move WinServerStarter class definition around to hopefully get the definition of ONMainWindow::key_types. - src/onmainwindow.h: remove WWrapper definition, seems to be unused. - src/onmainwindow.{cpp,h}: don't try to initialize non-static ssh_key_type_ member variable directly, but in the constructor. Also, use the initializer list to initialize the other member variables directly. - src/onmainwindow.cpp: re-order initializer list of WinServerStarter constructor. - src/onmainwindow.h: actually, we do need startSshd () to be public. Move it back. - src/onmainwindow.cpp: only pass non-Cygwin paths to QFile::exists () in generateKey (). Refactor a bit to drop the now-unnecessary ret variable. - src/onmainwindow.cpp: use QFileInfo to get the key file's basename in exportDirs () instead of the old, potentially faulty string replace dance. - src/x2goutils.{cpp,h}: make find_binary () and add_to_path () available on all UNIX-based operating systems. We will need it for sftp-server discovery. - src/pulsemanager.cpp: refactor PA binary searching code. Less duplication now. - src/onmainwindow.cpp: fix sshd_config generation on UNIX by searching for the real sftp-server binary path. - src/pulsemanager.cpp: fix PA binary searching logic. We don't want to unconditionally set system_pulse_ to true if we found a binary... it might as well be the bundled one... - src/onmainwindow.cpp: move generateEtcFiles () call to startSshd (). We must make sure that the sshd_config file exists and is up-to-date, so let's always do it right before starting the sshd binary. - src/onmainwindow.cpp: stop duplicating the list of known sftp-server binary locations. - src/{onmainwindow,pulsemanager}.cpp: actually *use* what we got back from add_to_path ()... Fixes a few bugs here and there. - src/onmainwindow.cpp: remove trailing slash from Mageia/SUSE/Arch sftp-server binary search path. - src/x2goutils.cpp: add debugging output to add_to_path (). - src/x2goutils.cpp: fix add_to_path (). We want to remove trailing slashes from elements-to-add, not just fetch a trailing slash if encountered. Also, better don't compare a value from the original path value to itself, but rather to the things we want to add in order to remove duplicates, not remove most of the list of elements to add... - src/unixhelper.cpp: use cstdio instead of stdio.h. It's the C++ header and imports functions into the std namespace. - src/: create new file unix_stdint.h which imports the (u)int*_t types into the std namespace. - src/pulsemanager.h: use unix_stdint.h on non-Windows-platforms to get std::(u)int*_t types without having to use C++11. - src/pulsemanager.cpp: fix Linux-based code by fixing a really stupid typo ("erange" instead of "errno") and including the cerrno header. - src/pulsemanager.cpp: use std::ssize_t for the path length, because it could be -1 and std::size_t is not applicable. - src/pulsemanager.cpp: cast the return value of realloc () to char*, as GCC doesn't seem to like reassigning the pointer otherwise. - src/unix_stdint.h: use same hack to import ssize_t into std namespace. - src/onmainwindow.{cpp,h}: remove left-over parec references and code. - src/onmainwindow.cpp: convert piece of code to non-Windows and non-Darwin, so that not only Linux but other operating systems are covered as well. - src/mediawidget.cpp: unify some redundant code. - src/mediawidget.cpp: simplify expression. - src/mediawidget.cpp: add deprecation warning for ARTS. - src/mediawidget.cpp: add deprecation warning for ESounD. - macbuild.sh: make compatible with new llvm-based otool binary, which now uses proper return values. Be backwards-compatible with otool-classic. - res/i18n/: refresh translation files. - deduplicate.sh: remove file, now obsolete and merged into macbuild.sh. - res/i18n/: refresh translation files again after copyright date changes. - res/i18n/x2goclient_fr.ts: fixup French translation file. - res/i18n/x2goclient_sv.ts: fixup Swedish translation file a little bit and add comments/requests for re-translations for the next time. - macbuild.sh: switch to bundling PulseAudio 10.0 since it's now available in MacPorts. - src/x2goclient.cpp: no need to put new_argv_c_str std::vector onto the heap. Keep it on the stack. - src/x2goclient.cpp: use $PATH-exploration when re-executing x2goclient in UNIX cleanup helper mode. Fixes: #1139. - src/onmainwindow.cpp: correctly initialize pulseManager and pulseManagerThread member variables. Fixes crashes on OS X and Windows if X2Go Client terminates before these objects have been created at run time (for instance when running x2goclient --help.) - src/onmainwindow.cpp: correctly guard the new lines from the previous commit. Fixes compile issues on Linux and other systems. - res/i18n/x2goclient_es.ts: fixup Spanish translation file where necessary and add comments/requests for re-translations for the next time. - res/i18n/x2goclient_es.ts: remove obsolete comments and very minor whitespace fixup. - res/i18n/x2goclient_de.ts: fixup German translation file, including typo, whitespace, grammar fixes and other stuff. - res/i18n/x2goclient_sv.ts: remove obsolete comments and very minor whitespace fix. - macbuild.sh: sanitize find input, as BSD find and GNU find behave differently. - macbuild.sh: get rid of problems and use gfind directly. We need special GNU features. - src/appdialog.cpp: logic so that X2Go-Top applications are shown again. Used faulty logic when silencing compiler warnings. - res/i18n/x2goclient_fi.ts: fix up Finnish translation file, including typo, whitespace fixes and marking untranslated entries as such again, after removing the English "translation" text. . [ Bernard Cafarelli ] * New upstream version (4.1.0.0): - {res/,x2goclient.pro}: rename .rcc files to .qrc. Fixes: #1096. "rcc" is the extension for compiled resource files, whereas "qrc" is the mapping file. Qt5 stumbles upon that - i.e., it doesn't try to compile files with the "rcc" extension, while Qt4 did. - res/: fix up previous change, the renaming of changelog.qrc was not correctly committed. . [ Jason Alavaliant ] * New upstream version (4.1.0.0): - src/{onmainwindow.{cpp,h},help.cpp}: modify autostart option to support multiple autostart apps at the same time. Fixes: #1024. . [ Oleksandr Shneyder ] * New upstream version (4.1.0.0): - src/onmainwindow.cpp: fix SSHFS on Windows client. Fixes: #1105. . [ Sébastien Ducoulombier ] * New upstream version (4.1.0.0): - res/i18n/x2goclient_fr.ts: update French translation file. . [ Daniel Lindgren ] * New upstream version (4.1.0.0): - res/i18n/x2goclient_sv.ts: update Swedish translation file. - res/i18n/x2goclient_sv.ts: fixup Swedish translation file. . [ Ricardo Díaz Martín ] * New upstream release (4.1.0.0): - res/i18n/x2goclient_es.ts: update Spanish translation file. - res/i18n/x2goclient_es.ts: update Spanish translation file. . [ Stefan Baur ] * New upstream release (4.1.0.0): - res/i18n/x2goclient_de.ts: update German translation file. . [ Martti Pitkänen ] * New upstream version (4.1.0.0): - res/i18n/x2goclient_fi.ts: update Finnish translation file. ------------=_1487736911-17124-3 Content-Type: message/rfc822 Content-Disposition: inline Content-Transfer-Encoding: 7bit Received: (at submit) by bugs.x2go.org; 29 Nov 2016 04:55:51 +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.1 required=3.0 tests=BAYES_40,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,HTML_MESSAGE autolearn=ham version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 1FD065DA85 for ; Tue, 29 Nov 2016 05:55:50 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id S5-4YLrYia5f for ; Tue, 29 Nov 2016 05:55:32 +0100 (CET) Received: from mail-ua0-f174.google.com (mail-ua0-f174.google.com [209.85.217.174]) by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id CDA7F5DA82 for ; Tue, 29 Nov 2016 05:55:31 +0100 (CET) Received: by mail-ua0-f174.google.com with SMTP id 20so166853825uak.0 for ; Mon, 28 Nov 2016 20:55:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:from:date:message-id:subject:to; bh=w0gaWwV/DGc149H/oSdibbqRQOepp70H4aHSMif9zzQ=; b=jmOukeY7rIE25ge8NPQfgo//qQJw+tdGEpk3gSZOeQkwtRGVhITrr9Th5I2bcA8QF6 2gleBpNUOq24rTVkZBVhc6z/SYCkrFKSCM3DCTSA0WjKLdbmIKDHdMKFZ7xvrd6iL0KK KzN0AyzZlGIBBux2JvxQQWY49oeWWj37O8qwaKb89KB3Ra9pivJsdBX3QhgpEtfjgJj2 qhZKom4WcPDOhRnnzp4pALlPT3fBrYzfzWe4KnArMZidSeq4121er0gIIBqORdsl/gO/ +/wmaOCH2aWqWNLGARrYJZ+Ke956Tytj1R1NZQDJ565y5jC+vvwmjMCXB48qiA4Dutb8 gOEg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:from:date:message-id:subject:to; bh=w0gaWwV/DGc149H/oSdibbqRQOepp70H4aHSMif9zzQ=; b=LLpJ2KzvEMSTPPJ9e5cv73YHfJlDQSHbGsG/mit0bkO1wJnaPl31nvZOauc0JR2kWL V0crUryDJBOgm9qDqMNKrsxpRTG2pwKaUrQt6S61GPa4plQ34Zu7r19OH0YndoZhstSD 4b+TFh1CXK9T9sjX1fQRWGSVEbF3h8rSxbXHQYIg6uscbrrSpPqn32LkBjTVgOddjoK1 FcYi6AykbT1GCjK0vcs36ZqiOu01v3jullsm/uxN42EjGdrrqn5IWHvsxboea/GK5P3a 13LWu8j+ROrjIAco9tvxWYMQrMHVogDeEuJ+CwTerQbLDxeV/YGoFpx1XbZdP8AjZNvv L+Ag== X-Gm-Message-State: AKaTC02JX2XKC0mRYenBJW48NNszC+GAAKt9Wf4svlSDrrrLiK6zeExHOBuY/nKdMGJ4byXIYLrfs5ajVXSXeA== X-Received: by 10.176.17.18 with SMTP id e18mr19455640uab.19.1480395330546; Mon, 28 Nov 2016 20:55:30 -0800 (PST) MIME-Version: 1.0 Received: by 10.176.83.156 with HTTP; Mon, 28 Nov 2016 20:55:10 -0800 (PST) From: Michael Date: Tue, 29 Nov 2016 15:55:10 +1100 Message-ID: Subject: Share folder is broken for x2goclient-4.0.5.2-2016.09.20-setup.exe To: submit@bugs.x2go.org Content-Type: multipart/alternative; boundary=f403045e3b6ecdf1860542696537 --f403045e3b6ecdf1860542696537 Content-Type: text/plain; charset=UTF-8 Package: x2goclient-4.0.5.2-2016.09.20-setup Version: x2goclient-4.0.5.2-2016.09.20-setup Steps to reproduce: 1. Install x2goclient-4.0.5.2-2016.09.20-setup.exe 2. Make sure C:\Users\XXXXXXX\.x2go\etc does not exist, or delete it if existed. 3. Create a new session to a x2go server. 4. Enable share folder and watch the bug report window. >From the source code: 9931 void ONMainWindow::startWinServers() 9932 { 9952 generateHostKey(RSA_KEY_TYPE); For Windows, we only generate RSA_KEY, but in 8013 QString ONMainWindow::createRSAKey() 8014 { 8043 #ifdef Q_OS_WIN 8044 rsa.setFileName ( 8045 wapiShortFileName ( 8046 homeDir+"\\.x2go\\etc\\ssh_host_dsa_key.pub" ) ); 8056 if ( !rsa.open ( QIODevice::ReadOnly | QIODevice::Text ) ) 8057 { We check DSA_KEY instead, which of course would not exist. To fix this bug: 8046 - homeDir+"\\.x2go\\etc\\ssh_host_dsa_key.pub" ) ); 8046 + homeDir+"\\.x2go\\etc\\ssh_host_rsa_key.pub" ) ); --f403045e3b6ecdf1860542696537 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
Package: =
x2goclient-4.0.5.2-2016.09.20-setup
Version: x2goclient-4.0.5.2-2016.09.20-setup

Steps t=
o reproduce:
1. Install x2goclient-4.0.5.2-2016.09.20-se=
tup.exe

2. Make sure C:\Users\XXXXXXX\.x2go\etc does not exist, or d= elete it if existed.
3. Create a new session to a x2go server=
.
4. Enable share folder and watch the bug report window.
=
From the source code:

9931 void ONMainWindow::startWi= nServers()
9932 {
9952 generateHostKey(RSA_KEY_TYPE);
For Windows, we only generate RSA_KEY, but in

8013 QString ONMainWindow::createRSAKey= ()
8014 {
8043 #ifdef Q_OS_WIN
8044 rsa.setFileName (
8045 = wapiShortFileName (
8046 homeDir+"\\.x2go\\etc\= \ssh_host_dsa_key.pub" = ) );
8056 if ( !rsa.open ( QIODevice::ReadOnly | QIODevice::Text ) )=
8057 {


We check DSA_KEY instead, which of course would n= ot exist.
To fix this bug:

8046=C2=A0=C2=A0 - =C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0 homeDir+"\\.x2go\\etc\\ssh_host_dsa_key.pub&q= uot; ) );
8046=C2=A0=C2=A0 + =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = homeDir+"\\.x2go\\etc\\ssh_host_rsa_key.pub" ) );
--f403045e3b6ecdf1860542696537-- ------------=_1487736911-17124-3-- From ionic@ionic.de Sun Feb 26 06:02:20 2017 Received: (at control) by bugs.x2go.org; 26 Feb 2017 05:02:21 +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.7 required=3.0 tests=BAYES_20,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MISSING_SUBJECT,SPF_HELO_PASS,URIBL_BLOCKED autolearn=no version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 4747F5DBE3 for ; Sun, 26 Feb 2017 06:02:20 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ibExAaxu-P6H for ; Sun, 26 Feb 2017 06:02:12 +0100 (CET) Received: from Root24.de (powered.by.root24.eu [5.135.3.88]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 9C0C15DBE2 for ; Sun, 26 Feb 2017 06:02:12 +0100 (CET) Received: from nopileos.local (178.162.222.41.adsl.inet-telecom.org [178.162.222.41]) by mail.ionic.de (Postfix) with ESMTPSA id 1C87D4F0072E for ; Sun, 26 Feb 2017 06:02:12 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=ionic.de; s=default; t=1488085332; bh=xK1stWQLKGZHAjIA4s1CSigdGKJ52mZdvk764GHMkWw=; h=To:From:Date:From; b=W7wk/VKNaa7FBS5qyw/0Yj6txgh/fygiN3cfR0aX8WWgoSjz7+IHkT2Mt9LdFrQvq HId4m5JIoWwj33aFsNp+cc4RL92+XThGs5EIxF/2ak2x4olp8rFzr6fiYTdpvk/sKt btEoYVNBOwQrLYb9ufsiycSXar6Pu4AYQTp93jk8= To: control@bugs.x2go.org From: Mihai Moldovan Message-ID: <07e7f7f5-a86a-2b84-61e1-fe81c12507d2@ionic.de> Date: Sun, 26 Feb 2017 06:02:10 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="6cFLXPhxqfaMXMKCHmJflvx1gD6iXeDuG" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --6cFLXPhxqfaMXMKCHmJflvx1gD6iXeDuG Content-Type: multipart/mixed; boundary="K2o91J9qEnieeGcWhjwICGqaC4bevPgHv"; protected-headers="v1" From: Mihai Moldovan To: control@bugs.x2go.org Message-ID: <07e7f7f5-a86a-2b84-61e1-fe81c12507d2@ionic.de> --K2o91J9qEnieeGcWhjwICGqaC4bevPgHv Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable fixed 1110 4.1.0.0 thanks --K2o91J9qEnieeGcWhjwICGqaC4bevPgHv-- --6cFLXPhxqfaMXMKCHmJflvx1gD6iXeDuG Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBCgAGBQJYsmFTAAoJEB/WLtluJTqHJ1kP/3EWt6QNgvTNfC92J6qd9183 RFz3rynS1LJe7EPgpVxi0hgC/pjWaJRnY0dEO/Te+kJdf9p+Z4qMm3RddflL+qhW f/pQ9rte6c5+Jq4MH2ULTxmtvFJlDwVz+oRblSLuZE2XyVF8DhyYDkwvnbSr16OL dhSS03f90H2iQ7wxqkNim342/xItIGnA3ZsIavT6X+fMv1ZKji0Ivnr2Dmwdrzqn XmRGZ4HsUJ2LprZoUNLT4rTh+8C4cl/r2Xb2zS9VURVKSOCeiCGFrIfZ+9r0otOT XgbdZh3uFpQ/2afcpdQmcSy57eS1xiwljdGULteqPtUZnhCFotPjY5AGMIjpCCcj NmilfbcBYy+0aKh38ptbl2f2MmELEdFDLNQ90+4Ilyj6QzunEIz027dJZvAw6mLz t6pZqUcx37KMyo/5eDuRzgmlluG+BkLyaVXE4R7B/TxD69WIsy+A4FfI08pX9C3G dmxUvB8lW84QGNVN0w68Myy00ac4R4a8SfV7LPv1I+rcdE+lGBm2G0B9v7vZk76d mSkCxKzMiTfG6t0rpdVhvoV6JVGQcbrEe62SmJCQE6d/5pryiDZbDKOK0NlG9obv JhSJtAu4VVVX6VGMjUvtnPgKFTW+e5v72ZKYrOw3gMcFSerZdC+KYAOqfQvOkwAY /Zl0l4RxKp09SxXKhGis =wClQ -----END PGP SIGNATURE----- --6cFLXPhxqfaMXMKCHmJflvx1gD6iXeDuG-- From ionic@ionic.de Sun Feb 26 06:12:19 2017 Received: (at control) by bugs.x2go.org; 26 Feb 2017 05:12:20 +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.7 required=3.0 tests=BAYES_20,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MISSING_SUBJECT,SPF_HELO_PASS,URIBL_BLOCKED autolearn=no version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 63AB05DBE3 for ; Sun, 26 Feb 2017 06:12:19 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id BVEv9OYAbEcK for ; Sun, 26 Feb 2017 06:12:12 +0100 (CET) Received: from Root24.de (powered.by.root24.eu [5.135.3.88]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 53FE25DBE2 for ; Sun, 26 Feb 2017 06:12:12 +0100 (CET) Received: from nopileos.local (178.162.222.41.adsl.inet-telecom.org [178.162.222.41]) by mail.ionic.de (Postfix) with ESMTPSA id EFBAC4F00677 for ; Sun, 26 Feb 2017 06:12:11 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=ionic.de; s=default; t=1488085932; bh=a65hsLTmXlK9iBOEUCMLAQEjRZTHom7BvIUUZ9n5skc=; h=To:From:Date:From; b=s6+tGIW6s+oUQP/fDAy5Q49fI1G1UTC2KzJKL1yMnIDCD/KVK7/qI17X3eNv9h64m e+h6WA91DzniytbgCnBpokokyjQdvVCjiuLDnyW2XwlT3QFudVALAKRgq5OEYl1OLd +3rEKG9G9WHZJczb3uGFUMWGjNrDjRaEuHnQ1q0w= To: control@bugs.x2go.org From: Mihai Moldovan Message-ID: Date: Sun, 26 Feb 2017 06:12:10 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="rS7wgPWwGKFIlGVnQKKbkLcjBPF8Un6k6" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --rS7wgPWwGKFIlGVnQKKbkLcjBPF8Un6k6 Content-Type: multipart/mixed; boundary="DEaU2utlmHNC7X59repjXGJBMpdGmeuIb"; protected-headers="v1" From: Mihai Moldovan To: control@bugs.x2go.org Message-ID: --DEaU2utlmHNC7X59repjXGJBMpdGmeuIb Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable fixed 1110 -4.0.5.3 thanks --DEaU2utlmHNC7X59repjXGJBMpdGmeuIb-- --rS7wgPWwGKFIlGVnQKKbkLcjBPF8Un6k6 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBCgAGBQJYsmOqAAoJEB/WLtluJTqHdzMQAMNSkcskQh/AUpxNwxONM7Qu mb9gc96RAltXOa2S46EVM6+ild4I1w3YpqxCTdzauwRqo0oc7DWcvgxSmaF1dgjL ol1LraNA8Tq1nsxi0JWxdCmM0kjkTHwZ2XFF6FL/F/F9kudJzV5wH7ayeAWRsviJ CANxEekWWNKroTdQsMFHJ784F3Qb/ysKOjIqwHSw/vnYNNvihYcSwGlcBAqm0BMQ 4mf/2JRUKApLB3KbFQUXVI/idusDTajOo7MuYAxkZTprOQDKZmIXCi9pHIacL0oA WQo5TljnX+ue6RjiLBz7ydHQRbQeBTdH00LzIG3BgZixRr/wM4Tt7y/H9mlFr42G D5iWbol/0cdfHWK1xjYLaVx+8a2JzBVLksIkQhR+37DnI25ZoiSFjoLSe7RcG9bM 8wFDttEPHwvOSb8+FJ74vZmK5VHZhguK4kBX0nezqcukh8Wz2agfkZ13kMTUsINa 4yDBi7Wy1/Ak9F5e3Kc7GfcNB4dKcpfdmrJdDc36lrawVlz9qH0uzujr0uyD5uOn 1PkXxV1JRJpYx0YMXo6H9OlnaKLzSr5yrlayp9pctpWpTa1p2AhV019BYW2eVRGe 9IXX7KJzQ/7OcM3iRJDTMp8JdBplMa6POFWnwrOEDpXMrto+ZyLSNN0k0ndlrjwD xWWg6CrdsWJ9eSjTvcDD =uqoG -----END PGP SIGNATURE----- --rS7wgPWwGKFIlGVnQKKbkLcjBPF8Un6k6-- From ionic@ionic.de Tue Mar 7 11:34:35 2017 Received: (at control) by bugs.x2go.org; 7 Mar 2017 10:34:36 +0000 X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on ymir.das-netzwerkteam.de X-Spam-Level: * X-Spam-Status: No, score=1.7 required=3.0 tests=BAYES_40,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MISSING_SUBJECT,SPF_HELO_PASS,TVD_SPACE_RATIO autolearn=no autolearn_force=no version=3.4.1 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 073335DA85 for ; Tue, 7 Mar 2017 11:34:35 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id wLf_-yG9TAJu for ; Tue, 7 Mar 2017 11:34:28 +0100 (CET) Received: from Root24.de (powered.by.root24.eu [5.135.3.88]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 36F2C5DA2B for ; Tue, 7 Mar 2017 11:34:28 +0100 (CET) Received: from nopileos.local (178.162.222.41.adsl.inet-telecom.org [178.162.222.41]) by mail.ionic.de (Postfix) with ESMTPSA id 96FE24F0063E for ; Tue, 7 Mar 2017 11:34:27 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=ionic.de; s=default; t=1488882868; bh=05ZJXVvdgagNTQEtanMhxHrq5Oa1DQUJDfArnjgkWTc=; h=From:To:Date:From; b=G1rZKbosKRug9yhrMDDz2okXO7YNUxjakBqLiGff10n5RQw0DxhEga+PjWjdPOfPU WRSQiPmLQut3/T0xSaLTSXJXA4F9T4X2GgggfURZ6uYVR/Nnq0FY2nARTF/xkPl/jV 7yRUrN/auXwwIJu8DjoN5v46IN64v5ITntonRj18= From: Mihai Moldovan To: control@bugs.x2go.org Message-ID: Date: Tue, 7 Mar 2017 11:34:24 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="428xwwSF4fHWoUFTsEpUj8tXfF0W8aGOJ" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --428xwwSF4fHWoUFTsEpUj8tXfF0W8aGOJ Content-Type: multipart/mixed; boundary="Nhtcd9xoff9i0P3biWE6uI7iAi8E08cbp"; protected-headers="v1" From: Mihai Moldovan To: control@bugs.x2go.org Message-ID: --Nhtcd9xoff9i0P3biWE6uI7iAi8E08cbp Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable notfixed 1110 -4.0.5.3,4.0.5.3 thanks --Nhtcd9xoff9i0P3biWE6uI7iAi8E08cbp-- --428xwwSF4fHWoUFTsEpUj8tXfF0W8aGOJ Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBCgAGBQJYvoyxAAoJEB/WLtluJTqHaaIP/3wZcdCR/G+942UCVgDzESSk UEmlvx2dXLYa4NfKomkU0K9A/dO9w9hU8kPkvo4vXAA7m+FfZeOzYZYREknQ80er Bub0e+EDgYsrz+JRCeesdMbSVTq7mfXEluL2DQCOZTFICJkq7Fe5nkOr95tL3D5f n9MKyw8NwDUHt07mX2l/hiIedHzLO/b0gBsEowFnTHqhGXVxQAr/bqF+4coKUb2b b1L84jeVUBec0mq/hzT5hAITN2TWYldT3e8VRVXjgXzcfbu6lXpjNOSFeAEgW0Hu 0Fj04AEAQ1kORmfFb7zAY1BnYq7/0xBhrMFiPnlklRcViwvfYup9i/SCVVDYApQA Z1YcHuTF9Qi2YGBzgyitW70mnJKMpt+bINTXy8T3IeXHDtxBj4lREKDtUA3kuvVj o7wbcHk6qn+52zbtilhV5Vs+p+ddhyLBrxxuks69n4xH5kNHtpplR8B5LDQbNY1S 81j6mqvw1q4Tv/1q8sB4QbSR9aHNVNPLIfEHSa46RQVS1O3FIkERoRroHrQukyC7 ugU/xmRyFWaRFNpm4jIjNyVL4BYSvdle+xzz3L6FSQvEjBVkWEUZhfi//Zc3mUHy L/jnmRzLpS2qaH1DmTlnL1lCCK/AZjw9UC7/m+E/TClk2E2Xn3+D54AbH2fbo3kv z+jQDtu+8qkKc6OfLD/q =8LJl -----END PGP SIGNATURE----- --428xwwSF4fHWoUFTsEpUj8tXfF0W8aGOJ--