[Opensrf-commits] r980 - trunk/src/libopensrf

svn at svn.open-ils.org svn at svn.open-ils.org
Wed Jun 27 23:16:56 EDT 2007


Author: miker
Date: 2007-06-27 23:14:06 -0400 (Wed, 27 Jun 2007)
New Revision: 980

Modified:
   trunk/src/libopensrf/osrfConfig.c
Log:
Patch from Scott McKellar:

At one time. if we had trouble reading or parsing the configuration
file, we would issue log messages to complain.  However those log
messages didn't have anywhere to go because, not having loaded the
configuration file, we didn't know where to write the messages.

The earlier patch wrote the same messages to standard error as well
as to the logging machinery, so that they would be visible.

These extra writes are no longer necessary,  With recent changes to
the logging machinery, all log messages will go to standard error
if no other destination has been defined for them.

Accordingly this patch removes two messages, now redundant, that
were being written to standard error.



Modified: trunk/src/libopensrf/osrfConfig.c
===================================================================
--- trunk/src/libopensrf/osrfConfig.c	2007-06-28 01:07:55 UTC (rev 979)
+++ trunk/src/libopensrf/osrfConfig.c	2007-06-28 03:14:06 UTC (rev 980)
@@ -45,7 +45,6 @@
 	
 	xmlDocPtr doc = xmlParseFile(configFile);
 	if(!doc) {
-		fprintf( stderr, "osrfConfigInit: Unable to parse XML config file %s\n", configFile);
 		osrfLogWarning( OSRF_LOG_MARK, "Unable to parse XML config file %s", configFile);
 		return NULL;
 	}
@@ -56,7 +55,6 @@
 	xmlFreeDoc(doc);
 
 	if(!json_config ) {
-		fprintf( stderr, "osrfConfigInit: xmlDocToJSON failed for config %s\n", configFile);
 		osrfLogWarning( OSRF_LOG_MARK, "xmlDocToJSON failed for config %s", configFile);
 		return NULL;
 	}	



More information about the opensrf-commits mailing list