Return-Path: X-Original-To: apmail-aurora-reviews-archive@minotaur.apache.org Delivered-To: apmail-aurora-reviews-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 5843D11172 for ; Tue, 15 Apr 2014 23:41:09 +0000 (UTC) Received: (qmail 69086 invoked by uid 500); 15 Apr 2014 23:41:08 -0000 Delivered-To: apmail-aurora-reviews-archive@aurora.apache.org Received: (qmail 69050 invoked by uid 500); 15 Apr 2014 23:41:08 -0000 Mailing-List: contact reviews-help@aurora.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@aurora.incubator.apache.org Delivered-To: mailing list reviews@aurora.incubator.apache.org Received: (qmail 69042 invoked by uid 99); 15 Apr 2014 23:41:08 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 15 Apr 2014 23:41:08 +0000 X-ASF-Spam-Status: No, hits=-1998.8 required=5.0 tests=ALL_TRUSTED,HTML_MESSAGE,RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO mail.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with SMTP; Tue, 15 Apr 2014 23:41:06 +0000 Received: (qmail 68745 invoked by uid 99); 15 Apr 2014 23:40:46 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 15 Apr 2014 23:40:46 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id A40251D5F0E; Tue, 15 Apr 2014 23:40:41 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============7024100931971510910==" MIME-Version: 1.0 Subject: Re: Review Request 20377: Use git-describe to generate version number. From: "Jake Farrell" To: "Bill Farner" , "Jake Farrell" Cc: "Aurora" , "Kevin Sweeney" Date: Tue, 15 Apr 2014 23:40:41 -0000 Message-ID: <20140415234041.19850.15148@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Jake Farrell" X-ReviewGroup: Aurora X-ReviewRequest-URL: https://reviews.apache.org/r/20377/ X-Sender: "Jake Farrell" References: <20140415212308.19850.34482@reviews.apache.org> In-Reply-To: <20140415212308.19850.34482@reviews.apache.org> Reply-To: "Jake Farrell" X-ReviewRequest-Repository: aurora X-Virus-Checked: Checked by ClamAV on apache.org --===============7024100931971510910== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20377/#review40478 ----------------------------------------------------------- Thoughts on moving all the repeated calls in each BUILD file to ./pants and exporting as an env var? build.gradle looks fine, only issue is that source releases typically will not have .git included with them so we can not use git describe for the source release. Should change so that if git describe fails then it falls through to .auroraversion - Jake Farrell On April 15, 2014, 9:23 p.m., Kevin Sweeney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20377/ > ----------------------------------------------------------- > > (Updated April 15, 2014, 9:23 p.m.) > > > Review request for Aurora, Jake Farrell and Bill Farner. > > > Bugs: AURORA-320 > https://issues.apache.org/jira/browse/AURORA-320 > > > Repository: aurora > > > Description > ------- > > Use git-describe to generate version number instead of .auroraversion. > > > Diffs > ----- > > build.gradle ea488414d366ece89ba25e42b98f6ad7ddbf3378 > src/main/python/apache/aurora/client/BUILD 118c4a080b2aac88f1fb6365787b3bb55a665bea > src/main/python/apache/aurora/common/BUILD 4646c05972b678371e9b030e042b8899568b8374 > src/main/python/apache/aurora/config/BUILD d7467280843e3c5f9f16e46258a12b07b7b99f49 > src/main/python/apache/aurora/executor/BUILD 9f0cad0244c46d3175b492f65b37e1cb0619aa34 > src/main/python/apache/thermos/BUILD 19b242dba75017e57a25def26530fffcf0c758a6 > src/main/python/apache/thermos/common/BUILD a256f2e015c2984adc8aa34c3a7e99a50a978083 > src/main/python/apache/thermos/config/BUILD b0806057db4d221467a29ff5c2761113e7b1030c > src/main/python/apache/thermos/core/BUILD 77bd289cdfe8ffa17a011e7d4d8debe44d45310a > src/main/python/apache/thermos/monitoring/BUILD 448baadf641bf44d28e91b4d0c5f6303f4ae62e7 > src/main/python/apache/thermos/observer/BUILD 79168825d1fd5660a01a8b229a765149bc84b358 > src/main/thrift/org/apache/aurora/gen/BUILD dce379cedc0fe025e59dbed6186f650a58bc42c2 > src/main/thrift/org/apache/thermos/BUILD 2765575b3c29cfed4dea64e0132443a9874ce83c > > Diff: https://reviews.apache.org/r/20377/diff/ > > > Testing > ------- > > ./gradlew assemble > => dist/libs/aurora-0.3.0-209-g31ec70b.jar > > ./build-support/release/make-python-sdists > => dist/apache.aurora.client-0.3.0-209-g31ec70b.tar.gz > > echo 'also marks dirty' >> DISCLAIMER > > ./gradlew assemble > => dist/libs/aurora-0.3.0-209-g31ec70b-dirty.jar > > ./build-support/release/make-python-sdists > => dist/apache.aurora.client-0.3.0-209-g31ec70b-dirty.tar.gz > > > Thanks, > > Kevin Sweeney > > --===============7024100931971510910==--