Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 20 Jul 2018 11:45:36 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        Ian Lepore <ian@freebsd.org>
Cc:        Ravi Pokala <rpokala@freebsd.org>, src-committers <src-committers@freebsd.org>,  svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r336526 - head
Message-ID:  <CANCZdfr7uBbGfSrXMmDD-3CLnm1jiqeUe7_0bnMtcCWDk9gj1Q@mail.gmail.com>
In-Reply-To: <1532108303.1344.57.camel@freebsd.org>
References:  <201807200044.w6K0i4QQ079894@repo.freebsd.org> <FB81978A-C563-48E8-B259-A607126A7301@panasas.com> <1532108303.1344.57.camel@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Jul 20, 2018 at 11:38 AM, Ian Lepore <ian@freebsd.org> wrote:

> On Fri, 2018-07-20 at 09:56 -0700, Ravi Pokala wrote:
> > Hi Ian,
> >
> > -----Original Message-----
> > From: <owner-src-committers@freebsd.org> on behalf of Ian Lepore
> > Date: 2018-07-19, Thursday at 17:44
> > To: <src-committers@freebsd.org>, <svn-src-all@freebsd.org>,
> > Subject: svn commit: r336526 - head
> >
> > >
> > > Author: ian
> > > Date: Fri Jul 20 00:44:04 2018
> > > New Revision: 336526
> > > URL: https://svnweb.freebsd.org/changeset/base/336526
> > >
> > > Log:
> > >   Add ntpd to the list of users/groups to check before installing.
> > >
> > > Modified:
> > >   head/Makefile.inc1
> > When trying to installworld:
> >
> >     ERROR: Required ntpd user is missing, see /usr/src/UPDATING.
> >
> > But there is no UPDATING entry about this. (Also, shouldn't it be
> ${SRCTOP}/UPDATING?)
> >
> > Thanks,
> >
> > Ravi (rpokala@)
>
> I wondered about that too, when I accidentally stumbled across the
> CHECK_UIDS thing and realized I should add ntpd to it. I searched in
> UPDATING for some of the other things in the CHECK_UIDS list to see
> what had been said about such changes in the past, even searching
> through old-old svn history, and found nothing.
>
> I finally settled on the thought that the reference to UPDATING was to
> the stuff at the bottom that never changes. In particular to the
> section around line 1595 that documents the instalkernel, reboot,
> mergemaster, installworld sequence. The mergemaster part should take
> care of the new userid, I guess.
>

It should. While what you say is true, it's also been true that in the past
people have often listed new userid's in the UPDATING text at the top to
give people more info about whether or not it's safe to skip mergemaster
and/or make alternative plans if it isn't and they don't use mergemaster :)

Warner



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfr7uBbGfSrXMmDD-3CLnm1jiqeUe7_0bnMtcCWDk9gj1Q>