Re: [PATCH] MAINTAINERS: standardize git.kernel.org URLs

[Xose Vazquez Perez <xose.vazquez@gmail.com>]

On 6/25/25 7:27 PM, Jonathan Corbet wrote:

> Xose Vazquez Perez <xose.vazquez@gmail.com> writes:
> 
>> replace https: with git:, delete trailing /, and identify repos as "git"
>>
>> Cc: Jonathan Corbet <corbet@lwn.net> (maintainer:DOCUMENTATION)
>> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>> Cc: DOC ML <linux-doc@vger.kernel.org> (open list:DOCUMENTATION)
>> Cc: KERNEL ML <linux-kernel@vger.kernel.org> (open list)
>> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
>> ---
>>   MAINTAINERS | 50 +++++++++++++++++++++++++-------------------------
>>   1 file changed, 25 insertions(+), 25 deletions(-)

> So ... we're changing GitHub URLs from git: to https:, and Gitlab URLs
> from https: to git:?
> 
> Certainly we want to fix URLs that are broken, but is there any real
> reason to churn up the MAINTAINERS file to "fix" URLs that work?

The reason was to be *consistent* with the rest of the entries.
Because most of them(380) are using git:, vs 22 for https:

$ grep git.kernel.org MAINTAINERS | sed 's#://#:// #' | awk '{print $3}' | sort | uniq -c | sort -n
       1 git.kernel.org/pub/scm/linux/kernel/git/tj/sched_ext.git
      22 https://
     380 git://

But Greg has some concerns about this protocol:
https://lore.kernel.org/lkml/2025062654-lubricant-lettuce-3405@gregkh/
"This bypasses the mirror systems in place with the https: protocol,
please do not do this without a lot of justification and agreement of
the hosting providers involved as they will have a large increase in
resources if this were to change."

So, the right patch should replace git://git.kernel.org with https://git.kernel.org in all the tree.
$ git grep -l "git://git.kernel.org"
Documentation/admin-guide/LSM/apparmor.rst
Documentation/admin-guide/kdump/kdump.rst
Documentation/admin-guide/reporting-issues.rst
Documentation/admin-guide/workload-tracing.rst
Documentation/arch/arm/keystone/knav-qmss.rst
Documentation/dev-tools/sparse.rst
Documentation/filesystems/bcachefs/SubmittingPatches.rst
Documentation/filesystems/erofs.rst
Documentation/filesystems/f2fs.rst
Documentation/maintainer/pull-requests.rst
Documentation/mm/hwpoison.rst
Documentation/process/adding-syscalls.rst
Documentation/process/maintainer-kvm-x86.rst
Documentation/process/submitting-patches.rst
Documentation/sound/hd-audio/notes.rst
Documentation/timers/no_hz.rst
Documentation/translations/it_IT/process/adding-syscalls.rst
Documentation/translations/it_IT/process/submitting-patches.rst
Documentation/translations/sp_SP/process/adding-syscalls.rst
Documentation/translations/sp_SP/process/maintainer-kvm-x86.rst
Documentation/translations/sp_SP/process/submitting-patches.rst
Documentation/translations/zh_CN/admin-guide/reporting-issues.rst
Documentation/translations/zh_CN/dev-tools/sparse.rst
Documentation/translations/zh_CN/how-to.rst
Documentation/translations/zh_CN/maintainer/pull-requests.rst
Documentation/translations/zh_CN/mm/hwpoison.rst
Documentation/translations/zh_CN/process/submitting-patches.rst
Documentation/translations/zh_TW/admin-guide/reporting-issues.rst
Documentation/translations/zh_TW/dev-tools/sparse.rst
Documentation/translations/zh_TW/process/submitting-patches.rst
MAINTAINERS
drivers/net/wireless/intel/iwlwifi/iwl-drv.c
scripts/get_maintainer.pl
scripts/package/debian/copyright
tools/tracing/rtla/README.txt
tools/usb/usbip/src/usbip.c
tools/verification/rv/README.txt