Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 13 Oct 2022 10:10:16 -0700 (PDT)
Subject:     Re: [PATCH] MAINTAINERS: Move from git://github.com to https://github.com
In-Reply-To: <Y0hEt51q+Fs0ICbB@spud>
CC:     f.fainelli@gmail.com, sbranden@broadcom.com,
        linux-kernel@vger.kernel.org
From:   Palmer Dabbelt <palmer@rivosinc.com>
To:     Conor Dooley <conor@kernel.org>
Message-ID: <mhng-0881314d-117e-437e-920d-611c7fe0139f@palmer-ri-x1c9a>
Mime-Version: 1.0 (MHng)
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk

On Thu, 13 Oct 2022 10:02:47 PDT (-0700), Conor Dooley wrote:
> On Thu, Oct 13, 2022 at 08:32:58AM -0700, Palmer Dabbelt wrote:
>> Github deprecated the git:// links about a year ago, but it looks like
>> there's still a handful of them in the MAINTAINERS file.  Conor pointed
>> this out about the RISC-V KVM tree, but I figured it'd be better to just
>> fix them all -- I've got a bunch of insteadOf so I didn't even notice
>> the deprecation, but new contributors probably don't and might get a bit
>> confused.
>>
>> Reported-by: Conor Dooley <conor@kernel.org>
>> Link: https://github.blog/2021-09-01-improving-git-protocol-security-github/
>> Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>
>> ---
>> @@ -3990,7 +3990,7 @@ M:	Ray Jui <rjui@broadcom.com>
>>  M:	Scott Branden <sbranden@broadcom.com>
>>  R:	Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com>
>>  S:	Maintained
>> -T:	git git://github.com/broadcom/mach-bcm
>> +T:	git https://github.com/broadcom/mach-bcm
>
> This link is dead, I can't find a repo with that name on their GitHub
> profile.
> @Florian, should this be updated to the stblinux repo instead?
>
>>  F:	arch/arm/mach-bcm/
>>  N:	bcm281*
>>  N:	bcm113*
>> @@ -4055,7 +4055,7 @@ M:	Florian Fainelli <f.fainelli@gmail.com>
>>  R:	Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com>
>>  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>>  S:	Maintained
>> -T:	git git://github.com/broadcom/stblinux.git
>> +T:	git https://github.com/broadcom/stblinux.git
>>  F:	Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml
>>  F:	arch/arm/boot/dts/bcm7*.dts*
>>  F:	arch/arm/include/asm/hardware/cache-b15-rac.h
>
> Other than that, I clicked all the links - some look dead but that's not
> the point of this patch..

Thanks, I just spot checked a few to make sure nothing went way off the 
rails.  I guess we could leave the broken ones as git:// to make it 
clear they need to be updated?  Not sure that helps any, though.

> Tested-by: Conor Dooley <conor.dooley@microchip.com>
>
> Thanks,
> Conor.