Re: [PATCH 05/11] xfsprogs: missing and dummy calls for OS X support

[Jan Tulak <jtulak@xxxxxxxxxx>]

On Wed, Aug 19, 2015 at 10:09 AM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
>
> On Wed, Aug 19, 2015 at 07:45:57AM +1000, Dave Chinner wrote:
> > Hmmm, looks like I accidentally comitted an earlier version of this
> > patch yesterday. Jan, can you send delta patches to what I committed
> > to address more recent review comments? Sorry, my bad.

I guess my chaotic versioning can be blamed. :-)

> Can you revert it for now? I think most of these changes are incorrect

I agree, revert seems like a better idea. Especially if we want to split things
like removing the XATTR_ dependency move into standalone commits.

> and should either be dropped for now by disabling fsx

I proposed dropping some tools earlier, but there was a valid point about
the possibility to break something when not even trying to compile it.
i.e. with the XATTR_ -> XFS_XATTR mentioned in the first reply to this patch
(which I already moved to a separate patch), it is necessary to add

#include "xfs/xfs_arch.h"
#include "xfs/xfs_format.h"

into libhandle/handle.c. But

âÂ

I found

âÂ

this

âÂ

when I tried to compile it. If it was
in fsr and fsr wasn't compiling, I would most likely send a patch which would
be thrown

âÂ

back

âÂ

on my head immediately. ;-)

So I think

â â

it is better to have "bad dummy stuff" than not compile it
at all

â

.

> or will need

âÂ

a major rework.

Working on it. :-)

Cheers,
Jan

--
Jan Tulak
jtulak@xxxxxxxxxx / jan@xxxxxxxx