From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Liu Subject: Re: [git pull] vfs pile 1.5 Date: Tue, 26 Jul 2011 16:52:03 +0800 Message-ID: References: <20110725185649.GA22133@ZenIV.linux.org.uk> <20110726033535.GA11939@x4.trippels.de> <20110726082146.GB11939@x4.trippels.de> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Linus Torvalds , Al Viro , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, Christoph Hellwig To: Markus Trippelsdorf Return-path: Received: from mail-ey0-f171.google.com ([209.85.215.171]:34450 "EHLO mail-ey0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751165Ab1GZIwG convert rfc822-to-8bit (ORCPT ); Tue, 26 Jul 2011 04:52:06 -0400 In-Reply-To: <20110726082146.GB11939@x4.trippels.de> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: maybe some one have not use #ifdef CONFIG_FS_POSIX_ACL with fs/xfs/linux-2.6/xfs_iops.c:1025:14 :) 2011/7/26 Markus Trippelsdorf : > On 2011.07.25 at 21:52 -0700, Linus Torvalds wrote: >> On Mon, Jul 25, 2011 at 8:35 PM, Markus Trippelsdorf >> wrote: >> > >> > This breaks the build for "# CONFIG_FS_POSIX_ACL is not set" confi= gs: >> >> Of course it does. And as usual, my exhaustive testing was for the >> allyesconfig case, rather than doing both allyesconfig and allnoconf= ig >> like I should. >> >> Bah. >> >> Does the attached trivial patch fix everything for you? > > Yes, thanks. > > But now xfs fails to build when CONFIG_FS_POSIX_ACL is not set: > > =A0CC =A0 =A0 =A0fs/xfs/linux-2.6/xfs_xattr.o > fs/xfs/linux-2.6/xfs_iops.c:1025:14: error: =91xfs_get_acl=92 undecla= red here (not in a function) > > -- > Markus > -- > To unsubscribe from this list: send the line "unsubscribe linux-fsdev= el" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at =A0http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel= " in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html