This topic came up some time back and I pretty much ignored it, but the "itch" came back to find out why. Searching through the FreeNX/knx list came up with the reason sessions did not come up full-screen even though the session definition was set to fullscreen.
Apparently there is an oversight in /usr/bin/nxnode, the script is never retrieving the fullscreen flag from the parameters passed from the client. This is the case even in freenx-0.5.0-9.c4, the current version for CentOS 4. Here's a quick patch against the freenx-0.5.0-9.c4 version of /usr/bin/nxnode that resolves this issue:
***PATCH START*** --- nxnode.save 2006-09-13 12:01:31.000000000 -0500 +++ nxnode 2006-09-13 12:02:08.000000000 -0500 @@ -626,6 +626,7 @@ imagecompressionmethod=$(getparam imagecompressionmethod) imagecompressionlevel=$(getparam imagecompressionlevel) geometry=$(getparam geometry) + fullscreen=$(getparam fullscreen) keyboard=$(getparam keyboard) kbtype=$(getparam kbtype) media=$(getparam media) @@ -661,7 +662,6 @@ status=$(getparam status) host=$(getparam host)
- fullscreen="" [ "$geometry" = "fullscreen" ] && fullscreen="1"
[ "$EXPORT_USERIP" = "1" ] && export NXUSERIP="$userip" ***PATCH END***
After this patch, nxnode will respect the fullscreen flag if it is set in the configuration.
Hope that helps!
On Wed, 2006-09-13 at 13:26 -0500, Jay Leafey wrote:
This topic came up some time back and I pretty much ignored it, but the "itch" came back to find out why. Searching through the FreeNX/knx list came up with the reason sessions did not come up full-screen even though the session definition was set to fullscreen.
Apparently there is an oversight in /usr/bin/nxnode, the script is never retrieving the fullscreen flag from the parameters passed from the client. This is the case even in freenx-0.5.0-9.c4, the current version for CentOS 4. Here's a quick patch against the freenx-0.5.0-9.c4 version of /usr/bin/nxnode that resolves this issue:
THANK YOU!!!
It's a minor inconvenience to me. But it's a whole other ball game when you have 45 users, with some employee turnover, who "just have to hit CTRL-ALT-F". Not to mention embarrassing to have to explain that it's a bug in "Linux".
This is *much* appreciated. I'll be collecting up the thank you's from my users and sending them over in one batch to avoid list clutter. ;-)
-Steve
On Wed, 2006-09-13 at 13:26 -0500, Jay Leafey wrote:
This topic came up some time back and I pretty much ignored it, but the "itch" came back to find out why. Searching through the FreeNX/knx list came up with the reason sessions did not come up full-screen even though the session definition was set to fullscreen.
Apparently there is an oversight in /usr/bin/nxnode, the script is never retrieving the fullscreen flag from the parameters passed from the client. This is the case even in freenx-0.5.0-9.c4, the current version for CentOS 4. Here's a quick patch against the freenx-0.5.0-9.c4 version of /usr/bin/nxnode that resolves this issue:
***PATCH START*** --- nxnode.save 2006-09-13 12:01:31.000000000 -0500 +++ nxnode 2006-09-13 12:02:08.000000000 -0500 @@ -626,6 +626,7 @@ imagecompressionmethod=$(getparam imagecompressionmethod) imagecompressionlevel=$(getparam imagecompressionlevel) geometry=$(getparam geometry)
fullscreen=$(getparam fullscreen) keyboard=$(getparam keyboard) kbtype=$(getparam kbtype) media=$(getparam media)@@ -661,7 +662,6 @@ status=$(getparam status) host=$(getparam host)
fullscreen="" [ "$geometry" = "fullscreen" ] && fullscreen="1" [ "$EXPORT_USERIP" = "1" ] && export NXUSERIP="$userip"***PATCH END***
After this patch, nxnode will respect the fullscreen flag if it is set in the configuration.
Hope that helps!
Jay,
Thanks for the patch.
The patch is rolled in and freenx-0.5.0-10.c4.noarch.rpm is pushed to the centos mirrors.
-- Johnny Hughes
Johnny Hughes wrote:
The patch is rolled in and freenx-0.5.0-10.c4.noarch.rpm is pushed to the centos mirrors.
Many thanks! Like I said, this has been annoying me since June and I finally had a chance to research it. I'm just pleased to be able to give something back!
On Wed, 2006-09-13 at 16:59 -0500, Jay Leafey wrote:
Johnny Hughes wrote:
The patch is rolled in and freenx-0.5.0-10.c4.noarch.rpm is pushed to the centos mirrors.
Many thanks! Like I said, this has been annoying me since June and I finally had a chance to research it. I'm just pleased to be able to give something back!
For the impatient (like me) I found it at:
http://public.planetmirror.com/pub/centos/4.4/extras/i386/RPMS/freenx-0.5.0-...
and it works! :-)
-Steve
-
Jay Leafey -
Johnny Hughes -
Steve Bergman