X2Go Bug report logs - #1393
sftp-server binary not found gentoo (profile 17.1)

Toggle useless messages

Message #5 received at submit@bugs.x2go.org (full text, mbox, reply):

From: Tobias Ehrig <me@t0by.eu>

To: submit@bugs.x2go.org

Subject: sftp-server binary not found gentoo (profile 17.1)

Date: Tue, 18 Jun 2019 09:12:48 +0200

Package: net-misc/x2goclient
Version: 4.1.2.1

On Gentoo (eselect profile 17.1) the x2goclient can't find the binary
sftp-server which is located at

/usr/lib64/misc/sftp-server


Here is a patch

diff --git a/x2goclient-4.1.2.1/src/onmainwindow.cpp b/x2goclient-
4.1.2.1/src/onmainwindow.cpp
index 405b124..16eeeb5 100644
--- a/x2goclient-4.1.2.1/src/onmainwindow.cpp
+++ b/x2goclient-4.1.2.1/src/onmainwindow.cpp
@@ -10444,6 +10444,7 @@ void ONMainWindow::generateEtcFiles()
                      << "/usr/lib/ssh" /* Mageia, OpenSUSE, SLE{S,D} <
12 x86, SLE{S,D} 12, Arch */
                      << "/usr/lib64/ssh" /* SLE{S,D} < 12 x86_64 */
                      << "/usr/lib/misc" /* Gentoo */
+                     << "/usr/lib64/misc" /* Gentoo 17.1 x64 */
                      << "/usr/libexec"; /* Slackware, OS X */
 
 #if QT_VERSION < 0x050000

Message #10 received at 1393@bugs.x2go.org (full text, mbox, reply):

From: Mihai Moldovan <ionic@ionic.de>

To: Tobias Ehrig <me@t0by.eu>, 1393@bugs.x2go.org

Subject: Re: [X2Go-Dev] Bug#1393: sftp-server binary not found gentoo (profile 17.1)

Date: Tue, 18 Jun 2019 11:04:31 +0200

[Message part 1 (text/plain, inline)]

Control: reassign -1 x2goclient

* On 6/18/19 9:12 AM, Tobias Ehrig wrote:
> Package: net-misc/x2goclient
> Version: 4.1.2.1
> 
> On Gentoo (eselect profile 17.1) the x2goclient can't find the binary
> sftp-server which is located at
> 
> /usr/lib64/misc/sftp-server
> 
> 
> Here is a patch

I was inclined to NAK this at first.

/usr/lib on a multilib Gentoo system is typically a symlink to /usr/lib64, while
on non-multilib systems, as far as I remember, it's just a normal directory
containing all library files.

The entry containing /usr/lib/misc should hence cover your use case as well. Did
the 17.1 profile remove the /usr/lib symlink? I'm currently still on 17.0, but
it looks like the whole point of 17.1 is to remove the compat symlink, so you
seem to be on the right track.


I think I'll apply a slightly modified patch that favors /usr/lib64/misc instead
(mostly because that's the more common arch now, we want to avoid calling 32-bit
binaries on 64-bit systems even if they are installed and it's highly unlikely
to have a 32-64-bit multilib system [i.e., main system 32-bit, with 64-bit as an
additional architecture]).



Mihai

[signature.asc (application/pgp-signature, attachment)]

Message #21 received at 1393@bugs.x2go.org (full text, mbox, reply):

From: Mihai Moldovan <ionic@ionic.de>

To: 1393-submitter@bugs.x2go.org

Cc: control@bugs.x2go.org, 1393@bugs.x2go.org

Subject: X2Go issue (in src:x2goclient) has been marked as pending for release

Date: Tue, 18 Jun 2019 11:39:35 +0200 (CEST)

tag #1393 pending
fixed #1393 4.1.2.2
thanks

Hello,

X2Go issue #1393 (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=567223f

The issue will most likely be fixed in src:x2goclient (4.1.2.2).

light+love
X2Go Git Admin (on behalf of the sender of this mail)

---
commit 567223f5b55fb0398322f01cb0f158da6eb2a524
Author: Mihai Moldovan <ionic@ionic.de>
Date:   Tue Jun 18 11:38:29 2019 +0200

    src/onmainwindow.cpp: add (and prefer) non-compat-symlink scp server location for 64-bit-based Gentoo distros (17.1+ profiles).
    
    Fixes: #1393.

diff --git a/debian/changelog b/debian/changelog
index 284692b..ed1b3ab 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -102,6 +102,9 @@ x2goclient (4.1.2.2-0x2go1) UNRELEASED; urgency=medium
       version available in EPEL 6.
     - src/onmainwindow.cpp: unbreak builds by fixing syntax error.
     - src/onmainwindow.cpp: also unbreak old EPEL 6 builds.
+    - src/onmainwindow.cpp: add (and prefer) non-compat-symlink scp server
+      location for 64-bit-based Gentoo distros (17.1+ profiles). Fixes:
+      #1393.
   * debian/control:
     + Add build-depend on pkg-config.
   * x2goclient.spec:

Message #33 received at 1393@bugs.x2go.org (full text, mbox, reply):

From: Lars Wendler <polynomial-c@gentoo.org>

To: Mihai Moldovan <ionic@ionic.de>

Cc: 1393@bugs.x2go.org, Tobias Ehrig <me@t0by.eu>, voyageur@gentoo.org

Subject: Re: [X2Go-Dev] Bug#1393: Bug#1393: sftp-server binary not found gentoo (profile 17.1)

Date: Tue, 18 Jun 2019 15:27:11 +0200

[Message part 1 (text/plain, inline)]

Hello,

On Tue, 18 Jun 2019 11:04:31 +0200 Mihai Moldovan wrote:

>Control: reassign -1 x2goclient
>
>* On 6/18/19 9:12 AM, Tobias Ehrig wrote:
>> Package: net-misc/x2goclient
>> Version: 4.1.2.1
>> 
>> On Gentoo (eselect profile 17.1) the x2goclient can't find the binary
>> sftp-server which is located at
>> 
>> /usr/lib64/misc/sftp-server
>> 
>> 
>> Here is a patch  
>
>I was inclined to NAK this at first.
>
>/usr/lib on a multilib Gentoo system is typically a symlink
>to /usr/lib64, while on non-multilib systems, as far as I remember,
>it's just a normal directory containing all library files.

IMHO you should NAK such a patch as this is definitely an issue coming
from our 17.1 profile's new lib directory structure.
If you commit the suggested patch, all 32bit 17.1 profile installations
will suffer from the bug our 64bit 17.1 profile installations currently
suffer from.


>The entry containing /usr/lib/misc should hence cover your use case as
>well. Did the 17.1 profile remove the /usr/lib symlink? I'm currently
>still on 17.0, but it looks like the whole point of 17.1 is to remove
>the compat symlink, so you seem to be on the right track.

We changed the following directories on 64bit arches with the 17.1
profile:

17.0:

# ls -ld /lib*
lrwxrwxrwx 1 root root    5 May 17 10:24 /lib -> lib64
drwxr-xr-x 1 root root 1174 May 17 10:26 /lib32
drwxr-xr-x 1 root root 5078 Jun 13 18:50 /lib64


17.1:

# ls -ld /lib*
drwxr-xr-x 1 root root 1382 May 13 22:54 /lib
drwxr-xr-x 1 root root 5340 Jun 18 10:23 /lib64


As you can see, "lib32" became "lib" and "lib" is no longer a symlink
pointing to "lib64".

My suggestion is to let the Gentoo package maintainers handle this in
the affected x2go package's ebuild files. We have a function called
"get_libdir" that can be used with sed to set the correct lib dir
name in the affected code. 

>I think I'll apply a slightly modified patch that
>favors /usr/lib64/misc instead (mostly because that's the more common
>arch now, we want to avoid calling 32-bit binaries on 64-bit systems
>even if they are installed and it's highly unlikely to have a
>32-64-bit multilib system [i.e., main system 32-bit, with 64-bit as an
>additional architecture]).
>
>
>
>Mihai
>

Kind regards

-- 
Lars Wendler
Gentoo package maintainer
GPG: 21CC CF02 4586 0A07 ED93  9F68 498F E765 960E 9B39

[Message part 2 (application/pgp-signature, inline)]

Message #38 received at 1393@bugs.x2go.org (full text, mbox, reply):

From: Mihai Moldovan <ionic@ionic.de>

To: Lars Wendler <polynomial-c@gentoo.org>

Cc: 1393@bugs.x2go.org, Tobias Ehrig <me@t0by.eu>, voyageur@gentoo.org

Subject: Re: Bug#1393: sftp-server binary not found gentoo (profile 17.1)

Date: Wed, 19 Jun 2019 12:52:18 +0200

[Message part 1 (text/plain, inline)]

* On 6/18/19 3:27 PM, Lars Wendler wrote:
> IMHO you should NAK such a patch as this is definitely an issue coming
> from our 17.1 profile's new lib directory structure.
> If you commit the suggested patch, all 32bit 17.1 profile installations
> will suffer from the bug our 64bit 17.1 profile installations currently
> suffer from.

Can you elaborate on that, please? I understand that there's potential for
breakage with a compat symlink. One example would be software that is not
correctly trimmed to using lib32 instead of lib and hence trying to use the
wrong, 64-bit, directory at compile time, unless additional patching is done.

I fail to see a lot of breakage potential with the current patch I applied that
favors lib64. Let's create a matrix:

 System | 32-bit OpenSSH installed | 64-bit OpenSSH installed | [G]ood/[B]ad
--------+--------------------------+--------------------------+---------------
{32}    |           X              |             -            |       G
{32}    |           -              |             X            | B (nonsense)
{32}    |           X              |             X            | B (nonsense?)
{64}    |           X              |             -            | B (nonsense)
{64}    |           -              |             X            |       G
{64}    |           X              |             X            | G (nonsense?)
{32,64} |           X              |             -            |       G
{32,64} |           -              |             X            |       B
{64,32} |           X              |             -            |       B
{64,32} |           -              |             X            |       G
{64,32} |           X              |             X            | G (nonsense?)

The failure cases are pretty rare and mostly boil down to users messing up their
system - for instance, why would anyone install a 64-bit version of OpenSSH on a
32-bit non-multilib system? The only cases that would be bad is having a 32-bit
OpenSSH version installed on a {64,32} multilib system or a 64-bit OpenSSH
version on a {32,64} multilib system - i.e., when the installed OpenSSH version
does *not* match the primary arch. These cases should be very rare and indicate
a hosed system to begin with.

Additionally, none of these failure cases are actually fatal. A 32-bit OpenSSH
sftp-server binary should work fine on a 64-bit system. As long as we actually
find a binary, we should be mostly good to go.


> We changed the following directories on 64bit arches with the 17.1
> profile:
> 
> 17.0:
> 
> # ls -ld /lib*
> lrwxrwxrwx 1 root root    5 May 17 10:24 /lib -> lib64
> drwxr-xr-x 1 root root 1174 May 17 10:26 /lib32
> drwxr-xr-x 1 root root 5078 Jun 13 18:50 /lib64
> 
> 
> 17.1:
> 
> # ls -ld /lib*
> drwxr-xr-x 1 root root 1382 May 13 22:54 /lib
> drwxr-xr-x 1 root root 5340 Jun 18 10:23 /lib64
> 
> 
> As you can see, "lib32" became "lib" and "lib" is no longer a symlink
> pointing to "lib64".

Hmm, that's just nitpicking maybe and too late for any change now that the
profile went on to be stable, but I don't get why lib32 was renamed to lib.
Wouldn't it have been better to just keep lib32 and lib64 around (which
describes both directories's purpose vividly) and just remove the compat
symlink? Just using "lib" is ambiguous and can be confusing.


> My suggestion is to let the Gentoo package maintainers handle this in
> the affected x2go package's ebuild files. We have a function called
> "get_libdir" that can be used with sed to set the correct lib dir
> name in the affected code.

I could implement a Makefile variable that is being passed down via CFLAGS (no
sed hackery) and add it as the first option. I'd like to keep the fallback list,
though. Not sure if that would actually be really useful to anyone, though.



Mihai

[0] {primary arch,secondary arch}

[signature.asc (application/pgp-signature, attachment)]

Message #43 received at 1393@bugs.x2go.org (full text, mbox, reply):

From: Lars Wendler <polynomial-c@gentoo.org>

To: Mihai Moldovan <ionic@ionic.de>

Cc: 1393@bugs.x2go.org, Tobias Ehrig <me@t0by.eu>, voyageur@gentoo.org

Subject: Re: Bug#1393: sftp-server binary not found gentoo (profile 17.1)

Date: Wed, 19 Jun 2019 13:30:16 +0200

[Message part 1 (text/plain, inline)]

On Wed, 19 Jun 2019 12:52:18 +0200 Mihai Moldovan wrote:

>* On 6/18/19 3:27 PM, Lars Wendler wrote:
>> IMHO you should NAK such a patch as this is definitely an issue
>> coming from our 17.1 profile's new lib directory structure.
>> If you commit the suggested patch, all 32bit 17.1 profile
>> installations will suffer from the bug our 64bit 17.1 profile
>> installations currently suffer from.  
>
>Can you elaborate on that, please? I understand that there's potential
>for breakage with a compat symlink. One example would be software that
>is not correctly trimmed to using lib32 instead of lib and hence
>trying to use the wrong, 64-bit, directory at compile time, unless
>additional patching is done.

I was reading the suggested patch wrong. I thought it
replaces /usr/lib/misc with /usr/lib64/misc bit I just saw that the
latter path was rather added to the list.

>I fail to see a lot of breakage potential with the current patch I
>applied that favors lib64. Let's create a matrix:
>
> System | 32-bit OpenSSH installed | 64-bit OpenSSH installed |
> [G]ood/[B]ad
>--------+--------------------------+--------------------------+---------------
>{32}    |           X              |             -            |       G
>{32}    |           -              |             X            | B
>(nonsense) {32}    |           X              |
>X            | B (nonsense?) {64}    |           X
>|             -            | B (nonsense) {64}    |
>-              |             X            |       G {64}
>|           X              |             X            | G (nonsense?)
>{32,64} |           X              |             -            |
>G {32,64} |           -              |             X
>|       B {64,32} |           X              |
>-            |       B {64,32} |           -
>|             X            |       G {64,32} |
>X              |             X            | G (nonsense?)
>
>The failure cases are pretty rare and mostly boil down to users
>messing up their system - for instance, why would anyone install a
>64-bit version of OpenSSH on a 32-bit non-multilib system? The only
>cases that would be bad is having a 32-bit OpenSSH version installed
>on a {64,32} multilib system or a 64-bit OpenSSH version on a {32,64}
>multilib system - i.e., when the installed OpenSSH version does *not*
>match the primary arch. These cases should be very rare and indicate a
>hosed system to begin with.
>
>Additionally, none of these failure cases are actually fatal. A 32-bit
>OpenSSH sftp-server binary should work fine on a 64-bit system. As
>long as we actually find a binary, we should be mostly good to go.
>
>
>> We changed the following directories on 64bit arches with the 17.1
>> profile:
>> 
>> 17.0:
>> 
>> # ls -ld /lib*
>> lrwxrwxrwx 1 root root    5 May 17 10:24 /lib -> lib64
>> drwxr-xr-x 1 root root 1174 May 17 10:26 /lib32
>> drwxr-xr-x 1 root root 5078 Jun 13 18:50 /lib64
>> 
>> 
>> 17.1:
>> 
>> # ls -ld /lib*
>> drwxr-xr-x 1 root root 1382 May 13 22:54 /lib
>> drwxr-xr-x 1 root root 5340 Jun 18 10:23 /lib64
>> 
>> 
>> As you can see, "lib32" became "lib" and "lib" is no longer a symlink
>> pointing to "lib64".  
>
>Hmm, that's just nitpicking maybe and too late for any change now that
>the profile went on to be stable, but I don't get why lib32 was
>renamed to lib. Wouldn't it have been better to just keep lib32 and
>lib64 around (which describes both directories's purpose vividly) and
>just remove the compat symlink? Just using "lib" is ambiguous and can
>be confusing.

IIRC it was an attempt to remove the lib symlinks. Unfortunately I
haven't found the original post about why that change was introduced
anymore or why it was done this way. All I can remember is some talking
about "other distros are doing it likewise".

>
>> My suggestion is to let the Gentoo package maintainers handle this in
>> the affected x2go package's ebuild files. We have a function called
>> "get_libdir" that can be used with sed to set the correct lib dir
>> name in the affected code.  
>
>I could implement a Makefile variable that is being passed down via
>CFLAGS (no sed hackery) and add it as the first option. I'd like to
>keep the fallback list, though. Not sure if that would actually be
>really useful to anyone, though.

Not necessary. If both paths are present I suppose there will be no
more regressions.

>Mihai
>
>[0] {primary arch,secondary arch}
>

Cheers
Lars

-- 
Lars Wendler
Gentoo package maintainer
GPG: 21CC CF02 4586 0A07 ED93  9F68 498F E765 960E 9B39

[Message part 2 (application/pgp-signature, inline)]

Message #48 received at 1393@bugs.x2go.org (full text, mbox, reply):

From: X2Go Release Manager X2Go Release Manager <git-admin@x2go.org>

To: 1393-submitter@bugs.x2go.org

Cc: control@bugs.x2go.org, 1393@bugs.x2go.org

Subject: X2Go issue (in src:x2goclient) has been marked as closed

Date: Wed, 12 Feb 2020 22:46:56 +0100 (CET)

close #1393
thanks

Hello,

we are very hopeful that X2Go issue #1393 reported by you
has been resolved in the new release (4.1.2.2) of the
X2Go source project »src:x2goclient«.

You can view the complete changelog entry of src:x2goclient (4.1.2.2)
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=e70a27961cf391406c13e52fa54f4731fa8ca189;hp=84dcac55b72c6c09f2cdddcd5bde5137a441223d

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.2.2-0x2go1
Status: RELEASE
Timestamp: 1581543720
Date: Wed, 12 Feb 2020 22:42:00 +0100
Fixes: 1307 1320 1393 1418 1425 1428
Changes:
 x2goclient (4.1.2.2-0x2go1) RELEASED; urgency=medium
 .
   [ Oleksandr Shneyder ]
   * New upstream version (4.1.2.2):
     - Add SSH instruction to list of errors if keyboard-interactive auth has
       failed.
     - Add new brocker feature. Broker can send to client some configuration in
       the section START_CLIENT_CONFIG - END_CLIENT_CONFIG. For the moment is
       supported option events=bool. If true, client sending to broker events:
       CONNECTING, CONNECTED, SUSPENDING, TERMINATING, FINISHED. It's not a real
       session state, but rather the state of X2Go Client.
     - Save session command in resumingSession.command.
     - Send in broker event the duration of session on client.
     - If client is configured to send events, the broker can ask client to send
       ALIVE events. To do this, broker should send config option "liveevent"
       (int).
       It means, if liveevent=10, client will send ALIVE event to broker every
       10 seconds when the client connected to X2Go Session.
     - Broker can send to client command to suspend or terminate the session as
       an answer to the client event message. For this broker should send to
       client SUSPEND ID or TERMINATE ID.
     - Client now sends "login" parameter to the broker when executing task
       "selectsession". Before client just sent a username on the broker and it
       was imposiible to find out user name on X2Go server, which is not always
       the same as broker username. This won't break a compatibility with
       previous broker as they just will ignore this parameter.
     - X2Gokdrive support for X2GoClient.
     - Fix authentification on http broker when sending event.
     - Fix resuming sessions on multiple servers.
     - Initializing sessionId, command and display for direct RDP to complay
       with standard sessions.
     - Direct RDP sesion send to broker event CONNECTED instead of CONNECTING at
       start of RDP client.
     - Fixing setting correct session command when resuming the session.
     - Don't exit if sending event to http broker has failed.
     - If using x2go broker and ssh proxy check for the option
       "usebrokeruserforproxy" in the session file. If it's "true", use the same
       username for the proxy as broker login.
     - Broker can send to client the number of suspended and running sessions
       for each session type.
       Client will display this information to user on session button.
     - update copyright years in about dialog.
     - Do not show password in debug output of HTTP broker.
     - Fix crashing if the path for automount in the session config file in
       wrong format. Supporting both Windows and Linux path in automount format.
       Check if path exists before mounting.
     - Do not show Pulseaudio warning on Windows, if Pulseaudio is disabled in
       settings or by command line.
     - Improve request handling in HTTP broker.
     - Add possibility to suspend and terminate sessions from session selection
       dialog in broker mode.
     - Do not delete session from list if terminating of session is canceled.
 .
   [ Mihai Moldovan ]
   * New upstream version (4.1.2.2):
     - src/printwidget.cpp: add QButtonGroup include, fixes compile errors with
       Qt 5.11+. Fixes: #1307.
     - macbuild.sh: switch to bundling PulseAudio 12.1 since it's now available
       in MacPorts.
     - x2goclient.pro: detect libssh and libssh_threads via pkgconfig on
       UNIX-based platforms. libssh_threads is optional if the libssh version
       is high enough.
     - x2goclient.pro: let static Linux configuration override the new
       pkgconfig-based libssh usage.
     - x2goclient.pro: make system a non-variable call.
     - x2goclient.pro: fix qmake control flow for libssh logic, we really
       wanted to use else if for the Windows section and have the general else
       section being the general fallback.
     - x2goclient.pro: fix quoting issue in system() call. Qmake's quoting
       rules are really weird.
     - macbuild.sh: switch to bundling PulseAudio 12.2 since it's now available
       in MacPorts.
     - res/i18n/x2goclient_cs.ts: whitespace and other fixups.
     - src/x2goclient.cpp: change HTTP to HTTPS links where needed.
     - src/x2goclient.cpp: do not spawn UNIX cleanup helper if arguments that
       let X2Go Client terminate right away have been passed. More work is
       needed here, but that should cover the most basic bits.
     - src/onmainwindow.{cpp,h}: remove left-over references to agentCheckTimer
       and slotCheckAgentProcess ().
     - src/onmainwindow.cpp: fix various broken qCritical () calls.
     - src/{onmainwindow.{cpp,h},{sshmasterconnection,help}.cpp}: let users
       control libssh debugging via the new parameters --libssh-debug and
       --libssh-packetlog. Properly document the new switches and their
       implications.
     - src/sshmasterconnection.cpp: drop DEBUG preprocessor conditional blocks.
       We have never actually disabled it and x2goDebug () will not write out
       anything if debugging has not been requested, so we already have a soft
       condition check for every x2goDebug () call.
     - src/x2goclient.cpp: do not use std::array since that's a C++11 feature,
       just live with plain C arrays for now.
     - x2goclient.pro: fix libssh_threads logic again. Ubuntu Bionic/18.04
       ships a pre-release version of 0.8.0, which is tagged as that version,
       but really based upon 0.7.x and still has/needs a libssh_threads
       library. Big "Wat?" to you, Canonical. Fixes: #1320.
     - src/x2goclient.cpp: fix array length calculation.
     - Windows: Update PuTTY from 0.70 to 0.71. Fixes quite a list of issues
       discovered through the EU-funded bug bounty programme. Not all are
       relevant to X2Go Client.
     - Windows: upgrade bundled VcXsrv to 1.20.1.4.
     - src/sshmasterconnection.cpp: use info (protocol) log level with
       --libssh-debug and debug (packet) log level with --libssh-packetlog.
     - src/sshmasterconnection.cpp: fix compiler warning due to mismatching
       argument with older libssh versions.
     - Makefile: clear out LDFLAGS, add new (cleared) LIBS variable and pass
       down as QMAKE_LIBS. The previous behavior was leading to compile
       failures on arm64 machines (albeit only old distro versions,
       apparently), because QMAKE_LFLAGS is being put on the command line
       *before* the actual object files to be included/linked, which is an
       error for system libraries. Builds will still fail after this commit,
       but that's okay.
     - x2goclient.pro: add libraries that we always want to use on Linux
       correctly to LIBS and use a bigger catch clause. That should work much
       better, since linux-g++ and linux-g++-64 are x86-specific, while there
       are way more potentially useful targets like linux-aarch64-gnu-g++,
       clang, llvm, icc etc.
     - x2goclient.pro: while a linux scope seems to work fine with Qt5, Qt4
       doesn't recognize that, so add another linux-* catch.
     - Windows: add support for x3270-fonts, including adding a pretty nasty
       but necessary fonts.* file regeneration hook.
     - {x2goclient,help,onmainwindow}.cpp: implement new
       --bypass-cleanup-helper parameter.
     - src/onmainwindow.cpp: fix building with Qt 4.6, currently the latest
       version available in EPEL 6.
     - src/onmainwindow.cpp: unbreak builds by fixing syntax error.
     - src/onmainwindow.cpp: also unbreak old EPEL 6 builds.
     - src/onmainwindow.cpp: add (and prefer) non-compat-symlink scp server
       location for 64-bit-based Gentoo distros (17.1+ profiles). Fixes:
       #1393.
     - res/i18n/x2goclient_*.ts: replace changed source strings, no
       retranslations required.
     - src/httpbrokerclient.cpp: do not leak password length in debug output
       either.
     - src/sessionwidget.cpp: stop parsing proxy address as "host:port: if the
       address contains a colon. Fixes: #1418. We don't do this for the
       "normal" host address and this behavior clashes with IPv6 addresses. We
       COULD, theoretically, keep it, but that would require writing an
       arbitrary address parser. Too much effort for little gain.
     - src/onmainwindow.cpp: also remove proxy "host:port" parsing at connect
       time. Fixes: #1418.
     - macbuild.sh: switch to bundling PulseAudio 13.0 since it's now available
       in MacPorts.
     - src/onmainwindow.cpp: disable "left click" action on OS X/macOS. This
       platform always gets a left click event, even when right clicked. May
       sound weird first, but this behavior is consistent between all
       applications - tray icons can be clicked via either button and will
       always trigger a context menu. Let X2Go Client behave the same way.
     - src/sshprocess.cpp: strip ~/, ~user{,/}, ${HOME}{,/} and $HOME{,/} from
       destination paths in scp mode. Fixes: #1428. This was already necessary
       for pascp (PuTTY-based Windows solution for Kerberos support), but newer
       libssh versions with the CVE-2019-14889 also interpret paths as literal
       strings.
     - src/SVGFrame.cpp: simplify implementation a lot. Only paint data onto
       the widget in the paintEvent () function. Drop setting a palette based
       upon the rendered SVG image, which has been very wrong to do from the
       beginning and only lead to visual glitches. Correctly repaint
       single-frame SVG files.
     - src/sshmasterconnection.cpp: work around lupdate warning by adding
       another block in the #else preprocessor branch.
     - res/i18n/x2goclient_*.ts: update translation files.
     - Windows: Update PuTTY from 0.71 to 0.73. Fixes another round of issues
       discovered through the EU-funded bug bounty programme and other security
       problems.
     - src/onmainwindow.cpp: stop PulseAudio from spamming logs after a client
       disconnects. Fixes: #1425. Patch based on one submitted by Ville
       Salmela.
     - src/onmainwindow.cpp: instead of searching for "sftp-binary", try to
       look for "sftp-server" in the Qt 5.x code path. Luckily this only
       affects cases where we bundle sftp-server (which we don't) or where
       sftp-server is actually part of $PATH (which it usually shouldn't be).
     - Windows: upgrade bundled VcXsrv to 1.20.6.0.
     - Windows: upgrade PulseAudio from 7.1 to 13.0.
     - Windows: remove libz. Was previously used by libzip, which has not been
       shipped for quite some time.
     - Windows: Update Win32 OpenSSL from 1.0.2n to 1.0.2u. Implicit.
     - Windows: Upgrade libssh from 0.7.4 to 0.9.3 (while maintaining Pageant
       support). Drop libssh_threads library not provided (or used) any longer.
     - x2goclient.pro: update comment mentioning that libssh does not provide
       pkgconfig files on non-UNIX/Windows platforms any longer and drop
       libssh_thread linkage.
     - res/i18n/x2goclient_nl.ts: add missing singular form to a multi-form
       entry.
     - res/i18n/x2goclient_cs.ts: fix up whitespace and mark actually finished
       translation string as such, also removing my previous comment.
     - res/i18n/x2goclient_nb_no.ts: fix up whitespace and typos.
     - res/i18n/x2goclient_de.ts: better translate UNIX cleanup helper and fix
       "X2Go Client" as "X2GoClient" globally.
     - misc: pre-release copyright update.
     - src/onmainwindow.cpp: update in-code copyright notice.
     - res/i18n/x2goclient_*.ts: update copyright notice in about dialog
       manually.
     - src/onmainwindow.cpp: add comment for translators encouraging them to
       add themselves to copyright notice.
     - res/i18n/x2goclient_*.ts: update after source code changes and comment
       addition.
     - res/i18n/x2goclient_*.ts: mark copyright translation string as
       unfinished for already translated languages (but Finnish) to alert
       translators of the new comment.
     - man/man1/x2goclient.1: pre-release date update.
   * debian/control:
     + Add build-depend on pkg-config.
   * x2goclient.spec:
     + Build-depend on pkg-config.
     + EPEL packages need to depend upon pkgconfig for now, not pkg-config.
     + Pull in redhat-rpm-config manually. This should probably be done by
       something else, like... gcc or qmake or qt(4)-dev, but it isn't.
     + Try to ignore gettext-tools-mini on *SuSE to force installation of the
       new/renamed gettext-runtime-mini package.
     + It's %endif, not %fi.
     + Revert the gettext-tools-mini change, was caused by an outdated
       obs-build version.
 .
   [ Mike Gabriel ]
   * New upstream version (4.1.2.2):
     - Add support for Nix OS for common_sftp_dirs's hard-coded
       sftp-server paths.
     - src/sessionwidget.cpp: Rename shadowing sessiontype to 'X2Go/X11 Desktop
       Sharing'.
     - man/man1/x2goclient.1: Add explanation for the --broker-name option.
     - res/resources.qrc: Add Czech translation file.
     - Update qt_<lang>.qm files from Qt5 v5.11.2.
     - Rename radion button lables for xfreerdp options' style.
   * debian/control:
     + In x2goclient R, prefer freerdp2-x11 over rdesktop over freerdp-x11
       (which is broken these days anyway).
     + In x2goclient D, add x2gokdriveclient.
     + In x2goclient D, add hello as an alternative to x2gokdriveclient. Works
       around non-available x2gokdriveclient on Debian jessie.
 .
   [ Tom Ruzicka ]
   * New upstream release (4.1.2.2):
     - Add new translation: Czech.
     - Second update round of Czech translation + enable it in x2goclient.pro.
 .
   [ Ulrich Sibiller ]
   * New upstream release (4.1.2.2):
     - src/sshmasterconnection.cpp: close channel on failure in checkLogin ().
     - src/sshmasterconnection.{cpp,h}: skip checkLogin() if hostname starts
       with "!". Some special ssh proxies will not allow arbitrary commands.
       checkLogin() will break these sessions because it tries to run the echo
       command on the proxy. By specifying a "!" as the first character of the
       (proxy) hostname you can instruct x2goclient ot skip the checkLogin()
       call altogether. Note that this will break proxies that require you to
       change you password or some other type of interaction. As this is added
       to SshMasterConnection it is also valid to specify that for the server
       hostname although this is not very useful.
 .
   [ Jos Wolfkamp ]
   * New upstream version (4.1.2.2):
     - res/i18n/x2goclient_nl.ts: update Dutch translation file.
 .
   [ Sébastien Ducoulombier ]
   * New upstream version (4.1.2.2):
     - res/i18n/x2goclient_fr.ts: update French translation file.
     - res/i18n/x2goclient_fr.ts: add missing sentence and UTF-8-ize entries.
 .
   [ Ruda Vallo ]
   * New upstream version (4.1.2.2):
     - res/i18n/x2goclient_cs.ts: update Czech translation file.
 .
   [ Klaus Ade Johnstad ]
   * New upstream version (4.1.2.2):
     - res/i18n/x2goclient_nb_no.ts: update Bokmål (Norway) translation file.
     - res/i18n/x2goclient_nb_no.ts: update Bokmål (Norway) translation file.
 .
   [ Stefan Baur ]
   * New upstream release (4.1.2.2):
     - res/i18n/x2goclient_de.ts: update German translation file.

Send a report that this bug log contains spam.