From: jhf@rivenstone.net (Joseph Fannin)
To: linux-kernel@vger.kernel.org
Cc: torvalds@transmeta.com
Subject: [PATCH] Fix 2.5-bk build error
Date: Fri, 1 Nov 2002 12:28:07 -0500 [thread overview]
Message-ID: <20021101172807.GA982@caphernaum.rivenstone.net> (raw)
In-Reply-To: <Pine.LNX.4.44.0211011239290.6949-100000@serv>
[-- Attachment #1: Type: text/plain, Size: 1960 bytes --]
On Fri, Nov 01, 2002 at 12:56:20PM +0100, Roman Zippel wrote:
> On Thu, 31 Oct 2002, Joseph Fannin wrote:
>
> > > # Meta block cache for Extended Attributes (ext2/ext3)
> > > config FS_MBCACHE
> > > tristate
> > > depends on EXT2_FS_XATTR || EXT3_FS_XATTR
> > > default m if EXT2_FS=m || EXT3_FS=m
> > > default y if EXT2_FS=y || EXT3_FS=y
> >
> > "If multiple default statements are visible only the first is
> > used."
> >
> > So the two default lines above need to be reversed. This seems
> > backwards to me (the last should be used), but I've said enough.
>
> Well, I had to pick something and using the first is easier to implement,
> it's just different to cml1, which used the last definition.
> BTW xconfig is a nice way to see how the config back end works, you can
> enable "Show All Options" and above entry will also be visible and you can
> watch how the value changes depending on the inputs.
> BTW2 in the future above can be simplified into
>
> config FS_MBCACHE
> tristate
> depends on EXT2_FS_XATTR || EXT3_FS_XATTR
> default EXT2_FS || EXT3_FS
>
Okay, here's a patch that does that. Linus, this fixes a build
error in your current -bk tree that happens when one of ext[23] is a
module and the other is built-in. Please apply it.
diff -urN linux-2.5.45/fs/Kconfig linux/fs/Kconfig
--- linux-2.5.45/fs/Kconfig 2002-11-01 11:42:04.000000000 -0500
+++ linux/fs/Kconfig 2002-11-01 11:59:50.000000000 -0500
@@ -1457,8 +1457,7 @@
config FS_MBCACHE
tristate
depends on EXT2_FS_XATTR || EXT3_FS_XATTR
- default m if EXT2_FS=m || EXT3_FS=m
- default y if EXT2_FS=y || EXT3_FS=y
+ default EXT2_FS || EXT3_FS
# Posix ACL utility routines (for now, only ext2/ext3)
config FS_POSIX_ACL
--
Joseph Fannin
jhf@rivenstone.net
"Bull in pure form is rare; there is usually some contamination by data."
-- William Graves Perry Jr.
[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]
next prev parent reply other threads:[~2002-11-01 17:24 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-10-31 8:28 [PATCH] [BK] 0/11 Ext2/3 Updates: Extended attributes, ACL, etc tytso
2002-11-01 0:24 ` Joseph Fannin
2002-11-01 0:47 ` Joseph Fannin
2002-11-01 1:06 ` Joseph Fannin
2002-11-01 11:56 ` Roman Zippel
2002-11-01 17:28 ` Joseph Fannin [this message]
2002-11-02 6:54 ` [PATCH] Fix 2.5-bk build error Theodore Ts'o
2002-11-02 11:53 ` Roman Zippel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20021101172807.GA982@caphernaum.rivenstone.net \
--to=jhf@rivenstone.net \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@transmeta.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®