Messages in this thread | | | Date | Thu, 13 Oct 2022 10:10:16 -0700 (PDT) | Subject | Re: [PATCH] MAINTAINERS: Move from git://github.com to https://github.com | From | Palmer Dabbelt <> |
| |
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.
| |