mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
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 --]

  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®