From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (8.14.3/8.14.3/SuSE Linux 0.8) with ESMTP id qA1GIRtv036962 for ; Thu, 1 Nov 2012 11:18:27 -0500 Date: Thu, 1 Nov 2012 11:20:16 -0500 From: Ben Myers Subject: Re: [patch 1/4] xfstests: update version and changelog Message-ID: <20121101162016.GC9783@sgi.com> References: <20121031192545.662795881@sgi.com> <1066591451.20029469.1351717231452.JavaMail.root@redhat.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1066591451.20029469.1351717231452.JavaMail.root@redhat.com> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: xfs-bounces@oss.sgi.com Errors-To: xfs-bounces@oss.sgi.com To: Nathan Scott Cc: xfs@oss.sgi.com Hey Nathan, On Wed, Oct 31, 2012 at 05:00:31PM -0400, Nathan Scott wrote: > They all look good to me, thanks Ben. One minor thought, > for future updates... > > ----- Original Message ----- > > ... > > Index: b/VERSION > > =================================================================== > > --- a/VERSION 2012-10-31 13:44:44.931946786 -0500 > > +++ b/VERSION 2012-10-31 13:44:50.832006406 -0500 > > @@ -3,5 +3,5 @@ > > # > > PKG_MAJOR=1 > > PKG_MINOR=1 > > -PKG_REVISION=0 > > +PKG_REVISION=1 > > PKG_BUILD=1 > > Index: b/configure.ac > > =================================================================== > > --- a/configure.ac 2012-10-31 13:45:41.661946816 -0500 > > +++ b/configure.ac 2012-10-31 13:45:48.072005921 -0500 > > @@ -1,4 +1,4 @@ > > -AC_INIT([xfstests], [1.1.0]) > > +AC_INIT([xfstests], [1.1.1]) > > AC_PREREQ(2.50) > > AC_CONFIG_SRCDIR([src/xfsctl.c]) > > AC_PACKAGE_GLOBALS(xfstests) > > I wonder if generating the VERSION file from configure makes > sense, since we are duplicating that info nowadays? I think it's a good idea. There are a few other deps on PKG_* so maybe it would be better to just have configure.ac use PKG* instead of hard coding the version there. > Oh, feel free to add Reviewed-by me for all of these patches, > if you like. Cool, thanks! Regards, Ben _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs