* [PATCH] nfs-utils: nfsidmap.man: Fix section number @ 2022-04-12 7:00 Salvatore Bonaccorso 2022-04-12 14:28 ` Steve Dickson 0 siblings, 1 reply; 7+ messages in thread From: Salvatore Bonaccorso @ 2022-04-12 7:00 UTC (permalink / raw) To: linux-nfs Cc: Steve Dickson, Chuck Lever, Bryan Schumaker, Ben Hutchings, Salvatore Bonaccorso From: Ben Hutchings <benh@debian.org> The nfsidmap manual page is supposed to be in section 8, but calls the .TH macro with a section argument of 5. This results in an incorrect header and causes debhelper (in Debian) to install it in the section 5 directory. Fix that. Signed-off-by: Ben Hutchings <benh@debian.org> [Salvatore Bonaccorso: Slightly modify commit message to mention that the Problem is found in Debian through installing the manpage via debhelper.] Signed-off-by: Salvatore Bonaccorso <carnil@debian.org> --- utils/nfsidmap/nfsidmap.man | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utils/nfsidmap/nfsidmap.man b/utils/nfsidmap/nfsidmap.man index 2af16f3157ff..1911c41be6f9 100644 --- a/utils/nfsidmap/nfsidmap.man +++ b/utils/nfsidmap/nfsidmap.man @@ -2,7 +2,7 @@ .\"@(#)nfsidmap(8) - The NFS idmapper upcall program .\" .\" Copyright (C) 2010 Bryan Schumaker <bjschuma@netapp.com> -.TH nfsidmap 5 "1 October 2010" +.TH nfsidmap 8 "1 October 2010" .SH NAME nfsidmap \- The NFS idmapper upcall program .SH SYNOPSIS -- 2.35.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] nfs-utils: nfsidmap.man: Fix section number 2022-04-12 7:00 [PATCH] nfs-utils: nfsidmap.man: Fix section number Salvatore Bonaccorso @ 2022-04-12 14:28 ` Steve Dickson 2022-04-12 19:25 ` Salvatore Bonaccorso 0 siblings, 1 reply; 7+ messages in thread From: Steve Dickson @ 2022-04-12 14:28 UTC (permalink / raw) To: Salvatore Bonaccorso, linux-nfs Cc: Chuck Lever, Bryan Schumaker, Ben Hutchings Hello, On 4/12/22 3:00 AM, Salvatore Bonaccorso wrote: My mailer was unable to process the attachment Please in-line the patch and resend it. steved. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] nfs-utils: nfsidmap.man: Fix section number 2022-04-12 14:28 ` Steve Dickson @ 2022-04-12 19:25 ` Salvatore Bonaccorso 2022-04-17 8:21 ` Salvatore Bonaccorso 0 siblings, 1 reply; 7+ messages in thread From: Salvatore Bonaccorso @ 2022-04-12 19:25 UTC (permalink / raw) To: Steve Dickson; +Cc: linux-nfs, Chuck Lever, Bryan Schumaker, Ben Hutchings Hi Steve, On Tue, Apr 12, 2022 at 10:28:50AM -0400, Steve Dickson wrote: > Hello, > > On 4/12/22 3:00 AM, Salvatore Bonaccorso wrote: > > My mailer was unable to process the attachment > Please in-line the patch and resend it. That is very strange, I used git send-email to submit it, and I do not see where it got mangled, as it is present as well in https://lore.kernel.org/linux-nfs/20220412070016.720489-1-carnil@debian.org/ Any idea what happened? Here it is again, inlined in this message. Regards, Salvatore From da390ced58885b0ed80be3722d4d913909e7c543 Mon Sep 17 00:00:00 2001 From: Ben Hutchings <benh@debian.org> Date: Mon, 14 Mar 2022 00:19:33 +0100 Subject: [PATCH] nfsidmap.man: Fix section number The nfsidmap manual page is supposed to be in section 8, but calls the .TH macro with a section argument of 5. This results in an incorrect header and causes debhelper (in Debian) to install it in the section 5 directory. Fix that. Signed-off-by: Ben Hutchings <benh@debian.org> [Salvatore Bonaccorso: Slightly modify commit message to mention that the Problem is found in Debian through installing the manpage via debhelper] Signed-off-by: Salvatore Bonaccorso <carnil@debian.org> --- utils/nfsidmap/nfsidmap.man | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utils/nfsidmap/nfsidmap.man b/utils/nfsidmap/nfsidmap.man index 2af16f3157ff..1911c41be6f9 100644 --- a/utils/nfsidmap/nfsidmap.man +++ b/utils/nfsidmap/nfsidmap.man @@ -2,7 +2,7 @@ .\"@(#)nfsidmap(8) - The NFS idmapper upcall program .\" .\" Copyright (C) 2010 Bryan Schumaker <bjschuma@netapp.com> -.TH nfsidmap 5 "1 October 2010" +.TH nfsidmap 8 "1 October 2010" .SH NAME nfsidmap \- The NFS idmapper upcall program .SH SYNOPSIS -- 2.35.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] nfs-utils: nfsidmap.man: Fix section number 2022-04-12 19:25 ` Salvatore Bonaccorso @ 2022-04-17 8:21 ` Salvatore Bonaccorso 2022-04-19 19:17 ` Steve Dickson 0 siblings, 1 reply; 7+ messages in thread From: Salvatore Bonaccorso @ 2022-04-17 8:21 UTC (permalink / raw) To: Steve Dickson; +Cc: linux-nfs, Chuck Lever, Bryan Schumaker, Ben Hutchings Hi Steve, On Tue, Apr 12, 2022 at 09:25:28PM +0200, Salvatore Bonaccorso wrote: > Hi Steve, > > On Tue, Apr 12, 2022 at 10:28:50AM -0400, Steve Dickson wrote: > > Hello, > > > > On 4/12/22 3:00 AM, Salvatore Bonaccorso wrote: > > > > My mailer was unable to process the attachment > > Please in-line the patch and resend it. > > That is very strange, I used git send-email to submit it, and I do not > see where it got mangled, as it is present as well in > > https://lore.kernel.org/linux-nfs/20220412070016.720489-1-carnil@debian.org/ > > Any idea what happened? > > Here it is again, inlined in this message. > > Regards, > Salvatore > > From da390ced58885b0ed80be3722d4d913909e7c543 Mon Sep 17 00:00:00 2001 > From: Ben Hutchings <benh@debian.org> > Date: Mon, 14 Mar 2022 00:19:33 +0100 > Subject: [PATCH] nfsidmap.man: Fix section number > > The nfsidmap manual page is supposed to be in section 8, but calls the > .TH macro with a section argument of 5. This results in an incorrect > header and causes debhelper (in Debian) to install it in the section 5 > directory. Fix that. > > Signed-off-by: Ben Hutchings <benh@debian.org> > [Salvatore Bonaccorso: Slightly modify commit message to mention that > the Problem is found in Debian through installing the manpage via > debhelper] > Signed-off-by: Salvatore Bonaccorso <carnil@debian.org> > --- > utils/nfsidmap/nfsidmap.man | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/utils/nfsidmap/nfsidmap.man b/utils/nfsidmap/nfsidmap.man > index 2af16f3157ff..1911c41be6f9 100644 > --- a/utils/nfsidmap/nfsidmap.man > +++ b/utils/nfsidmap/nfsidmap.man > @@ -2,7 +2,7 @@ > .\"@(#)nfsidmap(8) - The NFS idmapper upcall program > .\" > .\" Copyright (C) 2010 Bryan Schumaker <bjschuma@netapp.com> > -.TH nfsidmap 5 "1 October 2010" > +.TH nfsidmap 8 "1 October 2010" > .SH NAME > nfsidmap \- The NFS idmapper upcall program > .SH SYNOPSIS > -- > 2.35.1 Was this version now correctly processed by your mailer? Regards, Salvatore ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] nfs-utils: nfsidmap.man: Fix section number 2022-04-17 8:21 ` Salvatore Bonaccorso @ 2022-04-19 19:17 ` Steve Dickson 2022-04-19 19:56 ` Salvatore Bonaccorso 0 siblings, 1 reply; 7+ messages in thread From: Steve Dickson @ 2022-04-19 19:17 UTC (permalink / raw) To: Salvatore Bonaccorso Cc: linux-nfs, Chuck Lever, Bryan Schumaker, Ben Hutchings Hey! Sorry for the delay... On 4/17/22 4:21 AM, Salvatore Bonaccorso wrote: > Hi Steve, > > On Tue, Apr 12, 2022 at 09:25:28PM +0200, Salvatore Bonaccorso wrote: >> Hi Steve, >> >> On Tue, Apr 12, 2022 at 10:28:50AM -0400, Steve Dickson wrote: >>> Hello, >>> >>> On 4/12/22 3:00 AM, Salvatore Bonaccorso wrote: >>> >>> My mailer was unable to process the attachment >>> Please in-line the patch and resend it. >> >> That is very strange, I used git send-email to submit it, and I do not >> see where it got mangled, as it is present as well in >> >> https://lore.kernel.org/linux-nfs/20220412070016.720489-1-carnil@debian.org/ >> >> Any idea what happened? >> >> Here it is again, inlined in this message. >> >> Regards, >> Salvatore >> >> From da390ced58885b0ed80be3722d4d913909e7c543 Mon Sep 17 00:00:00 2001 >> From: Ben Hutchings <benh@debian.org> >> Date: Mon, 14 Mar 2022 00:19:33 +0100 >> Subject: [PATCH] nfsidmap.man: Fix section number >> >> The nfsidmap manual page is supposed to be in section 8, but calls the >> .TH macro with a section argument of 5. This results in an incorrect >> header and causes debhelper (in Debian) to install it in the section 5 >> directory. Fix that. >> >> Signed-off-by: Ben Hutchings <benh@debian.org> >> [Salvatore Bonaccorso: Slightly modify commit message to mention that >> the Problem is found in Debian through installing the manpage via >> debhelper] >> Signed-off-by: Salvatore Bonaccorso <carnil@debian.org> >> --- >> utils/nfsidmap/nfsidmap.man | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/utils/nfsidmap/nfsidmap.man b/utils/nfsidmap/nfsidmap.man >> index 2af16f3157ff..1911c41be6f9 100644 >> --- a/utils/nfsidmap/nfsidmap.man >> +++ b/utils/nfsidmap/nfsidmap.man >> @@ -2,7 +2,7 @@ >> .\"@(#)nfsidmap(8) - The NFS idmapper upcall program >> .\" >> .\" Copyright (C) 2010 Bryan Schumaker <bjschuma@netapp.com> >> -.TH nfsidmap 5 "1 October 2010" >> +.TH nfsidmap 8 "1 October 2010" >> .SH NAME >> nfsidmap \- The NFS idmapper upcall program >> .SH SYNOPSIS >> -- >> 2.35.1 > > Was this version now correctly processed by your mailer? Yeah I got it... thanks... but... Are you saying that this man page should be in chapter 5 or it is fact Debian installs into chapter 5? In RHEL and Fedora we install it in chapter 8... If it belongs in a different chapter I have no problem changing it... I just need to know why. steved. steved. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] nfs-utils: nfsidmap.man: Fix section number 2022-04-19 19:17 ` Steve Dickson @ 2022-04-19 19:56 ` Salvatore Bonaccorso 2022-04-19 20:39 ` Steve Dickson 0 siblings, 1 reply; 7+ messages in thread From: Salvatore Bonaccorso @ 2022-04-19 19:56 UTC (permalink / raw) To: Steve Dickson; +Cc: linux-nfs, Chuck Lever, Bryan Schumaker, Ben Hutchings Hi Steve, On Tue, Apr 19, 2022 at 03:17:45PM -0400, Steve Dickson wrote: > Hey! > > Sorry for the delay... > > On 4/17/22 4:21 AM, Salvatore Bonaccorso wrote: > > Hi Steve, > > > > On Tue, Apr 12, 2022 at 09:25:28PM +0200, Salvatore Bonaccorso wrote: > > > Hi Steve, > > > > > > On Tue, Apr 12, 2022 at 10:28:50AM -0400, Steve Dickson wrote: > > > > Hello, > > > > > > > > On 4/12/22 3:00 AM, Salvatore Bonaccorso wrote: > > > > > > > > My mailer was unable to process the attachment > > > > Please in-line the patch and resend it. > > > > > > That is very strange, I used git send-email to submit it, and I do not > > > see where it got mangled, as it is present as well in > > > > > > https://lore.kernel.org/linux-nfs/20220412070016.720489-1-carnil@debian.org/ > > > > > > Any idea what happened? > > > > > > Here it is again, inlined in this message. > > > > > > Regards, > > > Salvatore > > > > > > From da390ced58885b0ed80be3722d4d913909e7c543 Mon Sep 17 00:00:00 2001 > > > From: Ben Hutchings <benh@debian.org> > > > Date: Mon, 14 Mar 2022 00:19:33 +0100 > > > Subject: [PATCH] nfsidmap.man: Fix section number > > > > > > The nfsidmap manual page is supposed to be in section 8, but calls the > > > .TH macro with a section argument of 5. This results in an incorrect > > > header and causes debhelper (in Debian) to install it in the section 5 > > > directory. Fix that. > > > > > > Signed-off-by: Ben Hutchings <benh@debian.org> > > > [Salvatore Bonaccorso: Slightly modify commit message to mention that > > > the Problem is found in Debian through installing the manpage via > > > debhelper] > > > Signed-off-by: Salvatore Bonaccorso <carnil@debian.org> > > > --- > > > utils/nfsidmap/nfsidmap.man | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/utils/nfsidmap/nfsidmap.man b/utils/nfsidmap/nfsidmap.man > > > index 2af16f3157ff..1911c41be6f9 100644 > > > --- a/utils/nfsidmap/nfsidmap.man > > > +++ b/utils/nfsidmap/nfsidmap.man > > > @@ -2,7 +2,7 @@ > > > .\"@(#)nfsidmap(8) - The NFS idmapper upcall program > > > .\" > > > .\" Copyright (C) 2010 Bryan Schumaker <bjschuma@netapp.com> > > > -.TH nfsidmap 5 "1 October 2010" > > > +.TH nfsidmap 8 "1 October 2010" > > > .SH NAME > > > nfsidmap \- The NFS idmapper upcall program > > > .SH SYNOPSIS > > > -- > > > 2.35.1 > > > > Was this version now correctly processed by your mailer? > Yeah I got it... thanks... but... > Are you saying that this man page should be in chapter 5 > or it is fact Debian installs into chapter 5? > > In RHEL and Fedora we install it in chapter 8... If it > belongs in a different chapter I have no problem changing > it... I just need to know why. Apologies, so this was apparently not very clear. That it belongs into section 8 is right. Debian installed it into the wrong section because the section in the manpage itself refers to 5. This is what the patch tries to inline, and fix the wrong argument to the .TH macro. Because the manpage incorrectly calls the .TH macro with a section argument of 5 the debhelper program in Debian installs the manpage into section 5. Does this clarifies? Regards, Salvatore ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] nfs-utils: nfsidmap.man: Fix section number 2022-04-19 19:56 ` Salvatore Bonaccorso @ 2022-04-19 20:39 ` Steve Dickson 0 siblings, 0 replies; 7+ messages in thread From: Steve Dickson @ 2022-04-19 20:39 UTC (permalink / raw) To: Salvatore Bonaccorso Cc: linux-nfs, Chuck Lever, Bryan Schumaker, Ben Hutchings On 4/19/22 3:56 PM, Salvatore Bonaccorso wrote: > Hi Steve, > > On Tue, Apr 19, 2022 at 03:17:45PM -0400, Steve Dickson wrote: >> Hey! >> >> Sorry for the delay... >> >> On 4/17/22 4:21 AM, Salvatore Bonaccorso wrote: >>> Hi Steve, >>> >>> On Tue, Apr 12, 2022 at 09:25:28PM +0200, Salvatore Bonaccorso wrote: >>>> Hi Steve, >>>> >>>> On Tue, Apr 12, 2022 at 10:28:50AM -0400, Steve Dickson wrote: >>>>> Hello, >>>>> >>>>> On 4/12/22 3:00 AM, Salvatore Bonaccorso wrote: >>>>> >>>>> My mailer was unable to process the attachment >>>>> Please in-line the patch and resend it. >>>> >>>> That is very strange, I used git send-email to submit it, and I do not >>>> see where it got mangled, as it is present as well in >>>> >>>> https://lore.kernel.org/linux-nfs/20220412070016.720489-1-carnil@debian.org/ >>>> >>>> Any idea what happened? >>>> >>>> Here it is again, inlined in this message. >>>> >>>> Regards, >>>> Salvatore >>>> >>>> From da390ced58885b0ed80be3722d4d913909e7c543 Mon Sep 17 00:00:00 2001 >>>> From: Ben Hutchings <benh@debian.org> >>>> Date: Mon, 14 Mar 2022 00:19:33 +0100 >>>> Subject: [PATCH] nfsidmap.man: Fix section number >>>> >>>> The nfsidmap manual page is supposed to be in section 8, but calls the >>>> .TH macro with a section argument of 5. This results in an incorrect >>>> header and causes debhelper (in Debian) to install it in the section 5 >>>> directory. Fix that. >>>> >>>> Signed-off-by: Ben Hutchings <benh@debian.org> >>>> [Salvatore Bonaccorso: Slightly modify commit message to mention that >>>> the Problem is found in Debian through installing the manpage via >>>> debhelper] >>>> Signed-off-by: Salvatore Bonaccorso <carnil@debian.org> >>>> --- >>>> utils/nfsidmap/nfsidmap.man | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/utils/nfsidmap/nfsidmap.man b/utils/nfsidmap/nfsidmap.man >>>> index 2af16f3157ff..1911c41be6f9 100644 >>>> --- a/utils/nfsidmap/nfsidmap.man >>>> +++ b/utils/nfsidmap/nfsidmap.man >>>> @@ -2,7 +2,7 @@ >>>> .\"@(#)nfsidmap(8) - The NFS idmapper upcall program >>>> .\" >>>> .\" Copyright (C) 2010 Bryan Schumaker <bjschuma@netapp.com> >>>> -.TH nfsidmap 5 "1 October 2010" >>>> +.TH nfsidmap 8 "1 October 2010" >>>> .SH NAME >>>> nfsidmap \- The NFS idmapper upcall program >>>> .SH SYNOPSIS >>>> -- >>>> 2.35.1 >>> >>> Was this version now correctly processed by your mailer? >> Yeah I got it... thanks... but... >> Are you saying that this man page should be in chapter 5 >> or it is fact Debian installs into chapter 5? >> >> In RHEL and Fedora we install it in chapter 8... If it >> belongs in a different chapter I have no problem changing >> it... I just need to know why. > > Apologies, so this was apparently not very clear. That it belongs into > section 8 is right. Debian installed it into the wrong section because > the section in the manpage itself refers to 5. This is what the patch > tries to inline, and fix the wrong argument to the .TH macro. > > Because the manpage incorrectly calls the .TH macro with a section > argument of 5 the debhelper program in Debian installs the manpage > into section 5. > > Does this clarifies?Yes it does... Thank you! Committed... (re-tagged: nfs-utils-2-6-2-rc4) steved. ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-04-19 20:40 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-04-12 7:00 [PATCH] nfs-utils: nfsidmap.man: Fix section number Salvatore Bonaccorso 2022-04-12 14:28 ` Steve Dickson 2022-04-12 19:25 ` Salvatore Bonaccorso 2022-04-17 8:21 ` Salvatore Bonaccorso 2022-04-19 19:17 ` Steve Dickson 2022-04-19 19:56 ` Salvatore Bonaccorso 2022-04-19 20:39 ` Steve Dickson
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.