> On June 24, 2012, 8:38 p.m., Ralph Goers wrote: > > Also, won't this be a problem on Windows? Last I checked it didn't support > > shell scripts from the command line without requiring cygwin.
Thanks, Ralph! It's really a good question to consider this if Flume NG support build on Windows. Current mechanism is the same as Hadoop and Hbase way now, which will use sh script to generate version info when package. I found a JIRA about build on Windows 7 x64: https://issues.apache.org/jira/browse/FLUME-1174. Do we have plan for build on Windows? From https://cwiki.apache.org/FLUME/, only below are declared on unix-like systems: Ubuntu 9.4+ (DEB compatible) Centos 5.3+ (RPM compatible) RHEL 5.5+ SLES 11 Mac OS X Regards Leslin - Leslin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5463/#review8521 ----------------------------------------------------------- On June 25, 2012, 2:20 p.m., Leslin (Hong Xiang Lin) wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5463/ > ----------------------------------------------------------- > > (Updated June 25, 2012, 2:20 p.m.) > > > Review request for Flume. > > > Description > ------- > > Add feature to let flume-ng get version info: > eslin@ubtServer:/usr/lib/flume-ng/bin$ flume-ng version > Apache Flume version: 1.2.0-incubating-SNAPSHOT > Built-By: 1.2.0-incubating-SNAPSHOT > > > This addresses bug FLUME-1240. > https://issues.apache.org/jira/browse/FLUME-1240 > > > Diffs > ----- > > bin/flume-ng 43d1766 > flume-ng-core/pom.xml e4c8104 > flume-ng-core/scripts/saveVersion.sh PRE-CREATION > flume-ng-core/src/main/java/org/apache/flume/VersionAnnotation.java > PRE-CREATION > flume-ng-core/src/main/java/org/apache/flume/tools/VersionInfo.java > PRE-CREATION > flume-ng-core/src/test/java/org/apache/flume/tools/TestVersionInfo.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/5463/diff/ > > > Testing > ------- > > Test with: > flume-ng version > flume-ng agent -n agent -f /usr/lib/flume-ng/conf/flume-conf.properties > > > Thanks, > > Leslin (Hong Xiang Lin) > >
