Configure 'logging' to generate log files on Windows/Linux/Mac and sy…

…slog where available / Configure 'logging' to do async buffered logging via a thread to avoid impacting app performance with logging

.gitignore

@@ -54,3 +54,4 @@ pkg
 .gladiator
 
 tmp
+log

CHANGELOG.md

@@ -6,6 +6,8 @@
 - Support scaffold commands for gems with `scaffold:gem:cw` pattern (`cs` as other suffix)
 - Support listing commands with `list:gems:cw` pattern (`cs` as other suffix)
 - Add -BinstalldirChooser=true / -Bcopyright=string / -Bvendor=string / -Bwin.menuGroup=string to Package class to support Windows packaging
+- Configure 'logging' to generate log files on Windows/Linux/Mac and syslog where available
+- Configure 'logging' to do async buffered logging via a thread to avoid impacting app performance with logging
 
 ### 0.4.1
 

TODO.md

@@ -6,8 +6,6 @@ Here is a list of tasks to do (moved to [CHANGELOG.md](CHANGELOG.md) once done).
 
 ### 0.5.0
 
-- Configure 'logging' to generate log files on Windows/Linux/Mac and syslog where available
-- Configure 'logging' to do async buffered logging via a thread to avoid impacting app performance with logging
 - Update all logger calls to be lazy blocks
 - Upgrade to glimmer 0.10.0 to take advantage of the new logging library
 

lib/ext/glimmer/config.rb

@@ -22,9 +22,50 @@ def import_swt_packages
         @@import_swt_packages
       end
 
+      # Returns Logging Devices. Default is [:stdout, :syslog]
+      def logging_devices
+        unless defined? @@logging_devices
+          @@logging_devices = [:stdout, :syslog]
+        end
+        @@logging_devices
+      end
+
+      # Logging Devices is an array of these possible values: :stdout (default), :stderr, :file, :syslog (default), :stringio
+      def logging_devices=(devices)
+        @@logging_devices = devices
+        reset_logger!
+      end
+
+      def logging_device_file_options
+        @@logging_device_file_options = {size: 1_000_000, age: 'daily', roll_by: 'number'} unless defined? @@logging_device_file_options
+        @@logging_device_file_options
+      end
+
+      def logging_device_file_options=(custom_options)
+        @@logging_device_file_options = custom_options
+        reset_logger!
+      end
+
+      def logging_appender_options
+        @@logging_appender_options = {async: true, auto_flushing: 25, write_size: 5, immediate_at: [:error, :fatal]} unless defined? @@logging_appender_options
+        @@logging_appender_options
+      end
+
+      def logging_appender_options=(custom_options)
+        @@logging_appender_options = custom_options
+        reset_logger!
+      end
+
       def reset_logger!
-        self.logger = Logging.logger(STDOUT).tap do |logger| 
+        self.logger = Logging.logger['glimmer'].tap do |logger| 
           logger.level = Logger::ERROR
+          appenders = []
+          appenders << Logging.appenders.stdout(logging_appender_options) if logging_devices.include?(:stdout)
+          appenders << Logging.appenders.stderr(logging_appender_options) if logging_devices.include?(:stderr)
+          appenders << Logging.appenders.rolling_file('log/glimmer.log', logging_appender_options.merge(logging_device_file_options)) if logging_devices.include?(:file)
+          Syslog.close if Syslog.opened? if logging_devices.include?(:syslog)
+          appenders << Logging.appenders.syslog('glimmer', logging_appender_options) if logging_devices.include?(:syslog)
+          logger.appenders = appenders
         end
       end
     end

spec/spec_helper.rb

@@ -144,3 +144,7 @@ module GlimmerSpec
     expect(widget.getStyle & style_value).to eq(style_value)
   end
 end
+
+# Enable when testing logging manually
+# Glimmer::Config.logging_devices = [:stdout, :file, :syslog]
+# Glimmer::Config.logger.level = :debug