[icedtea-web] RFC: integrate logging into netx and plugin

[Omair Majid]

Hi,

The attached patch integrates the logging option from the deployment 
configuration into netx and plugin. Both plugin and netx will now log 
(if logging is enabled) their stdout/stderr to the java.stdout and 
java.stderr files under the logging directory. Running the plugin with 
the environment variable ICEDTEAPLUGIN_DEBUG=true will force stdout and 
stderr to be redirected to the logging files.

ChangeLog:
2010-11-11  Omair Majid  <omajid at redhat.com>

     * netx/net/sourceforge/jnlp/runtime/DeploymentConfiguration.java:
     Add KEY_ENABLE_LOGGING.
     (loadDefaultProperties): Use KEY_ENABLE_LOGGING.
     * netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java: Add
     redirectStreams, STDERR_FILE and STDOUT_FILE.
     (initialize): Call initializeStreams.
     (initializeStreams): New method. Redirects or duplicates stdout and
     stderr to the logging files as required.
     (setRedirectStreams): New method. Sets whether stdout/stderr streams
     should be redirected.
     * plugin/icedteanp/java/sun/applet/PluginMain.java:
     (PluginMain): Move code for creating logging files into JNLPRuntime.
     Call JNLPRuntime.setRedirectStreams to redirect streams.
     (TeeOutputStream): Move to its own class.
     * netx/net/sourceforge/jnlp/util/TeeOutputStream.java: Moved from
     PluginMain into this new class.

Any thoughts or comments?

Cheers,
Omair
-------------- next part --------------
A non-text attachment was scrubbed...
Name: icedtea-web-logging-01.patch
Type: text/x-patch
Size: 10610 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20101111/46f2b55a/icedtea-web-logging-01.patch