Community
Participate
Working Groups
This extends the automatic proxy lookup for environment variables on Linux to include Gnome support. See bug 226462 Patch will be provided.
Created attachment 97568 [details] gnome native code Called by the UnixProxyProvider (already provided as part of bug 226462). The crash/hang that was happening in the gnome support has been resolved in this patch. The problem was the initialization of gconf was done on a thread other than the main thread and that seemed to cause problems sometimes. The fix was to move the initialization of gconf to the main thread. The problems disappeared after that. This patch contains 210 lines of code (the one native file)
Created attachment 97569 [details] binary for gnome shared library
Created attachment 97572 [details] Corrected patch to remove extra files I couple of files that did not belong in the patch were present. They are gone. This only has the native C code.
Created attachment 97600 [details] Some changes in Francis' patch
Created attachment 97612 [details] Some changes in Francis' patch
Library and the latest patch released to HEAD.
I filed bug 230040 related to this work. This patch https://bugs.eclipse.org/bugs/attachment.cgi?id=97572 has only the native code which is correct in all cases, the subsequent patches did not affect the native code. This patch https://bugs.eclipse.org/bugs/attachment.cgi?id=97572 caused the problem by it's modification to the UnixProxyProvider. Before the patch, the UnixProxyProvider initialized the gconf native code on the UI thread (and as part of the boot -- so it was not lazy). This patch revised the UnixProxyProvider to lazily call the native code, but not necessarily on the UI thread, depending on which code first called the UnixProxyProvider. In the case where the first thing to access remote information (and thus call the UnixProxyProvider) is not done on the UI thread, a hang can result -- an example of this is examining a CVS repository. This is the subject of bug 230040.
Closing on the advice of Szymon.
Verified. New bug raised against the initialization issue.
Created attachment 98756 [details] Added BUILD_INFO file This was missing from the original patch and should be added; it describes how the shared library was built.
BUILD_INFO.txt file added to HEAD.