Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 25104 - mod_xslt should not have debugging on
Summary: mod_xslt should not have debugging on
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Apache Team - Bugzilla Reports
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-07-23 02:00 UTC by Take Vos
Modified: 2004-04-23 06:59 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
patch to fix logging a bit (mod_xslt-1.0.5a-nodebug.patch,1.93 KB, patch)
2003-07-23 03:10 UTC, Take Vos
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Take Vos 2003-07-23 02:00:45 UTC
I noticed that mod_xslt writes notice messages, even if LogLevel in apache is set to warn. 
And then I noticed the #define DEBUG in mod-xslt-1.0.5a/src/logging.c 
 
I added the following patch to files directory voor the ebuild: 
diff -urN mod-xslt-1.0.5a/src/logging.c mod-xslt-1.0.5a-nodebug/src/logging.c 
--- mod-xslt-1.0.5a/src/logging.c       2002-08-13 12:03:07.000000000 +0200 
+++ mod-xslt-1.0.5a-nodebug/src/logging.c       2003-07-23 10:47:38.000000000 +0200 
@@ -49,7 +49,6 @@ 
 #ifndef LOGLEVEL 
 #define LOGLEVEL 8 
 #endif 
-#define DEBUG 
 
 #ifndef LOGPREFIX 
 #define LOGPREFIX "mod-xslt: %s" 
 
That solved it for me. 

Reproducible: Always
Steps to Reproduce:
1. Set LogLevel in apache2.conf to warn (that is the default) 
2. (re)load a page that uses mod_xslt 
3. Look in /var/log/apache2/error_log for [notice] mod_xslt:
Comment 1 Take Vos 2003-07-23 02:11:09 UTC
Oops that did not work. I'll try to make an other patch. 
Comment 2 Take Vos 2003-07-23 03:10:56 UTC
Created attachment 14900 [details, diff]
patch to fix logging a bit
Comment 3 Take Vos 2003-07-23 03:12:01 UTC
new better patch. 
 
 
Comment 4 Chuck Short (RETIRED) gentoo-dev 2004-04-23 06:59:11 UTC
Patch has been added.