Debian Bug report logs - #819160
ltsp-client-core: Please provide seperate usr/share/ltsp/screen.d/rdesktop and xfreerdp scripts

Toggle useless messages

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

From: Tomas Martišius <tomas@puga.vdu.lt>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: ltsp-client-core: Please provide seperate usr/share/ltsp/screen.d/rdesktop and xfreerdp scripts

Date: Thu, 24 Mar 2016 12:36:38 +0200

Package: ltsp-client-core
Version: 5.5.6-2
Severity: important

Please provide seperate scripts usr/share/ltsp/screen.d/rdesktop and usr/share/ltsp/screen.d/xfreerdp,
but not a symlink from rdesktop to xfreerdp, because in new versions of xfreerdp (especially xfreerdp2) command line syntax and params are different.

Best regards,

Tomas Martišius


-- System Information:
Debian Release: stretch/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.4.0-1-amd64 (SMP w/2 CPU cores)
Locale: LANG=lt_LT.UTF-8, LC_CTYPE=lt_LT.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages ltsp-client-core depends on:
ii  console-setup    1.139
ii  initramfs-tools  0.123
ii  iproute          1:4.3.0-1
ii  iproute2         4.3.0-1+b1
ii  kbd              2.0.3-2
ii  libc6            2.22-3
ii  libpopt0         1.16-10
ii  lsb-base         9.20160110
ii  lsb-release      9.20160110
ii  nbd-client       1:3.13-1
ii  pxelinux         3:6.03+dfsg-11
ii  python           2.7.11-1
ii  syslinux-common  3:6.03+dfsg-11
ii  tftp-hpa         5.2+20150808-1

Versions of packages ltsp-client-core recommends:
ii  acpi-support-base  0.142-8
ii  dbus               1.10.8-1
ii  dnsutils           1:9.9.5.dfsg-12.1
ii  pciutils           1:3.3.1-1.1
ii  procps             2:3.3.11-3

Versions of packages ltsp-client-core suggests:
ii  alsa-utils                   1.1.0-2
ii  cron                         3.0pl1-128
ii  cryptsetup                   2:1.7.0-2
ii  cups-bsd                     2.1.3-4
ii  gdm3 [x-display-manager]     3.18.0-2
ii  inputattach                  1:1.4.9-1
ii  ldm                          2:2.2.18-1
ii  libasound2-plugins           1.1.0-1
ii  ltspfsd                      1.4-2
ii  lxdm [x-display-manager]     0.5.3-1
ii  netcat-openbsd [netcat]      1.105-7
ii  ntpdate                      1:4.2.8p4+dfsg-3+b1
ii  numlockx                     1.2-7
ii  pulseaudio                   8.0-1
ii  python-serial                3.0.1-1
ii  rdesktop                     1.8.3-1+b1
ii  rsyslog [system-log-daemon]  8.16.0-1+b1
ii  sane-utils                   1.0.25-2
ii  sddm [x-display-manager]     0.13.0-1
ii  sshfs                        2.5-1
ii  x11-xserver-utils            7.7+7

-- no debconf information

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

From: Alkis Georgopoulos <alkisg@gmail.com>

To: Tomas Martišius <tomas@puga.vdu.lt>, 819160@bugs.debian.org

Subject: Re: Bug#819160: ltsp-client-core: Please provide seperate usr/share/ltsp/screen.d/rdesktop and xfreerdp scripts

Date: Thu, 24 Mar 2016 12:50:37 +0200

The idea is that the xfreerdp script doesn't contain any parameters at 
all, and you provide them via the
    SCREEN_07="xfreerdp <params>" lts.conf directive, or the
    SCREEN_07="rdesktop <params>" directive,
so as long as it doesn't enforce any parameters, why are two separate 
scripts needed?

The only hardcoded option in that script is
   RDP_OPTIONS=${RDP_OPTIONS:-"-f -u ''"}
...which is only there for compatibility with older LTSP versions, and 
can surely be replaced or completely omitted in lts.conf.

Message #15 received at 819160@bugs.debian.org (full text, mbox, reply):

From: Tomas Martišius <tomas@puga.vdu.lt>

To: Alkis Georgopoulos <alkisg@gmail.com>, 819160@bugs.debian.org

Subject: Re: Bug#819160: ltsp-client-core: Please provide seperate usr/share/ltsp/screen.d/rdesktop and xfreerdp scripts

Date: Thu, 24 Mar 2016 13:24:32 +0200

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

P.S. RDP_OPTIONS:-"-f -u ''" for xfreerdp must be "/f /u:" too.

Best regards,

Tomas

Not really... Then there are option RDP_SERVER in lts.conf the script
generates command such as this for example:

xfreerdp /f /sec:rdp /rfx +fonts /cert-ignore /sound:sys:pulse
/smartcard 192.168.0.16,

and there all stuff "/f /sec:rdp /rfx +fonts /cert-ignore
/sound:sys:pulse /smartcard" is from RDP_OPTIONS, but 192.168.0.16 is
form RDP_SERVER.

Even if RDP_SERVER is not set, it is generated from, "server", but this
makes new xfreerdp mad, because it needs /v:192.168.0.16 as a param,
and users get "black screen" with mouse pointer in it.

Best regards,

Tomas

2016.03.24 12:50, Alkis Georgopoulos rašė:
> The idea is that the xfreerdp script doesn't contain any parameters at
> all, and you provide them via the
>     SCREEN_07="xfreerdp <params>" lts.conf directive, or the
>     SCREEN_07="rdesktop <params>" directive,
> so as long as it doesn't enforce any parameters, why are two separate
> scripts needed?
>
> The only hardcoded option in that script is
>    RDP_OPTIONS=${RDP_OPTIONS:-"-f -u ''"}
> ...which is only there for compatibility with older LTSP versions, and
> can surely be replaced or completely omitted in lts.conf.
>

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

Message #20 received at 819160@bugs.debian.org (full text, mbox, reply):

From: Tomas Martišius <tomas@puga.vdu.lt>

To: Alkis Georgopoulos <alkisg@gmail.com>, 819160@bugs.debian.org

Subject: Re: Bug#819160: ltsp-client-core: Please provide seperate usr/share/ltsp/screen.d/rdesktop and xfreerdp scripts

Date: Thu, 24 Mar 2016 13:12:52 +0200

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

Not really... Then there are option RDP_SERVER in lts.conf the script
generates command such as this for example:

xfreerdp /f /sec:rdp /rfx +fonts /cert-ignore /sound:sys:pulse
/smartcard 192.168.0.16,

and there all stuff "/f /sec:rdp /rfx +fonts /cert-ignore
/sound:sys:pulse /smartcard" is from RDP_OPTIONS, but 192.168.0.16 is
form RDP_SERVER.

Even if RDP_SERVER is not set, it is generated from, "server", but this
makes new xfreerdp mad, because it needs /v:192.168.0.16 as a param,
and users get "black screen" with mouse pointer in it.

Best regards,

Tomas

2016.03.24 12:50, Alkis Georgopoulos rašė:
> The idea is that the xfreerdp script doesn't contain any parameters at
> all, and you provide them via the
>     SCREEN_07="xfreerdp <params>" lts.conf directive, or the
>     SCREEN_07="rdesktop <params>" directive,
> so as long as it doesn't enforce any parameters, why are two separate
> scripts needed?
>
> The only hardcoded option in that script is
>    RDP_OPTIONS=${RDP_OPTIONS:-"-f -u ''"}
> ...which is only there for compatibility with older LTSP versions, and
> can surely be replaced or completely omitted in lts.conf.
>

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

Message #25 received at 819160@bugs.debian.org (full text, mbox, reply):

From: Alkis Georgopoulos <alkisg@gmail.com>

To: Tomas Martišius <tomas@puga.vdu.lt>, 819160@bugs.debian.org

Subject: Re: Bug#819160: ltsp-client-core: Please provide seperate usr/share/ltsp/screen.d/rdesktop and xfreerdp scripts

Date: Thu, 24 Mar 2016 18:08:58 +0200

Try something like this:
    SCREEN_07="xfreerdp /f /sec:rdp /rfx +fonts /cert-ignore 
/sound:sys:pulse /smartcard /v:192.168.0.16"
i.e. put all your parameters in one line without setting the deprecated 
RDP_OPTIONS and RDP_SERVER variables.

As I said, those are only there for backwards compatibility.
Initially I wrote xfreerdp as a separate screen script and I didn't 
include those at all; then I put them there as an afterthought so that I 
could merge it with the older rdesktop script.
What I don't like about them is that there's no matching between those 
options and the screen number; i.e. they would have to be RDP_OPTIONS_07 
and RDP_SERVER_07 instead, which isn't very pretty.
They were implemented as separate options for the web interface of 
ltsp-cluster, but that's not developed or supported anymore.

If that answer satisfies you, then please close this bug report,
otherwise please open a bug report upstream in LTSP as the problem isn't 
in the Debian packaging but in the upstream code, so 
https://bugs.launchpad.net/ltsp/ is the proper place for it.

Cheers,
Alkis

Message #30 received at 819160@bugs.debian.org (full text, mbox, reply):

From: Vagrant Cascadian <vagrant@debian.org>

To: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>

Cc: 892626@bugs.debian.org, 819160@bugs.debian.org

Subject: Re: Bug#892626: Switch from freerdp-x11 to freerdp2-x11

Date: Fri, 06 Apr 2018 13:10:52 -0700

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

On 2018-04-06, Mike Gabriel wrote:
> On Sun, 11 Mar 2018 13:59:14 +0000 Mike Gabriel 
> <mike.gabriel@das-netzwerkteam.de> wrote:
>  > the freerdp-x11 (1.1 series) package shall not be included in buster.
>  > Please replace ltsp-client's dependency on it by a dependency on
>  > freerdp2-x11.
>  >
>  > When switching to the new FreeRDP2 version, make sure you use the new
>  > command line syntax for the xfreerdp command.

I don't personally use FreeRDP (or even LTSP anymore for that matter),
do you think you could provide references to a summary of the changes in
syntax?

Or maybe even a patch? :)

With my LTSP hat on, old and worn out though it may be, I'd think we'd
maybe still want to provide support for the old variant for backports
and legacy setups, so it might be appropriate to create a new script,
especially in light of:

#819160: ltsp-client-core: Please provide seperate
 usr/share/ltsp/screen.d/rdesktop and xfreerdp scripts

  https://bugs.debian.org/819160


> Raising this bug to RC level. The ltsp-client package is the last 
> package that prevents us from having the old freerdp (FreeRDP 1.1) 
> src:package removed from Debian.

Apologies for taking so long to explore this.


live well,
  vagrant

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

Message #35 received at 819160@bugs.debian.org (full text, mbox, reply):

From: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>

To: Vagrant Cascadian <vagrant@debian.org>

Cc: 892626@bugs.debian.org, 819160@bugs.debian.org

Subject: Re: Bug#892626: Switch from freerdp-x11 to freerdp2-x11

Date: Fri, 06 Apr 2018 20:24:18 +0000

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

Hi Vagrant

On  Fr 06 Apr 2018 22:10:52 CEST, Vagrant Cascadian wrote:

> On 2018-04-06, Mike Gabriel wrote:
>> On Sun, 11 Mar 2018 13:59:14 +0000 Mike Gabriel
>> <mike.gabriel@das-netzwerkteam.de> wrote:
>>  > the freerdp-x11 (1.1 series) package shall not be included in buster.
>>  > Please replace ltsp-client's dependency on it by a dependency on
>>  > freerdp2-x11.
>>  >
>>  > When switching to the new FreeRDP2 version, make sure you use the new
>>  > command line syntax for the xfreerdp command.
>
> I don't personally use FreeRDP (or even LTSP anymore for that matter),
> do you think you could provide references to a summary of the changes in
> syntax?
>
> Or maybe even a patch? :)
>
> With my LTSP hat on, old and worn out though it may be, I'd think we'd
> maybe still want to provide support for the old variant for backports
> and legacy setups, so it might be appropriate to create a new script,
> especially in light of:
>
> #819160: ltsp-client-core: Please provide seperate
>  usr/share/ltsp/screen.d/rdesktop and xfreerdp scripts
>
>   https://bugs.debian.org/819160
>
>
>> Raising this bug to RC level. The ltsp-client package is the last
>> package that prevents us from having the old freerdp (FreeRDP 1.1)
>> src:package removed from Debian.
>
> Apologies for taking so long to explore this.
>
>
> live well,
>   vagrant

I'll see what I can do after having returned from VAC (around the 15th  
April)...

Mike
-- 

DAS-NETZWERKTEAM
mike gabriel, herweg 7, 24357 fleckeby
mobile: +49 (1520) 1976 148
landline: +49 (4354) 8390 139

GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22  0782 9AF4 6B30 2577 1B31
mail: mike.gabriel@das-netzwerkteam.de, http://das-netzwerkteam.de

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

Message #40 received at 819160@bugs.debian.org (full text, mbox, reply):

From: Alkis Georgopoulos <alkisg@gmail.com>

To: Vagrant Cascadian <vagrant@debian.org>, 892626@bugs.debian.org, Mike Gabriel <mike.gabriel@das-netzwerkteam.de>

Cc: 819160@bugs.debian.org

Subject: Re: Bug#892626: Switch from freerdp-x11 to freerdp2-x11

Date: Sat, 7 Apr 2018 12:58:42 +0300

Στις 06/04/2018 11:10 μμ, ο Vagrant Cascadian έγραψε:
> With my LTSP hat on, old and worn out though it may be, I'd think we'd
> maybe still want to provide support for the old variant for backports
> and legacy setups, so it might be appropriate to create a new script,
> especially in light of:
> 
> #819160: ltsp-client-core: Please provide seperate
>   usr/share/ltsp/screen.d/rdesktop and xfreerdp scripts
> 


As I wrote in that bug report, one generic "run something under a 
minimal xorg environment" script should be enough for any 
rdp/vnc/whatever-like application.

We shouldn't keep supporting RDP_SERVER etc just because at some point 
in the past a company had an UI for that.
A full command line provided in a single lts.conf variable, that may 
even contain "server" or "$SERVER" wherever appropriate, should be more 
than enough.

Message #45 received at 819160-close@bugs.debian.org (full text, mbox, reply):

From: Vagrant Cascadian <vagrant@debian.org>

To: 819160-close@bugs.debian.org

Subject: Bug#819160: fixed in ltsp 5.18.04-1

Date: Tue, 24 Apr 2018 02:56:57 +0000

Source: ltsp
Source-Version: 5.18.04-1

We believe that the bug you reported is fixed in the latest version of
ltsp, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 819160@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Vagrant Cascadian <vagrant@debian.org> (supplier of updated ltsp package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Mon, 23 Apr 2018 18:42:19 -0700
Source: ltsp
Binary: ltsp-server ltsp-server-standalone ltsp-client-core ltsp-client ltsp-client-builder
Architecture: source
Version: 5.18.04-1
Distribution: unstable
Urgency: medium
Maintainer: LTSP Debian Maintainers <pkg-ltsp-devel@lists.alioth.debian.org>
Changed-By: Vagrant Cascadian <vagrant@debian.org>
Description:
 ltsp-client - complete LTSP client environment
 ltsp-client-builder - build an LTSP environment in the installer target (udeb)
 ltsp-client-core - basic LTSP client environment
 ltsp-server - basic LTSP server environment
 ltsp-server-standalone - complete LTSP server environment
Closes: 819160 888583 892626
Changes:
 ltsp (5.18.04-1) unstable; urgency=medium
 .
   * New upstream version.
 .
   [ Kim B. Heino ]
   * ltsp-update-kernels: update SELinux context even if in permissive
     mode.
   * Add support for multiple log targets in SYSLOG_HOST variable.
     (LP: #1690824)
   * ltsp-update-kernels: cleanup also .vmlinuz-*.hmac files.
   * init-ltsp.d/09-hostname: strip search domains from hostname.
   * Many updates to Redhat support.
   * ltsp-update-sshkeys: include ed25519 keys.
   * 50-set-time: if /etc/localtime is symlink handle it as such.
 .
   [ Alkis Georgopoulos ]
   * Update Gentoo codebase a bit.
   * Check that ntpdate exists before calling it.
   * Prohibit the kernel from removing the ip address.
   * Fedora: don't set valid_flt=forever when BOOTPROTO=dhcp.
 .
   [ Vagrant Cascadian ]
   * Sync translations with launchpad.
   * xfreerdp: Adjust default arguments when called as xfreerdp.
     (Closes: #819160)
 .
   * Debian packaging:
     - debian/patches:
       + Remove patches applied upstream.
       + Remove ancient, unused kFreeBSD patches.
     - debian/control:
       + ltsp-client*: Prefer freerdp2-x11 over other RDP clients.
         (Closes: #892626)
       + Update Standards-Version to 4.1.4, no changes needed.
       + ltsp-client-builder: Depend on console-setup-udeb for both Debian
         and Ubuntu (Closes: #888583).
Checksums-Sha1:
 5a49021c2540fe3202b5e6d384e322a7366bed7b 2289 ltsp_5.18.04-1.dsc
 68c50cd6b74e3d7a790fadd889cc687c80cba835 238440 ltsp_5.18.04.orig.tar.xz
 a6d09f0cc5149b9fe8a9827ed81546bfef61d738 75904 ltsp_5.18.04-1.debian.tar.xz
 4f9417c8114759abd15e22a2c6db2f924d68af56 7238 ltsp_5.18.04-1_amd64.buildinfo
Checksums-Sha256:
 e4bc504554b63936a71c34c3c59977628e70a278345c234a3fa45ce9075d4b25 2289 ltsp_5.18.04-1.dsc
 7d451e8202e5148876f5e26bb5d911c5a16633ae2622f96d5e2c2d1cf7d8db5c 238440 ltsp_5.18.04.orig.tar.xz
 399e695ea78e2ce9644e5b8cccbd4c714cc542c1597c2b5db6a8237aee44e263 75904 ltsp_5.18.04-1.debian.tar.xz
 b360688e3ad13308c2425cd84893e88eccdefcb36fcac719535e42f0db97e216 7238 ltsp_5.18.04-1_amd64.buildinfo
Files:
 92222ddadb2ae3acedd60735995462c8 2289 misc optional ltsp_5.18.04-1.dsc
 a58ce6e60b0014eaaf68e016a8dc379b 238440 misc optional ltsp_5.18.04.orig.tar.xz
 e9cc788be1a7428e3b93095ef0a790be 75904 misc optional ltsp_5.18.04-1.debian.tar.xz
 53db4abf004e9511751c50e730fd4a9a 7238 misc optional ltsp_5.18.04-1_amd64.buildinfo

-----BEGIN PGP SIGNATURE-----

iQJHBAEBCgAxFiEEcDardHbDotegGFCHt4uC1IFLkbYFAlrejL8THHZhZ3JhbnRA
ZGViaWFuLm9yZwAKCRC3i4LUgUuRtup2EACawQFp4m+2D11qMRR6NSWi77KzMbgj
o+NldNzVC7ayi/V3GpA5Lj2DU2ez9Feug6lvopOAgug9pbjMOB8An9htHd4qxGhz
7LWl1YoGcR09rw6EPYtCH671JSQ57YsRDAYr/dXNxfF1SmhRnstRKsQPoucdKfDJ
LUcJO1OMCnIfY25+OCdvXOqtotqhIgHWEDIlStoRYegi44otr0FDj371rrWhfyPF
TBmztM5gaaRmrM0Nxs6Y+i9R0BrsYURZjh/OeSMeudgxqFpD3hBoJTDskFSPxqcJ
naJ1J4aj+Peg4VYBj37+t34T/mcxNCjamKqlvnYmO35eiWgK4TuyTY6uQ95fyDHT
V3TjJ3hK8Q5K7ZV93bETzVgDPANKiIQmTH8YDue71LuylJG9gEKDdeqjam3zlS7D
ac2sarBposr7poFq2RQq1mXiwwSGU6+6kWjZCxnVxhxDdmB3w69vsNytmmqNhw4m
fhNPZRjUC4WBDZvIfDs/2e85MX3LRbRmhDlGSDdqL32hEDZ7WqUwV2O7Mli8T/NI
aqo0eBbh2WwAQGuT19whMvQbL/+CWzyw8ceEb7vUwSDNO2LkgjSVuVRTin8KwPqp
wQ4x1lgNHROg1IC6q5qxuU+Mkb8O4XEXM1jTqTS6X/x0WTuz6wHj8/+sfrJ+pyU0
dTe4FuvB+z8iqg==
=r346
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.