linux-kernel - Re: [PATCH] MAINTAINERS: Move from git://github.com to https://github.com

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <mhng-0881314d-117e-437e-920d-611c7fe0139f@palmer-ri-x1c9a>
Date:   Thu, 13 Oct 2022 10:10:16 -0700 (PDT)
From:   Palmer Dabbelt <palmer@...osinc.com>
To:     Conor Dooley <conor@...nel.org>
CC:     f.fainelli@...il.com, sbranden@...adcom.com,
        linux-kernel@...r.kernel.org
Subject:     Re: [PATCH] MAINTAINERS: Move from git://github.com to https://github.com

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@...nel.org>
>> Link: https://github.blog/2021-09-01-improving-git-protocol-security-github/
>> Signed-off-by: Palmer Dabbelt <palmer@...osinc.com>
>> ---
>> @@ -3990,7 +3990,7 @@ M:	Ray Jui <rjui@...adcom.com>
>>  M:	Scott Branden <sbranden@...adcom.com>
>>  R:	Broadcom internal kernel review list <bcm-kernel-feedback-list@...adcom.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@...il.com>
>>  R:	Broadcom internal kernel review list <bcm-kernel-feedback-list@...adcom.com>
>>  L:	linux-arm-kernel@...ts.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@...rochip.com>
>
> Thanks,
> Conor.