You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4cxx-dev@logging.apache.org by ca...@apache.org on 2007/11/29 00:14:15 UTC
svn commit: r599193 - in /logging/log4cxx/trunk: INSTALL
src/test/cpp/util/transformer.cpp
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);