public inbox for linux-man@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] nsswitch.conf.5: Current glibc reloads this configuration file
@ 2022-11-04 16:47 Florian Weimer
  2022-11-04 16:58 ` Alejandro Colomar
  0 siblings, 1 reply; 3+ messages in thread
From: Florian Weimer @ 2022-11-04 16:47 UTC (permalink / raw)
  To: linux-man

The implementation happened across multiple glibc commits, but the work
was completed for glibc 2.33.

Signed-off-by: Florian Weimer <fweimer@redhat.com>

---
 man5/nsswitch.conf.5 | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/man5/nsswitch.conf.5 b/man5/nsswitch.conf.5
index 098585cf1..7049455ad 100644
--- a/man5/nsswitch.conf.5
+++ b/man5/nsswitch.conf.5
@@ -402,11 +402,13 @@ for respective databases:
 .PD
 .RE
 .SH NOTES
-Within each process that uses
-.BR nsswitch.conf ,
-the entire file is read only once.
-If the file is later changed, the
-process will continue using the old configuration.
+Starting with glibc 2.33,
+.\" https://sourceware.org/bugzilla/show_bug.cgi?id=12459
+.B nsswitch.conf
+is automatically reloaded if the file is changed.  In earlier
+versions, the entire file was read only once within each process, and
+if the file was later changed, the process would continue using the
+old configuration.
 .PP
 Traditionally, there was only a single source for service information,
 often in the form of a single configuration

base-commit: b106cd5bf43fe806100bb5266b78f92bac90b6e0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] nsswitch.conf.5: Current glibc reloads this configuration file
  2022-11-04 16:47 [PATCH] nsswitch.conf.5: Current glibc reloads this configuration file Florian Weimer
@ 2022-11-04 16:58 ` Alejandro Colomar
  2022-11-04 17:50   ` Florian Weimer
  0 siblings, 1 reply; 3+ messages in thread
From: Alejandro Colomar @ 2022-11-04 16:58 UTC (permalink / raw)
  To: Florian Weimer; +Cc: linux-man


[-- Attachment #1.1: Type: text/plain, Size: 2422 bytes --]

Hi Florian!

On 11/4/22 17:47, Florian Weimer wrote:
> The implementation happened across multiple glibc commits, but the work
> was completed for glibc 2.33.
> 
> Signed-off-by: Florian Weimer <fweimer@redhat.com>

Thanks for the patch!

May I ask that you CC me in patches?  I sometimes miss patches sent only to the 
list.  (If there are any other patches from you that I ignored, please ping).

See a minor formatting comment below.

BTW, I remember some month ago having followed the discussion about adding 
arc4random(3).  Did you finally add it?  If so, would any of you (glibc 
programmers) mind sending a manual page?

Thanks a lot!

Cheers,

Alex

> 
> ---
>   man5/nsswitch.conf.5 | 12 +++++++-----
>   1 file changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git a/man5/nsswitch.conf.5 b/man5/nsswitch.conf.5
> index 098585cf1..7049455ad 100644
> --- a/man5/nsswitch.conf.5
> +++ b/man5/nsswitch.conf.5
> @@ -402,11 +402,13 @@ for respective databases:
>   .PD
>   .RE
>   .SH NOTES
> -Within each process that uses
> -.BR nsswitch.conf ,
> -the entire file is read only once.
> -If the file is later changed, the
> -process will continue using the old configuration.
> +Starting with glibc 2.33,
> +.\" https://sourceware.org/bugzilla/show_bug.cgi?id=12459
> +.B nsswitch.conf
> +is automatically reloaded if the file is changed.  In earlier
> +versions, the entire file was read only once within each process, and
> +if the file was later changed, the process would continue using the
> +old configuration.

Would you please reformat to use semantic line breaks?

man-pages(7):
    Use semantic newlines
        In  the  source of a manual page, new sentences should be
        started on new lines, long sentences should be split into
        lines at clause breaks (commas, semicolons,  colons,  and
        so on), and long clauses should be split at phrase bound‐
        aries.   This  convention,  sometimes  known as "semantic
        newlines", makes it easier to see the effect of  patches,
        which often operate at the level of individual sentences,
        clauses, or phrases.

>   .PP
>   Traditionally, there was only a single source for service information,
>   often in the form of a single configuration
> 
> base-commit: b106cd5bf43fe806100bb5266b78f92bac90b6e0
> 

-- 
<http://www.alejandro-colomar.es/>

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] nsswitch.conf.5: Current glibc reloads this configuration file
  2022-11-04 16:58 ` Alejandro Colomar
@ 2022-11-04 17:50   ` Florian Weimer
  0 siblings, 0 replies; 3+ messages in thread
From: Florian Weimer @ 2022-11-04 17:50 UTC (permalink / raw)
  To: Alejandro Colomar; +Cc: linux-man

* Alejandro Colomar:

> Hi Florian!
>
> On 11/4/22 17:47, Florian Weimer wrote:
>> The implementation happened across multiple glibc commits, but the work
>> was completed for glibc 2.33.
>> Signed-off-by: Florian Weimer <fweimer@redhat.com>
>
> Thanks for the patch!
>
> May I ask that you CC me in patches?  I sometimes miss patches sent
> only to the list.  (If there are any other patches from you that I
> ignored, please ping).

I'll try to remember that.

> See a minor formatting comment below.
>
> BTW, I remember some month ago having followed the discussion about
> adding arc4random(3).  Did you finally add it?  If so, would any of
> you (glibc programmers) mind sending a manual page?

It's in glibc 2.36.  I don't have a manual page yet.

Thanks,
Florian


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-11-04 17:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-11-04 16:47 [PATCH] nsswitch.conf.5: Current glibc reloads this configuration file Florian Weimer
2022-11-04 16:58 ` Alejandro Colomar
2022-11-04 17:50   ` Florian Weimer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox