fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] changelog2010-09-14 04:08 am

[dw-free] Enable warnings globally on dev servers

[commit: http://hg.dwscoalition.org/dw-free/rev/243791185af8]

http://bugs.dwscoalition.org/show_bug.cgi?id=2925

Stem the flow a little; let people opt out of the global warnings on the dev
servers.

Patch by [personal profile] fu.

Files modified:
  • cgi-bin/modperl_subs.pl
  • doc/config-local.pl.txt
--------------------------------------------------------------------------------
diff -r 68afe2e0fef4 -r 243791185af8 cgi-bin/modperl_subs.pl
--- a/cgi-bin/modperl_subs.pl	Tue Sep 14 11:41:09 2010 +0800
+++ b/cgi-bin/modperl_subs.pl	Tue Sep 14 12:08:18 2010 +0800
@@ -26,7 +26,7 @@ use LJ::Config;
 
 BEGIN {
     LJ::Config->load;
-    $^W = 1 if $LJ::IS_DEV_SERVER;
+    $^W = 1 if $LJ::IS_DEV_SERVER && $LJ::ENABLE_GLOBAL_WARNINGS;
 }
 
 use Apache::LiveJournal;
diff -r 68afe2e0fef4 -r 243791185af8 doc/config-local.pl.txt
--- a/doc/config-local.pl.txt	Tue Sep 14 11:41:09 2010 +0800
+++ b/doc/config-local.pl.txt	Tue Sep 14 12:08:18 2010 +0800
@@ -22,6 +22,9 @@
 
     # keep this enabled only if this site is a development server
     $IS_DEV_SERVER = 1;
+
+    # turn on warnings for all files. Will turn up a lot of stuff in the logs.
+    $ENABLE_GLOBAL_WARNINGS = 0;
 
     # home directory
     $HOME = $ENV{'LJHOME'};
--------------------------------------------------------------------------------