Re: man7 - useradd query

Re: man7 - useradd query

[Alejandro Colomar]

[-- Attachment #1: Type: text/plain, Size: 2473 bytes --]

Hi Rituparna,

On Thu, Oct 12, 2023 at 08:24:13PM +0530, Rituparna ghosh wrote:
> Hello
> I am not sure if this is the right place to raise this concern I have from
> man7 https://man7.org/linux/man-pages/man8/useradd.8.html

That page says it belongs to the shadow project, not to the Linux
man-pages project

COLOPHON         top

       This page is part of the shadow-utils (utilities for managing
       accounts and shadow password files) project.  Information about
       the project can be found at 
       ⟨https://github.com/shadow-maint/shadow⟩.  If you have a bug
       report for this manual page, send it to
       pkg-shadow-devel@alioth-lists.debian.net.  This page was obtained
       from the project's upstream Git repository
       ⟨https://github.com/shadow-maint/shadow⟩ on 2023-06-23.  (At that
       time, the date of the most recent commit that was found in the
       repository was 2023-06-22.)  If you discover any rendering
       problems in this HTML version of the page, or you believe there
       is a better or more up-to-date source for the page, or you have
       corrections or improvements to the information in this COLOPHON
       (which is not part of the original manual page), send a mail to
       man-pages@man7.org

I've CCed the mailing list for shadow, although the project is more
active on GitHub, in the link above.  I've also CCed Iker, a shadow
maintainer from Red Hat.

> 
> The useradd command here is showing an option   -F which is shown as
> invalid in RH 8.8 or RH 9.
> Is this correct?

From what I see in the source code of useradd.c in shadow, the feature
is compiled conditionally, #ifdef ENABLE_SUBIDS.  It may be that RHEL,
or your version, doesn't build with that option; I ignore it, I'm a
Debian user.

alx@debian:~/src/shadow/shadow/master$ grep -rl add.subids.for.system
man/useradd.8.xml
po/fr.po
tests/run_all.coverage
tests/run_all
src/useradd.c
alx@debian:~/src/shadow/shadow/master$ grep -rn -C1 add.subids.for.system src/
src/useradd.c-954-#ifdef ENABLE_SUBIDS
src/useradd.c:955:	(void) fputs (_("  -F, --add-subids-for-system   add entries to sub[ud]id even when adding a system user\n"), usageout);
src/useradd.c-956-#endif
--
src/useradd.c-1245-#ifdef ENABLE_SUBIDS
src/useradd.c:1246:			{"add-subids-for-system", no_argument,NULL, 'F'},
src/useradd.c-1247-#endif

Cheers,
Alex

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

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

Re: man7 - useradd query

[Serge E. Hallyn]

On Thu, Oct 12, 2023 at 06:43:15PM +0200, Alejandro Colomar wrote:
> Hi Rituparna,
> 
> On Thu, Oct 12, 2023 at 08:24:13PM +0530, Rituparna ghosh wrote:
> > Hello
> > I am not sure if this is the right place to raise this concern I have from
> > man7 https://man7.org/linux/man-pages/man8/useradd.8.html
> 
> That page says it belongs to the shadow project, not to the Linux
> man-pages project
> 
> COLOPHON         top
> 
>        This page is part of the shadow-utils (utilities for managing
>        accounts and shadow password files) project.  Information about
>        the project can be found at 
>        ⟨https://github.com/shadow-maint/shadow⟩.  If you have a bug
>        report for this manual page, send it to
>        pkg-shadow-devel@alioth-lists.debian.net.  This page was obtained
>        from the project's upstream Git repository
>        ⟨https://github.com/shadow-maint/shadow⟩ on 2023-06-23.  (At that
>        time, the date of the most recent commit that was found in the
>        repository was 2023-06-22.)  If you discover any rendering
>        problems in this HTML version of the page, or you believe there
>        is a better or more up-to-date source for the page, or you have
>        corrections or improvements to the information in this COLOPHON
>        (which is not part of the original manual page), send a mail to
>        man-pages@man7.org
> 
> I've CCed the mailing list for shadow, although the project is more
> active on GitHub, in the link above.  I've also CCed Iker, a shadow
> maintainer from Red Hat.
> 
> > 
> > The useradd command here is showing an option   -F which is shown as
> > invalid in RH 8.8 or RH 9.
> > Is this correct?
> 
> From what I see in the source code of useradd.c in shadow, the feature
> is compiled conditionally, #ifdef ENABLE_SUBIDS.  It may be that RHEL,
> or your version, doesn't build with that option; I ignore it, I'm a
> Debian user.

Yeah - but the manpage should likewise should not show the option if
it is not compiled in, I guess.  At the moment it looks like we
conditionally complie some manpages, like newuidmap.1, but I don't
see any sections compiled out in the man/*.xml files.

> alx@debian:~/src/shadow/shadow/master$ grep -rl add.subids.for.system
> man/useradd.8.xml
> po/fr.po
> tests/run_all.coverage
> tests/run_all
> src/useradd.c
> alx@debian:~/src/shadow/shadow/master$ grep -rn -C1 add.subids.for.system src/
> src/useradd.c-954-#ifdef ENABLE_SUBIDS
> src/useradd.c:955:	(void) fputs (_("  -F, --add-subids-for-system   add entries to sub[ud]id even when adding a system user\n"), usageout);
> src/useradd.c-956-#endif
> --
> src/useradd.c-1245-#ifdef ENABLE_SUBIDS
> src/useradd.c:1246:			{"add-subids-for-system", no_argument,NULL, 'F'},
> src/useradd.c-1247-#endif
> 
> Cheers,
> Alex
> 
> -- 
> <https://www.alejandro-colomar.es/>

Re: man7 - useradd query

[Alejandro Colomar]

[-- Attachment #1: Type: text/plain, Size: 1153 bytes --]

Hi Iker,

On Tue, Oct 17, 2023 at 08:44:08AM -0500, Serge E. Hallyn wrote:
[...]

> > > The useradd command here is showing an option   -F which is shown as
> > > invalid in RH 8.8 or RH 9.
> > > Is this correct?
> > 
> > From what I see in the source code of useradd.c in shadow, the feature
> > is compiled conditionally, #ifdef ENABLE_SUBIDS.  It may be that RHEL,
> > or your version, doesn't build with that option; I ignore it, I'm a
> > Debian user.
> 
> Yeah - but the manpage should likewise should not show the option if
> it is not compiled in, I guess.

This sounds reasonable.

>  At the moment it looks like we
> conditionally complie some manpages, like newuidmap.1, but I don't
> see any sections compiled out in the man/*.xml files.

You mean that currently it's either the entire page or nothing at all,
right?  I don't see anything that compiles part of a page either.

I imagine that you could do something like that by wrapping the
conditional part in something like <ifdef id="ENABLE_SUBIDS">...</ifdef>,
and write some script to handle it.

Cheers,
Alex

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

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

Re: man7 - useradd query

[Serge E. Hallyn]

On Tue, Oct 17, 2023 at 04:17:42PM +0200, Alejandro Colomar wrote:
> Hi Iker,
> 
> On Tue, Oct 17, 2023 at 08:44:08AM -0500, Serge E. Hallyn wrote:
> [...]
> 
> > > > The useradd command here is showing an option   -F which is shown as
> > > > invalid in RH 8.8 or RH 9.
> > > > Is this correct?
> > > 
> > > From what I see in the source code of useradd.c in shadow, the feature
> > > is compiled conditionally, #ifdef ENABLE_SUBIDS.  It may be that RHEL,
> > > or your version, doesn't build with that option; I ignore it, I'm a
> > > Debian user.
> > 
> > Yeah - but the manpage should likewise should not show the option if
> > it is not compiled in, I guess.
> 
> This sounds reasonable.
> 
> >  At the moment it looks like we
> > conditionally complie some manpages, like newuidmap.1, but I don't
> > see any sections compiled out in the man/*.xml files.
> 
> You mean that currently it's either the entire page or nothing at all,
> right?  I don't see anything that compiles part of a page either.

Right.

> I imagine that you could do something like that by wrapping the
> conditional part in something like <ifdef id="ENABLE_SUBIDS">...</ifdef>,
> and write some script to handle it.

Yeah.  It shouldn't be too bad.