[PATCH 0/3] filesystems.5: Update & wfix

[PATCH 0/3] filesystems.5: Update & wfix

[Alejandro Colomar]

Hi Michael,

Please merge these patches.

Thanks,

Alex

Ahelenia Ziemiańska (2):
  filesystems.5: fix link to user space tooling for ncpfs
  filesystems.5: note ncpfs removal from kernel

наб (1):
  filesystems.5: wfix: ntfs: remove FAT comparison

 man5/filesystems.5 | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)


base-commit: 300ef55cf2eebc7b123fba48a40bb05a14e261e2
-- 
2.29.2

[PATCH 1/3] filesystems.5: fix link to user space tooling for ncpfs

[Alejandro Colomar]

From: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>

Corresponds to Linux commit 1b83df308f69a5a3cc59be03bd7fb23e4bcebd8e

Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>
---
 man5/filesystems.5 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man5/filesystems.5 b/man5/filesystems.5
index b44dba184..6ec2de9f0 100644
--- a/man5/filesystems.5
+++ b/man5/filesystems.5
@@ -158,7 +158,7 @@ Novell NetWare.
 To use
 .BR ncpfs ,
 you need special programs, which can be found at
-.UR ftp://linux01.gwdg.de\:/pub\:/ncpfs
+.UR ftp://ftp.gwdg.de\:/pub\:/linux\:/misc\:/ncpfs
 .UE .
 .TP
 .B nfs
-- 
2.29.2

Re: [PATCH 1/3] filesystems.5: fix link to user space tooling for ncpfs

[Michael Kerrisk (man-pages)]

Hello Alex and Ahelenia,

On 12/18/20 4:30 PM, Alejandro Colomar wrote:
> From: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
> 
> Corresponds to Linux commit 1b83df308f69a5a3cc59be03bd7fb23e4bcebd8e
> 
> Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
> Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>

Patch applied.

Thanks,

Michael

> ---
>  man5/filesystems.5 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man5/filesystems.5 b/man5/filesystems.5
> index b44dba184..6ec2de9f0 100644
> --- a/man5/filesystems.5
> +++ b/man5/filesystems.5
> @@ -158,7 +158,7 @@ Novell NetWare.
>  To use
>  .BR ncpfs ,
>  you need special programs, which can be found at
> -.UR ftp://linux01.gwdg.de\:/pub\:/ncpfs
> +.UR ftp://ftp.gwdg.de\:/pub\:/linux\:/misc\:/ncpfs
>  .UE .
>  .TP
>  .B nfs
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/

[PATCH 2/3] filesystems.5: note ncpfs removal from kernel

[Alejandro Colomar]

From: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>

Relevant Linux commits:
 * moved to staging in 1bb8155080c652c4853e6228f8f0d262b3049699
   (describe: v4.15-rc1-129-g1bb8155080c6) in Nov 2017,
   described as "broken" and "obsolete"
 * purged in bd32895c750bcd2b511bf93917bf7ae723e3d0b6
   (describe: v4.17-rc3-1010-gbd32895c750b) in Jun 2018,
   "since no one has complained or even noticed it was gone"

Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>
---
 man5/filesystems.5 | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/man5/filesystems.5 b/man5/filesystems.5
index 6ec2de9f0..71be05230 100644
--- a/man5/filesystems.5
+++ b/man5/filesystems.5
@@ -152,8 +152,9 @@ filenames can be no longer than 8 characters, followed by an
 optional period and 3 character extension.
 .TP
 .B ncpfs
-is a network filesystem that supports the NCP protocol, used by
-Novell NetWare.
+is a network filesystem that supports the NCP protocol,
+used by Novell NetWare.
+It was was removed from the kernel in 4.17.
 .IP
 To use
 .BR ncpfs ,
-- 
2.29.2

Re: [PATCH 2/3] filesystems.5: note ncpfs removal from kernel

[Michael Kerrisk (man-pages)]

On 12/18/20 4:30 PM, Alejandro Colomar wrote:
> From: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
> 
> Relevant Linux commits:
>  * moved to staging in 1bb8155080c652c4853e6228f8f0d262b3049699
>    (describe: v4.15-rc1-129-g1bb8155080c6) in Nov 2017,
>    described as "broken" and "obsolete"
>  * purged in bd32895c750bcd2b511bf93917bf7ae723e3d0b6
>    (describe: v4.17-rc3-1010-gbd32895c750b) in Jun 2018,
>    "since no one has complained or even noticed it was gone"
> 
> Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
> Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>

Thanks! Patch applied.

Cheers,

Michael

> ---
>  man5/filesystems.5 | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/man5/filesystems.5 b/man5/filesystems.5
> index 6ec2de9f0..71be05230 100644
> --- a/man5/filesystems.5
> +++ b/man5/filesystems.5
> @@ -152,8 +152,9 @@ filenames can be no longer than 8 characters, followed by an
>  optional period and 3 character extension.
>  .TP
>  .B ncpfs
> -is a network filesystem that supports the NCP protocol, used by
> -Novell NetWare.
> +is a network filesystem that supports the NCP protocol,
> +used by Novell NetWare.
> +It was was removed from the kernel in 4.17.
>  .IP
>  To use
>  .BR ncpfs ,
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/

[PATCH 3/3] filesystems.5: wfix: ntfs: remove FAT comparison

[Alejandro Colomar]

From: наб <nabijaczleweli@nabijaczleweli.xyz>

The main point I was driving at with this patch was to fix
"Microsoft Window's FAT filesystems" (i.e. FAT filesystems which belong
 to Microsoft Window, which is decidedly wrong).

FAT32 first shipped with MS-DOS 7.1, as part of Windows 95 OSR2,
but it's a (relatively) simple logical extension of the previous FATx
filesystems (16 and 12 as we know and love them today,
 I don't think the PC ever saw 8), hence the "VFAT" driver name ‒
calling FAT-anything a Windows filesystem would be a flat-out lie,
calling it a Microsoft filesystem would be, uh, facetious.

NTFS (as part of Windows NT), on the other hand, is wholly different
WRT the scope and feature-set (it does borrow some layouting from FAT,
 but reading NTFS as FAT doesn't get you very far, or much).

The replacing bit is also questionable, especially in a.d. 2020:
while it is true that you cannot install NT on FAT (after a certain
 point? my memory ain't what it used to be), and must therefore
replace your existing FAT partitions with NTFS during upgrades;
Windows NT 4.0, the last product to be NT-branded came out in 1996,
i.e. you could not install Windows on FAT (and, therefore,
 upgrade it to NTFS, replacing it) during my entire lifetime.

Indeed, in $(date +%Y) we live in a post-NTFS world ‒ putting NTFS in
the same class as FAT beyond "is a filesystem" is a joke.

Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>
---
 man5/filesystems.5 | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/man5/filesystems.5 b/man5/filesystems.5
index 71be05230..7f2e02096 100644
--- a/man5/filesystems.5
+++ b/man5/filesystems.5
@@ -166,9 +166,8 @@ you need special programs, which can be found at
 is the network filesystem used to access disks located on remote computers.
 .TP
 .B ntfs
-replaces Microsoft Window's FAT filesystems (VFAT, FAT32).
-It has reliability, performance, and space-utilization enhancements
-plus features like ACLs, journaling, encryption, and so on.
+is the filesystem native to Microsoft Windows NT,
+supporting features like ACLs, journaling, encryption, and so on.
 .TP
 .B proc
 is a pseudo filesystem which is used as an interface to kernel data
-- 
2.29.2

Re: [PATCH 3/3] filesystems.5: wfix: ntfs: remove FAT comparison

[Michael Kerrisk (man-pages)]

Hello Alex and наб

On 12/18/20 4:30 PM, Alejandro Colomar wrote:
> From: наб <nabijaczleweli@nabijaczleweli.xyz>
> 
> The main point I was driving at with this patch was to fix
> "Microsoft Window's FAT filesystems" (i.e. FAT filesystems which belong
>  to Microsoft Window, which is decidedly wrong).
> 
> FAT32 first shipped with MS-DOS 7.1, as part of Windows 95 OSR2,
> but it's a (relatively) simple logical extension of the previous FATx
> filesystems (16 and 12 as we know and love them today,
>  I don't think the PC ever saw 8), hence the "VFAT" driver name ‒
> calling FAT-anything a Windows filesystem would be a flat-out lie,
> calling it a Microsoft filesystem would be, uh, facetious.
> 
> NTFS (as part of Windows NT), on the other hand, is wholly different
> WRT the scope and feature-set (it does borrow some layouting from FAT,
>  but reading NTFS as FAT doesn't get you very far, or much).
> 
> The replacing bit is also questionable, especially in a.d. 2020:
> while it is true that you cannot install NT on FAT (after a certain
>  point? my memory ain't what it used to be), and must therefore
> replace your existing FAT partitions with NTFS during upgrades;
> Windows NT 4.0, the last product to be NT-branded came out in 1996,
> i.e. you could not install Windows on FAT (and, therefore,
>  upgrade it to NTFS, replacing it) during my entire lifetime.
> 
> Indeed, in $(date +%Y) we live in a post-NTFS world ‒ putting NTFS in
> the same class as FAT beyond "is a filesystem" is a joke.
> 
> Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
> Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>

Thanks. Patch applied. I cleaned up the commit message a little
(removed leading white spaces, formatted to 66 columns as per the 
man-pages norm).

Cheers,

Michael

> ---
>  man5/filesystems.5 | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/man5/filesystems.5 b/man5/filesystems.5
> index 71be05230..7f2e02096 100644
> --- a/man5/filesystems.5
> +++ b/man5/filesystems.5
> @@ -166,9 +166,8 @@ you need special programs, which can be found at
>  is the network filesystem used to access disks located on remote computers.
>  .TP
>  .B ntfs
> -replaces Microsoft Window's FAT filesystems (VFAT, FAT32).
> -It has reliability, performance, and space-utilization enhancements
> -plus features like ACLs, journaling, encryption, and so on.
> +is the filesystem native to Microsoft Windows NT,
> +supporting features like ACLs, journaling, encryption, and so on.
>  .TP
>  .B proc
>  is a pseudo filesystem which is used as an interface to kernel data
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH 3/3] filesystems.5: wfix: ntfs: remove FAT comparison

[Alejandro Colomar (man-pages)]

[removed some CCs]

On 12/20/20 3:47 PM, Michael Kerrisk (man-pages) wrote:
> Hello Alex and наб
> 
> On 12/18/20 4:30 PM, Alejandro Colomar wrote:
>> From: наб <nabijaczleweli@nabijaczleweli.xyz>
[...]
>>
>> Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
>> Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>
> 
> Thanks. Patch applied. I cleaned up the commit message a little
> (removed leading white spaces, formatted to 66 columns as per the 
> man-pages norm).

Hi Michael,

Where is that 66 col norm?
I've only found 75 for code in general (in man-pages(7)),
but not 66 for the commit msg.

Thanks,

Alex

> 
> Cheers,
> 
> Michael

-- 
Alejandro Colomar
Linux man-pages comaintainer; https://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/

Re: [PATCH 3/3] filesystems.5: wfix: ntfs: remove FAT comparison

[Michael Kerrisk (man-pages)]

Hi Alejandro,

On Tue, 29 Dec 2020 at 12:11, Alejandro Colomar (man-pages)
<alx.manpages@gmail.com> wrote:
>
> [removed some CCs]
>
> On 12/20/20 3:47 PM, Michael Kerrisk (man-pages) wrote:
> > Hello Alex and наб
> >
> > On 12/18/20 4:30 PM, Alejandro Colomar wrote:
> >> From: наб <nabijaczleweli@nabijaczleweli.xyz>
> [...]
> >>
> >> Signed-off-by: Ahelenia Ziemiańska <nabijaczleweli@nabijaczleweli.xyz>
> >> Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>
> >
> > Thanks. Patch applied. I cleaned up the commit message a little
> > (removed leading white spaces, formatted to 66 columns as per the
> > man-pages norm).
>
> Hi Michael,
>
> Where is that 66 col norm?
> I've only found 75 for code in general (in man-pages(7)),
> but not 66 for the commit msg.

True. It's not documented anywhere. It's a limit that I apply so that
the changelog that I produce fits in 80 columns (66 = 80 - 2 * 8-space
indents). I quite frequently reformat patches for that limit. Don't
feel obliged to follow that, but I'll probably reformat your commit
messages :-).

Cheers,

Michael


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/