Ejegg has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/277793

Change subject: Enable syslog on OSX
......................................................................

Enable syslog on OSX

Modern-syslog may support it better. Let's see!

Removed OS check entirely, though syslog doesn't exist on Windows.
Not even trying to support developing this there.

Change-Id: I716c7cab16eaa6e448b032efbacce605337fcb55
---
M logger.js
1 file changed, 9 insertions(+), 15 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/dash 
refs/changes/93/277793/1

diff --git a/logger.js b/logger.js
index 00e8a89..b0ec072 100644
--- a/logger.js
+++ b/logger.js
@@ -1,24 +1,18 @@
-var hasSyslog = !!process.platform.match(/linux/),
-       LOG_DEBUG = 0,
+var LOG_DEBUG = 0,
        LOG_INFO = 1,   
        LOG_ERR = 2,
-       syslog,
+       syslog = require( 'modern-syslog' ),
        constMap = [];
 
-if ( hasSyslog ) {
-       syslog = require( 'modern-syslog' );
-       /*jslint bitwise: true*/
-       syslog.init( 'dash', syslog.LOG_PID | syslog.LOG_ODELAY, 
syslog.LOG_LOCAL0 );
-       /*jslint bitwise: false*/
-       constMap[LOG_DEBUG] = syslog.LOG_DEBUG;
-       constMap[LOG_INFO] = syslog.LOG_INFO;
-       constMap[LOG_ERR] = syslog.LOG_ERR;
-}
+/*jslint bitwise: true*/
+syslog.init( 'dash', syslog.LOG_PID | syslog.LOG_ODELAY, syslog.LOG_LOCAL0 );
+/*jslint bitwise: false*/
+constMap[LOG_DEBUG] = syslog.LOG_DEBUG;
+constMap[LOG_INFO] = syslog.LOG_INFO;
+constMap[LOG_ERR] = syslog.LOG_ERR;
 
 function log( level, message ) {
-       if ( hasSyslog ) {
-               syslog.log( constMap[level], message );
-       }
+       syslog.log( constMap[level], message );
        if ( level === LOG_ERR ) {
                console.error( message );
        } else {

-- 
To view, visit https://gerrit.wikimedia.org/r/277793
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I716c7cab16eaa6e448b032efbacce605337fcb55
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/dash
Gerrit-Branch: master
Gerrit-Owner: Ejegg <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to