<div dir="ltr">On Wed, Aug 19, 2015 at 10:09 AM, Christoph Hellwig <<a href="mailto:hch@infradead.org">hch@infradead.org</a>> wrote:<br>><br>> On Wed, Aug 19, 2015 at 07:45:57AM +1000, Dave Chinner wrote:<br>> > Hmmm, looks like I accidentally comitted an earlier version of this<br>> > patch yesterday. Jan, can you send delta patches to what I committed<br>> > to address more recent review comments? Sorry, my bad.<br><br>I guess my chaotic versioning can be blamed. :-)<br><br>> Can you revert it for now? I think most of these changes are incorrect<br><br>I agree, revert seems like a better idea. Especially if we want to split things<br>like removing the XATTR_ dependency move into standalone commits.<br><br>> and should either be dropped for now by disabling fsx <br><br>I proposed dropping some tools earlier, but there was a valid point about<br>the possibility to break something when not even trying to compile it.<br>i.e. with the XATTR_ -> XFS_XATTR mentioned in the first reply to this patch<br>(which I already moved to a separate patch), it is necessary to add<br><br>#include "xfs/xfs_arch.h"<br>#include "xfs/xfs_format.h"<br><br>into libhandle/handle.c. But<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​ </div>I found<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​ </div>this<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​ </div>when I tried to compile it. If it was<br>in fsr and fsr wasn't compiling, I would most likely send a patch which would<br>be thrown<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​ </div>back<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​ </div>on my head immediately. ;-)<br><br>So I think<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​ ​</div>it is better to have "bad dummy stuff" than not compile it<br>at all<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​</div>.<br><br>> or will need<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​ </div>a major rework.<br><br>Working on it. :-)<br><br>Cheers,<br>Jan<br><br>--<br>Jan Tulak<br><a href="mailto:jtulak@redhat.com">jtulak@redhat.com</a> / <a href="mailto:jan@tulak.me">jan@tulak.me</a><div><div><div class="gmail_extra">
</div></div></div></div>