<br><tt><font size=2>freenx-knx-bounces@kde.org wrote on 17/04/2013 00:49:03:<br>
<br>
> Thanks Chris, for working with me on this!!!<br>
> <br>
> I've attached a zip with the following 4 files, per your <br>
> instructions (except I happen to have my nxerver.log in an /nx <br>
> subdirectory of /var/log):<br>
</font></tt>
<br><tt><font size=2>Right-oh.</font></tt>
<br>
<br><tt><font size=2>I can see it modifying the type= but then ignoring
the new value</font></tt>
<br><tt><font size=2>so . . . a quick look at the code . . . </font></tt>
<br>
<br>
<br>
<br><tt><font size=2>If you look through your log file you can see where
the code</font></tt>
<br><tt><font size=2>has updated paramater type  from</font></tt>
<br>
<br><tt><font size=2>        type=vnc</font></tt>
<br><tt><font size=2>to</font></tt>
<br><tt><font size=2>        type=vnc-helper</font></tt>
<br>
<br><tt><font size=2>Here is the modified parameter set ( sorry if its
wrapped )</font></tt>
<br>
<br><tt><font size=2>&link=lan&backingstore=1&encryption=1&cache=16M&images=64M&shmem=1&shpix=1&strict=0&composite=1&media=0&session=VNC
2 Forte via Destiny NX&type=vnc&agent_server=forte%3A5900&agent_password=******&geometry=1024x768+128+112&client=linux&keyboard=pc102/us&screeninfo=1024x768x24+render&clientproto=3.2.0&login_method=SSH&type=vnc-helper&application=/usr/lib/nx/nxviewer_helper&freenx_export_agents=1&user=mikem&userip=10.0.1.101&uniqueid=181ECAF26280C670703B131ED401722B&display=2000&host=127.0.0.1</font></tt>
<br>
<br><tt><font size=2>The code which accomplishes this is in nxserver</font></tt>
<br>
<br><tt><font size=2>  function server_startrestore_session() from
line 1399</font></tt>
<br>
<br><tt><font size=2>  if [ "$ENABLE_EXTERNAL_NXVIEWER"
= "1" -a "$(getparam type)" = "vnc" ]</font></tt>
<br><tt><font size=2>               
then</font></tt>
<br><tt><font size=2>type="vnc-helper"</font></tt>
<br><tt><font size=2>application="$PATH_BIN/nxviewer_helper"</font></tt>
<br><tt><font size=2>PARAMS="$PARAMS&type=$type&application=$application&freenx_export_agents=1"</font></tt>
<br><tt><font size=2>CMDLINE=$PARAMS</font></tt>
<br><tt><font size=2>               
fi</font></tt>
<br>
<br><tt><font size=2>NOTE  the new paramwter is ADDED onto THE END
of variable:- PARAMS</font></tt>
<br>
<br><tt><font size=2>AND</font></tt>
<br><tt><font size=2>if you look back at your parameters you will see two
&type=</font></tt>
<br>
<br><tt><font size=2>This is then being ignored, so a quick look at the
ubuntu version of</font></tt>
<br><tt><font size=2>getparam shows why.</font></tt>
<br>
<br>
<br><tt><font size=2>This is a classic problem of maintaining old code
which you don't fully</font></tt>
<br><tt><font size=2>understand without testing ALL functionality.</font></tt>
<br>
<br><tt><font size=2>I can't see straight off why any change was neccessary
here tho,</font></tt>
<br><tt><font size=2>so I will add that fixing things which aren't broken
in old code</font></tt>
<br><tt><font size=2>'cos you have a tidy mind, is aone of the greatest
ways to</font></tt>
<br><tt><font size=2>inconvenience any number of (other) people.</font></tt>
<br>
<br>
<br><tt><font size=2>***** see attached demonstrator script ******</font></tt>
<br>
<br>
<br><tt><font size=2>You have anumber of options for a fix,</font></tt>
<br><tt><font size=2>the quick one being</font></tt>
<br><tt><font size=2> ****        to mod
your nxnode back to the old version,****</font></tt>
<br><tt><font size=2>           
     (from the attachment)</font></tt>
<br><tt><font size=2>until either</font></tt>
<br><tt><font size=2>the python enthusiast in question produces an update</font></tt>
<br><tt><font size=2>or</font></tt>
<br><tt><font size=2>they work through the scripts changing all instances
of</font></tt>
<br><tt><font size=2>"update by append" to "update by replace"</font></tt>
<br><tt><font size=2>(more uneccessary changes, but once you start on</font></tt>
<br><tt><font size=2>         olde code</font></tt>
<br><tt><font size=2> it's endless)</font></tt>
<br>
<br>
<br>
<br>
<br>
<br><tt><font size=2>Let us know how you get on with it  . . . .</font></tt>
<br>
<br>
<br><tt><font size=2>cb</font></tt>
<br>
<br><tt><font size=2>ok, what's left of lunch time ????</font></tt>
<br>
<br>