xfs
[Top] [All Lists]

Re: Review: xfsprogs with libblkid enabled for Debian

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: Review: xfsprogs with libblkid enabled for Debian
From: Nathan Scott <nathans@xxxxxxxxxx>
Date: Fri, 20 Aug 2010 07:41:16 +1000 (EST)
Cc: xfs <xfs@xxxxxxxxxxx>
In-reply-to: <20100819121153.GB16045@xxxxxxxxxxxxx>
----- "Christoph Hellwig" <hch@xxxxxxxxxxxxx> wrote:

> On Wed, Aug 18, 2010 at 03:06:28PM +1000, nathans@xxxxxxxxxx wrote:
> > Hi guys,
> > 
> > Attached patch adds support for libblkid in Debian builds.
> > There's a couple of other trivial things fixed up while the
> > package was building - man page typos reported by lintian,
> > fix source tarball generation (and deb build) when libblkid
> > is enabled (*must* descend into libdisk to pick up sources)
> > ... but its all pretty straight forward.
> > 
> > I've checked that mkfs.xfs is linked as expected too. :)
> 
> All the changes in here look good, but I'd be much happier if you'd
> submit separate changes with separate changelogs for the manpage
> updates and the libdisk packaging fix.
> 

Yep, will do.

> As for the blkid-dev build-dep: we need at least version 2.17 of it
> as that introduced the symbols we need, I think a versioned
> dependency for this would be in order.

Good point, will do.

> The doc/CHANGES update should include the names of external
> contributors
> where applicable, and currently the release date for is set as part
> of the release.sh process.

Oh, OK - hmm, will go through the git logs again & see what I can find.

> I'm fine with pushing out a new 3.1.3 release as it's time anyway,
> but
> can we also include Dave's patch to add XFS_IOC_ZERO_RANGE support to
> xfs_io and the xfsctl manpage first?

Sure, will wait for that before I commit anything.

> Next week also has Alex back from vacation, so I'd suggest waiting if
> he has any input on this.

No worries.

cheers.

-- 
Nathan

<Prev in Thread] Current Thread [Next in Thread>