diff mbox series

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

Message ID 20240826010949.78305-3-sj@kernel.org (mailing list archive)
State New
Headers show
Series None | expand

Commit Message

SeongJae Park Aug. 26, 2024, 1:09 a.m. UTC
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@kernel.org>
---
 Documentation/filesystems/9p.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Dominique Martinet Aug. 26, 2024, 1:18 a.m. UTC | #1
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@kernel.org>

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@treblig.org

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...)

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.
(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...)


Thanks,
SeongJae Park Aug. 26, 2024, 2:21 a.m. UTC | #2
On Mon, 26 Aug 2024 10:18:47 +0900 Dominique Martinet <asmadeus@codewreck.org> 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@kernel.org>
> 
> 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@treblig.org
> 
> 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
Jonathan Corbet Aug. 26, 2024, 10:41 p.m. UTC | #3
Dominique Martinet <asmadeus@codewreck.org> writes:

> 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@kernel.org>
>
> 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@treblig.org
>
> 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...)
>
> There's no patch queued for 9p, so if you (Jonathan) want to take the
> other patch please feel free to.

It wasn't sent to me ... how did we ever manage without b4?  I've
applied, it, thanks.

jon
diff mbox series

Patch

diff --git a/Documentation/filesystems/9p.rst b/Documentation/filesystems/9p.rst
index 1e0e0bb6fdf9..50dc82f33726 100644
--- a/Documentation/filesystems/9p.rst
+++ b/Documentation/filesystems/9p.rst
@@ -31,7 +31,7 @@  Other applications are described in the following papers:
 	* PROSE I/O: Using 9p to enable Application Partitions
 	  http://plan9.escet.urjc.es/iwp9/cready/PROSE_iwp9_2006.pdf
 	* VirtFS: A Virtualization Aware File System pass-through
-	  http://goo.gl/3WPDg
+          https://www.sciweavers.org/publications/virtfs-virtualization-aware-file-system-pass-through
 
 Usage
 =====