[FreeNX-kNX] Fw: Trouble with rdp tunneling on ubuntu . . . Qpatch for ubuntu nxserver

[chris at ccburton.com]

nxserver does a check for rdesktop/nxdesktop . . .

if "$ENABLE_EXTERNAL_NXDESKTOP" = "1" -a "$(getparam type)" = "windows"
ie.
if rdesktop is selected AND it's a windows session then . . .

        type="windows-helper"

and changes type to windows-helper

BUT then nxserver also changes CMDLINE in an unfortunate
manner. .

        PARAMS="$PARAMS&type=$type"
        CMDLINE=$PARAMS

ie. it ADDS

        &type=windows-helper

on to the end of CMDLINE and passes it on to nxnode.

That is to say CMDLINE now looks like

&link=isdn. .&type=windows. . .&type=windows-helper&freenx_export_agents=1

now containg TWO &type= entries

THEN

nxnode runs

        getparam type

on CMDLINE and hopes to get the last value of type=
to decide what to launch.

Well PREVIOUSLY this all worked out fine because
the previous version of getparam printed windows-helper
ie. the final occurence of type=

HOWEVER a patch in nxnode replaced getparam with
a python script which prints the

         FIRST occurance of type=

ie. it still outputs windows instead of windows-helper!!

nxnode then tries to run the non existent nxdesktop helper
and can't find it





Quick-bodge for anyone waiting . . . .



diff -u nxserver.000 nxserver
--- nxserver.000        2011-05-22 16:48:59.000000000 +0100          
+++ nxserver    2011-08-03 18:49:46.699813042 +0100      
@@ -1392,7 +1392,8 @@   
                if [ "$ENABLE_EXTERNAL_NXDESKTOP" = "1" -a "$(getparam 
type)" = "windows" ]  
                then
                        type="windows-helper"
-                       PARAMS="$PARAMS&type=$type&freenx_export_agents=1"
+                       PARAMS=$(echo $PARAMS|sed 
"s/type=windows/type=windows-helper/g")
+                       PARAMS="$PARAMS&freenx_export_agents=1"
                        CMDLINE=$PARAMS
                fi
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/freenx-knx/attachments/20110804/3d433710/attachment.html>