Bug 193522 - [Help] Help doesn't work on Eclipse 3.2.1 on AIX
Summary: [Help] Help doesn't work on Eclipse 3.2.1 on AIX
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: User Assistance (show other bugs)
Version: 3.2.1   Edit
Hardware: Other AIX Motif
: P3 major (vote)
Target Milestone: 3.8 M1   Edit
Assignee: Chris Goldthorpe CLA
QA Contact:
URL:
Whiteboard:
Keywords: needinfo
Depends on:
Blocks:
 
Reported: 2007-06-20 10:28 EDT by Shawn L. Pecze CLA
Modified: 2011-11-01 13:53 EDT (History)
1 user (show)

See Also:


Attachments
Log file (9.05 KB, application/octet-stream)
2007-06-20 10:30 EDT, Shawn L. Pecze CLA
no flags Details
Patch (894 bytes, patch)
2007-06-21 14:05 EDT, Chris Goldthorpe CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Shawn L. Pecze CLA 2007-06-20 10:28:50 EDT
Build ID: M20060921-0945

Steps To Reproduce:
1.  Launch eclipse
2.  Select Help -> Dynamic Help
3.  Click on any of the links shown
4.  No browser is launched


More information:
I found instructions here (http://www.eclipse.org/swt/faq.php) to set MOZILLA_FIVE_HOME and add that to the LD_LIBRARY_PATH, but that didn't seem to help
Comment 1 Shawn L. Pecze CLA 2007-06-20 10:30:03 EDT
Created attachment 71896 [details]
Log file

I'm attaching a .log file.  I noticed some NullPointerExceptions in there
Comment 2 Chris Goldthorpe CLA 2007-06-20 12:55:50 EDT
Looking at the log file I noticed the line:

!MESSAGE Could not load library: liblocalfile_1_0_0.a.

I would first want to see why this message is being shown, missing a shared library could certainly cause the help system to fail (I have seen variants of his problem on x86 Linux also).
Comment 3 Chris Goldthorpe CLA 2007-06-20 13:56:28 EDT
I'm rethinking my last comment, apparently liblocalfile is not a required library so this would not be the problem.
Comment 4 Shawn L. Pecze CLA 2007-06-20 14:51:42 EDT
Yeah, I did some googling of that library, and it said that it's not required on all platforms so it may not indicate an error.
Comment 5 Chris Goldthorpe CLA 2007-06-20 16:17:26 EDT
From the stack trace I can see that in the function ReusableHelpPart.contextMenuAboutToShow() focusControl is null. It should be possible to add a null pointer check and I would expect the problem to go away. Are you in a position to test a patch? I could alternatively build a deployable plugin if that is easier.
Comment 6 Shawn L. Pecze CLA 2007-06-21 09:56:27 EDT
Yes, I could test a patch.
Comment 7 Chris Goldthorpe CLA 2007-06-21 14:05:53 EDT
Created attachment 72069 [details]
Patch

Can you try this? It should make the null pointer go away, as far as I can tell this will allow the help view to function correctly.
Comment 8 Chris Goldthorpe CLA 2011-04-27 12:43:17 EDT
Is this still a problem?
Comment 9 Shawn L. Pecze CLA 2011-07-25 15:47:32 EDT
No, this is no longer a problem.  I'm not sure why I never updated the bug with the results of your patch.
Comment 10 Chris Goldthorpe CLA 2011-07-25 16:47:10 EDT
The SeeAlso part is no longer used which is why you are no longer seeing a problem. Closing.
Comment 11 Chris Goldthorpe CLA 2011-07-25 17:01:55 EDT
I took a look at this again. I had thought that the section code containing the patch was unreachable, and so far I have not yet been able to hit a breakpoint set on the modified line, however now I'm less sure about the unreachability of this code and since the patch simple adds a null pointer check I have committed it for Eclipse 3.8.