From unknown Thu Mar 28 22:11:53 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#541 closed by Stefan Baur () Message-ID: References: <54AEA1D6.3060508@baur-itcs.de> X-X2go-PR-Message: they-closed 541 X-X2go-PR-Package: x2goclient X-X2go-PR-Source: x2goclient Date: Thu, 08 Jan 2015 15:30:03 +0000 Content-Type: multipart/mixed; boundary="----------=_1420731003-19991-0" This is a multi-part message in MIME format... ------------=_1420731003-19991-0 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=utf-8 This is an automatic notification regarding your Bug report which was filed against the x2goclient package: #541: Patch to close Bug #525 has undesirable side effects It has been closed by Stefan Baur . 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 Stefan Baur by replying to this email. --=20 X2Go Bug Tracking System Contact owner@bugs.x2go.org with problems ------------=_1420731003-19991-0 Content-Type: message/rfc822 Content-Disposition: inline Content-Transfer-Encoding: 7bit Received: (at control) by bugs.x2go.org; 8 Jan 2015 15:26:16 +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=5.0 tests=BAYES_00,MISSING_SUBJECT autolearn=no version=3.3.2 Received: from mout.kundenserver.de (mout.kundenserver.de [212.227.17.13]) by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id 09E5C5DB53 for ; Thu, 8 Jan 2015 16:26:16 +0100 (CET) Received: from [192.168.0.3] ([188.105.114.135]) by mrelayeu.kundenserver.de (mreue102) with ESMTPSA (Nemesis) id 0Lp7GG-1XdE4R2bCe-00eqtp for ; Thu, 08 Jan 2015 16:26:15 +0100 Message-ID: <54AEA1D6.3060508@baur-itcs.de> Date: Thu, 08 Jan 2015 16:27:18 +0100 From: Stefan Baur User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: control@bugs.x2go.org Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Provags-ID: V03:K0:GYlIKJW85lwQ4aBFqA/7dZDZBUfH+omJjKvPRi9nz/fCOPYzvJC GucJPYJPmHAqmQKZNinLzbiO6SL5x5nhHF+vwz9kX+fMevqZKcMbbHb6gNwreyukXXUb/wA Op259WlvSEqdJGc48wWTADMdQZkL2GGO+3jWIA0SMMsb555mBIustfHVyWvny55a0DInDDz CLQaA/caxmatPaFrKLNWg== X-UI-Out-Filterresults: notjunk:1; -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 close 541 thanks - -- BAUR-ITCS UG (haftungsbeschränkt) Geschäftsführer: Stefan Baur Eichenäckerweg 10, 89081 Ulm | Registergericht Ulm, HRB 724364 Fon/Fax 0731 40 34 66-36/-35 | USt-IdNr.: DE268653243 -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (MingW32) iQEcBAEBAgAGBQJUrqHWAAoJEG7d9BjNvlEZPY0IAJfRrBIKtnMbDcm02eSNM6KO AE29vjbYerZZ5t3M1A8i6Nifl5P3FLoqFFh/PFt9bvXFiDTo4v4kBgQrdLe0PzVG aCEIUh/EXqzFjBk5UcC9E92XaASAP3hjeKaShocygMbg/RV9LOrqdLD/gPJknUz3 gkka5+vxKWY/633V2S7A2lSce52cZBO0vZd+VDJSPTuYOK6gWZJs2jpGsj2Apjrp zhHOjj/51KV9FyDLze+gYoGhHomF5RfAXPtM/Qe8p+YjJwBN2aDWcUJwSXQzjlTi 3lqz2pY29DpVA0M+9UW3RoSwDHCgSoPrLorJDcedVCq3KOL0P/Pb+lLDcT+ux7I= =2I+7 -----END PGP SIGNATURE----- ------------=_1420731003-19991-0 Content-Type: message/rfc822 Content-Disposition: inline Content-Transfer-Encoding: 7bit Received: (at submit) by bugs.x2go.org; 8 Jul 2014 22:19: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=-1.9 required=5.0 tests=BAYES_00 autolearn=ham version=3.3.2 Received: from mout.kundenserver.de (mout.kundenserver.de [212.227.17.13]) by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id E520F5DB24 for ; Wed, 9 Jul 2014 00:19:43 +0200 (CEST) Received: from appserv1.stefanbaur.home (dslb-188-105-141-120.pools.arcor-ip.net [188.105.141.120]) by mrelayeu.kundenserver.de (node=mreue102) with ESMTP (Nemesis) id 0LtX6a-1WdAiZ1NEi-010uuQ; Wed, 09 Jul 2014 00:19:43 +0200 Message-ID: <53BC6E79.1030906@stefanbaur.de> Date: Wed, 09 Jul 2014 00:19:37 +0200 From: Stefan Baur User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Icedove/24.6.0 MIME-Version: 1.0 To: submit@bugs.x2go.org Subject: Patch to close Bug #525 has undesirable side effects Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit X-Provags-ID: V02:K0:taIeLHQWDAC3YFXphUxwQtmj1YNt2nR9uembHP0UPvW ezG4fOZZpmfwqPNXsuSnRTT4V/aucaomXLVXXN0bM2j1UzS/a4 SPpp+GL8XMZS9AsL9W8kF5FacKZcfydPENeGYqvF3fA5pEOY/2 uk7Io105vFwOTjlGlLRVoOuW1DHibf4qlyjcclvNlVYo2XCNUm UwYnlJ4YSkAZUx+3QZgb6LmSCNd164/Y0ZTyAgXbwBQ+8wbtYu EW2wV650RUrVDtBv0gP7vqIdEqV7fWOeapmd/6sJ+AamVpbhPT bjzf8jYTHGHN4Lh5TwEWbYGx+3TAsiCWZSi9dc1vieYKveAsM4 aqfD7Eb7XgNPct8pc4RgLJqVbC3jXzeDJ5oEDt57/ZotdZ4OyS VBUhC3c1IaLeQ== Package: x2goclient Version: 4.0.2.1 Severity: grave Hi, Alex' latest patches introduce a new and more severe bug. His idea, assuming that Mike#1. Mike#2, Mihai and me understood him correctly, is, that on Windows, users shouldn't start x2goclient.exe any more, but rather the new x2gohelper.exe. ==>Alex, if we're misunderstanding what you wrote, please reply and explain your intentions. Assuming we're right, your plan breaks backward compatibility when it comes to desktop shortcuts on Windows, in two places. The smaller issue is that clicking the arrow on the session tile will create a desktop shortcut that is hardcoded to x2goclient.exe instead of x2gohelper.exe. This could be fixed with a small patch. However ... The bigger issue is that *all* existing links users may have on their desktops are broken, since they point to x2goclient.exe and can't easily be changed! I have at least one customer that makes heavy use of that feature, where it would mean manually updating over 600 desktop shortcuts. That is not going to be fun! We need to find a way to achieve what you were trying to achieve without breaking backward compatibility like that. >From what I understand so far, you're trying to build a "reaper" to clean up after a dying x2golient.exe, which may have left pulseaudio, xserver and ssh daemon running. I haven't tried to figure out what your current x2gohelper.exe is trying to do, and if you made changes to x2goclient.exe as well where functionality from x2goclient.exe is moved to x2gohelper.exe. If you moved it, rather than duplicating it, that's not going to help, because it might just as well be x2gohelper.exe that dies first and you'd again leave sshd, pulseaudio, and xserver dangling. Duplicating it might work, similar to how certain malware on Windows works (two processes checking on each other and re-spawning whichever gets killed first). So you'd have x2goclient.exe on the one hand permanently checking if x2gohelper.exe is running, respawning it when necessary, and upon shutdown, "trying" to clean up behind itself by terminating sshd, pulseaudio, xserver, and finally itself. While x2gohelper.exe would be checking if x2goclient.exe is still running - if not, it will terminate sshd, pulseaudio, xserver, (if any of them are still running) and finally itself. Before you start coding, I'd like to ask this question, though: Do we really want to have a reaper mechanism that tries to clean up after a malfunctioning x2goclient.exe, or wouldn't it be better to find out why x2goclient.exe is malfunctioning, fix these issues / catch those exceptions and deal with them appropriately (i.e. properly shutting down sshd, pulseaudio and xserver before going belly-up)? The latter would mean we need more bug reports and it requires analyzing what exactly went wrong, but it would be less disturbing to x2goclient development and roll-out. Whichever path you choose, Alex, please do keep in mind that we have a release planned in the near future and your latest commits aren't exactly helpful in getting 4.0.2.1 stabilized (and I *need* 4.0.2.1 out soon, my customers are desperately waiting for it because of the improved sound that PA5 brings). It would be really really cool if you could either do your development on a separate branch that could later become 4.0.2.2, or hold back until we have released 4.0.2.1. If you don't know how to do create your own branch for development, I'm sure there is more than just one Dev on the list that will help you. Kind Regards, Stefan ------------=_1420731003-19991-0--