Issue 4829 in pharo: Instant crash when right click (or get exception) while connected through VNC/RFB on an headless image

Status: New
Owner: ----

New issue 4829 by [hidden email]: Instant crash when right click (or  
get exception) while connected through VNC/RFB on an headless image
http://code.google.com/p/pharo/issues/detail?id=4829

Pharo image: core
Pharo core version: Pharo1.3 update #13298
Virtual machine used: Squeak-4.4.7.2357-linux_i386
Class browser used if applicable: OBSystemBrowserAdaptor

Steps to reproduce:
1. open a headless image (I'm using -vm-sound-null -vm-display-null)
2. install RFB and connect to that image with some native client
3. open a system browser and "right click" on the package pane for an  
instant crash or
4. evaluate something that gives a DNU like '5 kk' or whatever and also get  
an instant crash

PharoDebug.log <-- see attachment.

Notes:
1. everything looks like a perfectly normal backtrace except for the crash.
2. same image, same VM started with UI (non-headless) works just fine, no  
crash at all


Attachments:
        HeadlessVNCCrashPharoDebug.log  49.2 KB


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Accepted

Comment #1 on issue 4829 by [hidden email]: Instant crash when right  
click (or get exception) while connected through VNC/RFB on an headless  
image
http://code.google.com/p/pharo/issues/detail?id=4829

RFB needs some updating to work with Pharo versions where headless images  
use a NonInteractiveUIManager.
Basically, when someone connect, you want to switch it to an interactive  
one, so you don't get errors whenever there's a dialog popping up etc.
Then, when they disconnect, you switch it back to noninteractive, so errors  
will be written appropriately if they happen.

Sadly, only the interactive -> non-interactive is currently easy (UIManager  
default nonInteractiveManager , should probably add an interactiveManager  
method, plus change it to sound like an action rather than an accessor, as  
it does change the manager)

A workaround until the other way is accessable the same way you could  
probably do:

MorphicUIManager new beDefault.

and in disconnect:
UIManager default nonInteractiveManager


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Cc: [hidden email]
        Labels: RFB UIManager

Comment #2 on issue 4829 by [hidden email]: Instant crash when right  
click (or get exception) while connected through VNC/RFB on an headless  
image
http://code.google.com/p/pharo/issues/detail?id=4829

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Invalid

Comment #3 on issue 4829 by [hidden email]: Instant crash when right  
click (or get exception) while connected through VNC/RFB on an headless  
image
http://code.google.com/p/pharo/issues/detail?id=4829

Change needed in RFB package, not pharo?

(I close this. to get it open again, action is needed)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #4 on issue 4829 by [hidden email]: Instant crash when right  
click (or get exception) while connected through VNC/RFB on an headless  
image
http://code.google.com/p/pharo/issues/detail?id=4829

Right...

question:

who is currently maintaining it?


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Accepted

Comment #5 on issue 4829 by [hidden email]: Instant crash when right  
click (or get exception) while connected through VNC/RFB on an headless  
image
http://code.google.com/p/pharo/issues/detail?id=4829

No, some changes are needed to the UIManager to make it easier to switch  
between headless/headful seamlessly before RFB should be updated.

Which is why I cc'd Igor, he seems to be intimate with that part of the  
code :)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #6 on issue 4829 by [hidden email]: Instant crash when right  
click (or get exception) while connected through VNC/RFB on an headless  
image
http://code.google.com/p/pharo/issues/detail?id=4829

To replace current ui manager just do:
   MorphicUIManager new beDefault.

Which is no-op if you do it in interactive image.

Also, see UIManager>>boot: bootingFromDisk during: aBlock
this code responsible for replacing UI manager during image startup.

So, if you want to make everything to work with MorphicUIManager, you  
should change it there as well.

And if you want to also prevent morphic ui manager to be switched to  
non-interactive one when in headless mode, change the
MorphicUIManager>>onSnapshot: resuming
By removing (or commenting)
        (resuming and: [ Smalltalk isInteractive not ]) ifTrue: [
                        ^ self nonInteractiveManager onSnapshot: resuming ].

Another option to deal with errors is to turn an option "snapshot before  
quit" which you can find in settings under 'headless mode'.


And the most trivial way is to properly handle all errors while running in  
headless mode :)



_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #7 on issue 4829 by [hidden email]: Instant crash when right  
click (or get exception) while connected through VNC/RFB on an headless  
image
http://code.google.com/p/pharo/issues/detail?id=4829

Fantastic answer.

Thank you

Thank you

Thank you


sebastian

o/


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker