diff mbox series

[1/6] MAINTAINERS: fix kbuild repo branch

Message ID 20230117095946.2042832-2-jani.nikula@intel.com (mailing list archive)
State New, archived
Headers show
Series docs: kbuild makefile documentation cleanups | expand

Commit Message

Jani Nikula Jan. 17, 2023, 9:59 a.m. UTC
The MAINTAINERS repository entry specifies "kconfig" as the branch, but
the repository itself has "kbuild".

Cc: Masahiro Yamada <masahiroy@kernel.org>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Sedat Dilek Jan. 17, 2023, 1:12 p.m. UTC | #1
On Tue, Jan 17, 2023 at 11:15 AM Jani Nikula <jani.nikula@intel.com> wrote:
>
> The MAINTAINERS repository entry specifies "kconfig" as the branch, but
> the repository itself has "kbuild".
>
> Cc: Masahiro Yamada <masahiroy@kernel.org>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---
>  MAINTAINERS | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a36df9ed283d..98da2af0a653 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -11171,7 +11171,7 @@ M:      Masahiro Yamada <masahiroy@kernel.org>
>  L:     linux-kbuild@vger.kernel.org
>  S:     Maintained
>  Q:     https://patchwork.kernel.org/project/linux-kbuild/list/
> -T:     git git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git kconfig
> +T:     git git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git kbuild

We have two entries - one for "KCONFIG" and the other "KERNEL BUILD +
files below scripts/ (unless maintained elsewhere)".

Cannot say if kbuild is the place where KCONFIG changes are done.

But you are right there is no kconfig Git branch, anymore.

Some maintainers switch over to use https over git URL these days.

What do you prefer?

$ grep 'git git' MAINTAINERS | grep 'T:' | wc -l
668

$ grep 'git https' MAINTAINERS | grep 'T:' | wc -l
49

In my ~/.gitconfig I have:
...
[url "https://git.kernel.org/"]
       insteadOf = git://git.kernel.org/
[url "https://github.com/"]
       insteadOf = git://github.com/
...

BR,
-Sedat-


>  F:     Documentation/kbuild/kconfig*
>  F:     scripts/Kconfig.include
>  F:     scripts/kconfig/
> --
> 2.34.1
>
Masahiro Yamada Jan. 17, 2023, 2:47 p.m. UTC | #2
On Tue, Jan 17, 2023 at 7:00 PM Jani Nikula <jani.nikula@intel.com> wrote:
>
> The MAINTAINERS repository entry specifies "kconfig" as the branch, but
> the repository itself has "kbuild".
>
> Cc: Masahiro Yamada <masahiroy@kernel.org>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---
>  MAINTAINERS | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a36df9ed283d..98da2af0a653 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -11171,7 +11171,7 @@ M:      Masahiro Yamada <masahiroy@kernel.org>
>  L:     linux-kbuild@vger.kernel.org
>  S:     Maintained
>  Q:     https://patchwork.kernel.org/project/linux-kbuild/list/
> -T:     git git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git kconfig
> +T:     git git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git kbuild
>  F:     Documentation/kbuild/kconfig*
>  F:     scripts/Kconfig.include
>  F:     scripts/kconfig/
> --
> 2.34.1
>


I sometimes created the 'kconfig' branch when I had
a large batch of kconfig changes, but
8ca5297e7e38f2dc8c753d33a5092e7be181fff0 was
the last time I did that.

Indeed 'kconfig' does not always exist, so I am fine
with this (and rest of the patch set). Thanks.
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index a36df9ed283d..98da2af0a653 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11171,7 +11171,7 @@  M:	Masahiro Yamada <masahiroy@kernel.org>
 L:	linux-kbuild@vger.kernel.org
 S:	Maintained
 Q:	https://patchwork.kernel.org/project/linux-kbuild/list/
-T:	git git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git kconfig
+T:	git git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git kbuild
 F:	Documentation/kbuild/kconfig*
 F:	scripts/Kconfig.include
 F:	scripts/kconfig/