2022-10-13 22:15:45

by Palmer Dabbelt

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: git://github -> https://github.com for konis

Github deprecated the git:// links about a year ago, so let's move to
the https:// URLs instead.

Reported-by: Conor Dooley <[email protected]>
Link: https://github.blog/2021-09-01-improving-git-protocol-security-github/
Signed-off-by: Palmer Dabbelt <[email protected]>
---
I've split these up by github username so folks can take them
independently, as some of these repos have been renamed at github and
thus need more than just a sed to fix them.
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index d118acdce8b8..c9c396e2d8a0 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14363,7 +14363,7 @@ L: [email protected]
S: Supported
W: https://nilfs.sourceforge.io/
W: https://nilfs.osdn.jp/
-T: git git://github.com/konis/nilfs2.git
+T: git https://github.com/konis/nilfs2.git
F: Documentation/filesystems/nilfs2.rst
F: fs/nilfs2/
F: include/trace/events/nilfs2.h
--
2.38.0


2022-10-13 23:34:53

by Ryusuke Konishi

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: git://github -> https://github.com for konis

On Fri, Oct 14, 2022 at 6:49 AM Palmer Dabbelt wrote:
>
> Github deprecated the git:// links about a year ago, so let's move to
> the https:// URLs instead.
>
> Reported-by: Conor Dooley <[email protected]>
> Link: https://github.blog/2021-09-01-improving-git-protocol-security-github/
> Signed-off-by: Palmer Dabbelt <[email protected]>
> ---
> I've split these up by github username so folks can take them
> independently, as some of these repos have been renamed at github and
> thus need more than just a sed to fix them.
> ---
> MAINTAINERS | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index d118acdce8b8..c9c396e2d8a0 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -14363,7 +14363,7 @@ L: [email protected]
> S: Supported
> W: https://nilfs.sourceforge.io/
> W: https://nilfs.osdn.jp/
> -T: git git://github.com/konis/nilfs2.git
> +T: git https://github.com/konis/nilfs2.git
> F: Documentation/filesystems/nilfs2.rst
> F: fs/nilfs2/
> F: include/trace/events/nilfs2.h
> --
> 2.38.0
>

Ah, that's right, it needs to be fixed.

I'll pick this up unless it's decided to apply similar fixes all at once.
Thank you.

Ryusuke Konishi

2022-10-14 04:26:29

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: git://github -> https://github.com for konis

On Thu, 13 Oct 2022 16:06:43 PDT (-0700), [email protected] wrote:
> On Fri, Oct 14, 2022 at 6:49 AM Palmer Dabbelt wrote:
>>
>> Github deprecated the git:// links about a year ago, so let's move to
>> the https:// URLs instead.
>>
>> Reported-by: Conor Dooley <[email protected]>
>> Link: https://github.blog/2021-09-01-improving-git-protocol-security-github/
>> Signed-off-by: Palmer Dabbelt <[email protected]>
>> ---
>> I've split these up by github username so folks can take them
>> independently, as some of these repos have been renamed at github and
>> thus need more than just a sed to fix them.
>> ---
>> MAINTAINERS | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index d118acdce8b8..c9c396e2d8a0 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -14363,7 +14363,7 @@ L: [email protected]
>> S: Supported
>> W: https://nilfs.sourceforge.io/
>> W: https://nilfs.osdn.jp/
>> -T: git git://github.com/konis/nilfs2.git
>> +T: git https://github.com/konis/nilfs2.git
>> F: Documentation/filesystems/nilfs2.rst
>> F: fs/nilfs2/
>> F: include/trace/events/nilfs2.h
>> --
>> 2.38.0
>>
>
> Ah, that's right, it needs to be fixed.
>
> I'll pick this up unless it's decided to apply similar fixes all at once.
> Thank you.

It's easier on my end if you just pick it up. Just note the
"git://github -> https://github.com" typo. I can send a v2 if you want,
but IMO it's trival enough to just fix up.

Thanks!

>
> Ryusuke Konishi

2022-10-14 04:43:37

by Ryusuke Konishi

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: git://github -> https://github.com for konis

On Fri, Oct 14, 2022 at 12:51 PM Palmer Dabbelt wrote:
>
> On Thu, 13 Oct 2022 16:06:43 PDT (-0700), [email protected] wrote:
> > On Fri, Oct 14, 2022 at 6:49 AM Palmer Dabbelt wrote:
> >>
> >> Github deprecated the git:// links about a year ago, so let's move to
> >> the https:// URLs instead.
> >>
> >> Reported-by: Conor Dooley <[email protected]>
> >> Link: https://github.blog/2021-09-01-improving-git-protocol-security-github/
> >> Signed-off-by: Palmer Dabbelt <[email protected]>
> >> ---
> >> I've split these up by github username so folks can take them
> >> independently, as some of these repos have been renamed at github and
> >> thus need more than just a sed to fix them.
> >> ---
> >> MAINTAINERS | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index d118acdce8b8..c9c396e2d8a0 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -14363,7 +14363,7 @@ L: [email protected]
> >> S: Supported
> >> W: https://nilfs.sourceforge.io/
> >> W: https://nilfs.osdn.jp/
> >> -T: git git://github.com/konis/nilfs2.git
> >> +T: git https://github.com/konis/nilfs2.git
> >> F: Documentation/filesystems/nilfs2.rst
> >> F: fs/nilfs2/
> >> F: include/trace/events/nilfs2.h
> >> --
> >> 2.38.0
> >>
> >
> > Ah, that's right, it needs to be fixed.
> >
> > I'll pick this up unless it's decided to apply similar fixes all at once.
> > Thank you.
>
> It's easier on my end if you just pick it up. Just note the
> "git://github -> https://github.com" typo. I can send a v2 if you want,
> but IMO it's trival enough to just fix up.
>
> Thanks!
>
> >
> > Ryusuke Konishi

Ah, thanks. I will fix that typo (the omission of ''.com") myself.

Ryusuke Konishi