Author: carnold
Date: Wed Nov 28 15:14:14 2007
New Revision: 599193

URL: http://svn.apache.org/viewvc?rev=599193&view=rev
Log:
LOGCXX-212: Improve msg when sed not present on path, add gzip and zip to 
INSTALL

Modified:
    logging/log4cxx/trunk/INSTALL
    logging/log4cxx/trunk/src/test/cpp/util/transformer.cpp

Modified: logging/log4cxx/trunk/INSTALL
URL: 
http://svn.apache.org/viewvc/logging/log4cxx/trunk/INSTALL?rev=599193&r1=599192&r2=599193&view=diff
==============================================================================
--- logging/log4cxx/trunk/INSTALL (original)
+++ logging/log4cxx/trunk/INSTALL Wed Nov 28 15:14:14 2007
@@ -35,7 +35,7 @@
 
 GNU patch on command path.
 
-GNU sed on command path (required for tests)
+GNU sed, gzip and zip on command path (required for tests)
 
 Connection to internet
 

Modified: logging/log4cxx/trunk/src/test/cpp/util/transformer.cpp
URL: 
http://svn.apache.org/viewvc/logging/log4cxx/trunk/src/test/cpp/util/transformer.cpp?rev=599193&r1=599192&r2=599193&view=diff
==============================================================================
--- logging/log4cxx/trunk/src/test/cpp/util/transformer.cpp (original)
+++ logging/log4cxx/trunk/src/test/cpp/util/transformer.cpp Wed Nov 28 15:14:14 
2007
@@ -209,6 +209,9 @@
 
         apr_proc_t pid;
         stat = apr_proc_create(&pid,"sed", args, NULL, attr, pool);
+        if (stat != APR_SUCCESS) {
+            puts("Error invoking sed, sed must be on the path in order to run 
unit tests");
+        }
         assert(stat == APR_SUCCESS);
 
         apr_proc_wait(&pid, NULL, NULL, APR_WAIT);


Reply via email to