All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steve French <smfrench@gmail.com>
To: CIFS <linux-cifs@vger.kernel.org>
Cc: David Howells <dhowells@redhat.com>
Subject: cifs: fix indentation in make menuconfig options
Date: Mon, 30 Jan 2023 19:43:40 -0600	[thread overview]
Message-ID: <CAH2r5mvrsr8Gh0oA=8C+d8EEue1Hsx2OMqr9hQntejRo=H31dg@mail.gmail.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 1011 bytes --]

The options that are displayed for the smb3.1.1/cifs client
    in "make menuconfig" are confusing because some of them are
    not indented making them not appear to be related to cifs.ko
    Fix that by adding an if/endif (similar to what ceph and 9pm did)
    if fs/cifs/Kconfig

    Suggested-by: David Howells <dhowells@redhat.com>
    Signed-off-by: Steve French <stfrench@microsoft.com>

diff --git a/fs/cifs/Kconfig b/fs/cifs/Kconfig
index 1e021c0b0653..bbf63a9eb927 100644
--- a/fs/cifs/Kconfig
+++ b/fs/cifs/Kconfig
@@ -175,6 +175,8 @@ config CIFS_NFSD_EXPORT
        help
          Allows NFS server to export a CIFS mounted share (nfsd over cifs)

+if CIFS
+
 config CIFS_SMB_DIRECT
        bool "SMB Direct support"
        depends on CIFS=m && INFINIBAND && INFINIBAND_ADDR_TRANS ||
CIFS=y && INFINIBAND=y && INFINIBAND_ADDR_TRANS=y
@@ -198,3 +200,5 @@ config CIFS_ROOT
          Enables root file system support over SMB protocol.

          Most people say N here.
+
+endif


-- 
Thanks,

Steve

[-- Attachment #2: 0001-cifs-fix-indentation-in-make-menuconfig-options.patch --]
[-- Type: text/x-patch, Size: 1227 bytes --]

From e8c718e4cc5de2de456492147fa5cac20660fa7e Mon Sep 17 00:00:00 2001
From: Steve French <stfrench@microsoft.com>
Date: Mon, 30 Jan 2023 19:32:52 -0600
Subject: [PATCH] cifs: fix indentation in make menuconfig options

The options that are displayed for the smb3.1.1/cifs client
in "make menuconfig" are confusing because some of them are
not indented making them not appear to be related to cifs.ko
Fix that by adding an if/endif (similar to what ceph and 9pm did)
if fs/cifs/Kconfig

Suggested-by: David Howells <dhowells@redhat.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
---
 fs/cifs/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/fs/cifs/Kconfig b/fs/cifs/Kconfig
index 1e021c0b0653..bbf63a9eb927 100644
--- a/fs/cifs/Kconfig
+++ b/fs/cifs/Kconfig
@@ -175,6 +175,8 @@ config CIFS_NFSD_EXPORT
 	help
 	  Allows NFS server to export a CIFS mounted share (nfsd over cifs)
 
+if CIFS
+
 config CIFS_SMB_DIRECT
 	bool "SMB Direct support"
 	depends on CIFS=m && INFINIBAND && INFINIBAND_ADDR_TRANS || CIFS=y && INFINIBAND=y && INFINIBAND_ADDR_TRANS=y
@@ -198,3 +200,5 @@ config CIFS_ROOT
 	  Enables root file system support over SMB protocol.
 
 	  Most people say N here.
+
+endif
-- 
2.34.1


                 reply	other threads:[~2023-01-31  1:44 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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='CAH2r5mvrsr8Gh0oA=8C+d8EEue1Hsx2OMqr9hQntejRo=H31dg@mail.gmail.com' \
    --to=smfrench@gmail.com \
    --cc=dhowells@redhat.com \
    --cc=linux-cifs@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.