Return-Path: Delivered-To: apmail-lucene-hadoop-dev-archive@locus.apache.org Received: (qmail 32750 invoked from network); 2 Oct 2006 19:01:33 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 2 Oct 2006 19:01:33 -0000 Received: (qmail 48592 invoked by uid 500); 2 Oct 2006 19:01:32 -0000 Delivered-To: apmail-lucene-hadoop-dev-archive@lucene.apache.org Received: (qmail 48499 invoked by uid 500); 2 Oct 2006 19:01:32 -0000 Mailing-List: contact hadoop-dev-help@lucene.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: hadoop-dev@lucene.apache.org Delivered-To: mailing list hadoop-dev@lucene.apache.org Received: (qmail 48486 invoked by uid 99); 2 Oct 2006 19:01:31 -0000 Received: from idunn.apache.osuosl.org (HELO idunn.apache.osuosl.org) (140.211.166.84) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 02 Oct 2006 12:01:31 -0700 X-ASF-Spam-Status: No, hits=0.0 required=5.0 tests= Received: from [209.237.227.198] ([209.237.227.198:38420] helo=brutus.apache.org) by idunn.apache.osuosl.org (ecelerity 2.1.1.8 r(12930)) with ESMTP id 4C/65-24395-90261254 for ; Mon, 02 Oct 2006 12:01:29 -0700 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 9850A71430D for ; Mon, 2 Oct 2006 12:01:21 -0700 (PDT) Message-ID: <13294377.1159815681621.JavaMail.root@brutus> Date: Mon, 2 Oct 2006 12:01:21 -0700 (PDT) From: "Doug Cutting (JIRA)" To: hadoop-dev@lucene.apache.org Subject: [jira] Commented: (HADOOP-567) The build script should record the Hadoop version into the build In-Reply-To: <28945636.1159772659594.JavaMail.root@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N [ http://issues.apache.org/jira/browse/HADOOP-567?page=comments#action_12439273 ] Doug Cutting commented on HADOOP-567: ------------------------------------- This is very cool! +1 A few naming nits. The script name should end with '.sh'. And the field names in the generated class are 'static final' and hence their names should be all caps. It would also be good to get some javadoc in that class. And instead of BuildStamp, we might just call this Version or HadoopVersion, no? Finally, I'm not sure why util.VersionInfo is needed. Under what situations is this better than simply referencing the constants in the generated class? Lucene does something related with: http://lucene.apache.org/java/docs/api/org/apache/lucene/LucenePackage.html The version is built into the jar manifest in the jar-core target of: http://svn.apache.org/viewvc/lucene/java/trunk/common-build.xml?view=markup Then it's accessed with: http://svn.apache.org/viewvc/lucene/java/trunk/src/java/org/apache/lucene/LucenePackage.java?view=markup This does not have the subversion revision, which would be really nice. Also, in addition to the revision, it would be good to have the repository URL. Then one could always recreate the sources. Do you think there'd be much of an advantage to use Java's Package stuff for this? > The build script should record the Hadoop version into the build > ---------------------------------------------------------------- > > Key: HADOOP-567 > URL: http://issues.apache.org/jira/browse/HADOOP-567 > Project: Hadoop > Issue Type: New Feature > Components: util > Affects Versions: 0.6.2 > Reporter: Owen O'Malley > Assigned To: Owen O'Malley > Fix For: 0.7.0 > > Attachments: version.patch > > > It would be good to compile the Hadoop version, subversion revision, and compilation date into the hadoop.jar file. > The web/ui would display the version for each of the server home page. > I'd also add "bin/hadoop version" to print the version information. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira