xfs
[Top] [All Lists]

Re: New XFS git tree on oss.sgi.com

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: New XFS git tree on oss.sgi.com
From: Niv Sardi <xaiki@xxxxxxx>
Date: Thu, 04 Dec 2008 10:58:48 +1100
Cc: Lachlan McIlroy <lachlan@xxxxxxx>, xfs@xxxxxxxxxxx
In-reply-to: <20081203130414.GB9681@xxxxxxxxxxxxx> (Christoph Hellwig's message of "Wed, 3 Dec 2008 08:04:14 -0500")
References: <492BA7AD.5080007@xxxxxxx> <20081125140553.GA16553@xxxxxxxxxxxxx> <492CA245.3000709@xxxxxxx> <20081126032710.GA19523@xxxxxxxxxxxxx> <ncctz9llxzq.fsf@xxxxxxxxxxxxxxxxxxxxxxx> <20081203130414.GB9681@xxxxxxxxxxxxx>
User-agent: Gnus/5.110011 (No Gnus v0.11) Emacs/23.0.60 (i486-pc-linux-gnu)
Christoph Hellwig <hch@xxxxxxxxxxxxx> writes:

> On Wed, Dec 03, 2008 at 02:48:57PM +1100, Niv Sardi wrote:
>
>> > Also
>> > http://oss.sgi.com/cgi-bin/gitweb.cgi?p=xfs/xfs.git;a=commitdiff;h=ca830fdf6231d0683f4ea4e9223e234c3a509063doesn't
>> >  seem to be needed.  None of those symbols seems to be used by
>> > either dmapi or xfsidbg, the only two modules using xfs symbols in the
>> > tree.
>> 
>> That's exactly why it's there, the revertion is actually moving from
>> what was in ptools to something sane.
>
> ??  The commit above adds tons of unused exports.  But hey, I'll just
> submit a patch to sort it out when I get some time..

Yes, I already did that …that's the previous commit… but Lachland wanted
the tree not too move too much at first.
The patch you want is the exact revert of the commit you pointed out,
it'll be checked in soon.

Cheers,
-- 
Niv Sardi

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