Re: [PATCH 3/3] Docs/filesystems/9p: Convert a goo.gl URL to original one

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 26 Aug 2024 10:18:47 +0900 Dominique Martinet <asmadeus@xxxxxxxxxxxxx> wrote:

> SeongJae Park wrote on Sun, Aug 25, 2024 at 06:09:49PM -0700:
> > Google URL shortner will be unavailable soon[1].  Replace a shortened
> > URL on 9p.rst with its original one.
> > 
> > [1] https://developers.googleblog.com/en/google-url-shortener-links-will-no-longer-be-available/
> > 
> > Signed-off-by: SeongJae Park <sj@xxxxxxxxxx>
> 
> Sorry, we (9p maintainers) slacked on this one - there's been a better
> patch for this one:
> https://lkml.kernel.org/r/20240725180041.80862-1-linux@xxxxxxxxxxx
> 
> The sciweavers link doesn't actually work, so it doesn't make sense to
> replace as is.
> (although you could argue that it's probably been broken forever an
> nobody noticed, so it could just as well be removed...)

No problem.  I agree the other patch is better.

> 
> There's no patch queued for 9p, so if you (Jonathan) want to take the
> other patch please feel free to.
> If it goes through the 9p tree it'll get in the next time we send
> something as I don't feel the need to send Linus a PR just for this, so
> that might be a couple of months further down.

Either ways look good to me.

> (I don't like patch series where folks are added in Cc on individual
> patches because it's never clear who is expected to grab the patch in
> the end...)

Sorry for that.  I thoght lore.kernel.org should help getting the context, but
I understand no one can convince all.  I'll try to take a different approach if
I have a chance to send patches to you next time.


Thanks,
SJ

> 
> 
> Thanks,
> -- 
> Dominique




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux