You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@bigtop.apache.org by bm...@apache.org on 2012/04/12 03:17:57 UTC

svn commit: r1325090 - /incubator/bigtop/branches/hadoop-0.23/bigtop-packages/src/common/flume/install_flume.sh

Author: bmahe
Date: Thu Apr 12 01:17:57 2012
New Revision: 1325090

URL: http://svn.apache.org/viewvc?rev=1325090&view=rev
Log:
BIGTOP-510. log4j property must be set with property in for Flume NG pkg

Modified:
    incubator/bigtop/branches/hadoop-0.23/bigtop-packages/src/common/flume/install_flume.sh

Modified: incubator/bigtop/branches/hadoop-0.23/bigtop-packages/src/common/flume/install_flume.sh
URL: http://svn.apache.org/viewvc/incubator/bigtop/branches/hadoop-0.23/bigtop-packages/src/common/flume/install_flume.sh?rev=1325090&r1=1325089&r2=1325090&view=diff
==============================================================================
--- incubator/bigtop/branches/hadoop-0.23/bigtop-packages/src/common/flume/install_flume.sh (original)
+++ incubator/bigtop/branches/hadoop-0.23/bigtop-packages/src/common/flume/install_flume.sh Thu Apr 12 01:17:57 2012
@@ -146,6 +146,8 @@ chmod 755 $wrapper
 
 install -d -m 0755 $PREFIX/$ETC_DIR/conf.empty
 (cd ${BUILD_DIR}/conf && tar cf - .) | (cd $PREFIX/$ETC_DIR/conf.empty && tar xf -)
+# XXX FIXME: We should handle this upstream more gracefully
+sed -i -e "s|flume\.log\.dir=.*|flume.log.dir=/var/log/flume-ng|" $PREFIX/$ETC_DIR/conf.empty/log4j.properties
 touch $PREFIX/$ETC_DIR/conf.empty/flume.conf
 
 unlink $PREFIX/$FLUME_DIR/conf || /bin/true