8000 Merge pull request #125 from jimschubert/master · LikeABossProgrammer/log4js-node@83271e4 · GitHub
[go: up one dir, main page]
Skip to content

Commit 83271e4

Browse files
author
Gareth Jones
committed
Merge pull request log4js-node#125 from jimschubert/master
Allow for somewhat standard debugging calls
2 parents 4b7cf58 + f3271a3 commit 83271e4

File tree

3 files changed

+15
-7
lines changed

3 files changed

+15
-7
lines changed

‎lib/streams/BaseRollingFileStream.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,11 @@ var fs = require('fs'),
22
stream = require('stream'),
33
util = require('util');
44

5-
function debug(message) {
6-
// console.log(message);
5+
var debug;
6+
if (process.env.NODE_DEBUG && /\blog4js\b/.test(process.env.NODE_DEBUG)) {
7+
debug = function(message) { console.error('LOG4JS: (BaseRollingFileStream) %s', message); };
8+
} else {
9+
debug = function() { };
710
}
811

912
module.exports = BaseRollingFileStream;

‎lib/streams/DateRollingFileStream.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,11 @@ var BaseRollingFileStream = require('./BaseRollingFileStream'),
66

77
module.exports = DateRollingFileStream;
88

9-
function debug(message) {
10-
// console.log(message);
9+
var debug;
10+
if (process.env.NODE_DEBUG && /\blog4js\b/.test(process.env.NODE_DEBUG)) {
11+
debug = function(message) { console.error('LOG4JS: (DateRollingFileStream) %s', message); };
12+
} else {
13+
debug = function() { };
1114
}
1215

1316
function DateRollingFileStream(filename, pattern, options, now) {

‎lib/streams/RollingFileStream.js

Lines changed: 5 additions & 3 deletions
14
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,11 @@ var BaseRollingFileStream = require('./BaseRollingFileStream'),
44
fs = require('fs'),
55
async = require('async');
66

7-
function debug() {
8-
// util.debug(message);
9-
// console.log.apply(console, arguments);
7+
var debug;
8+
if (process.env.NODE_DEBUG && /\blog4js\b/.test(process.env.NODE_DEBUG)) {
9+
debug = function(message) { console.error('LOG4JS: (RollingFileStream) %s', message); };
10+
} else {
11+
debug = function() { };
1012
}
1113

12
module.exports = RollingFileStream;

0 commit comments

Comments
 (0)
0