cvs commit: src/etc/rc.d hostid

Ralf S. Engelschall rse at FreeBSD.org
Mon May 21 11:48:13 UTC 2007


On Mon, May 21, 2007, Ralf S. Engelschall wrote:

> On Mon, May 21, 2007, Mike Makonnen wrote:
>
> > On Mon, May 21, 2007 at 08:22:44AM +0000, Ralf S. Engelschall wrote:
> > > rse         2007-05-21 08:22:44 UTC
> > >
> > >   FreeBSD src repository
> > >
> > >   Modified files:
> > >     etc/rc.d             hostid
> > >   Log:
> > >   The standardized textual representation of UUIDs according to RFC 4122
> > >   and ISO/IEC-9834-8:2005 is with LOWER-CASE hexadecimal characters only,
> > >   so translate the (usually upper-case and this way not conforming)
> > >   representation of the BIOS UUID when reading it. Also be more strict
> > >   about the valid characters in the textual representation by checking for
> > >   just the hexadecimal characters.
> >
> > tr(1) lives in /usr/bin, which may not be available when / (root) is
> > mounted (it may live on a remotely mounted filesystem, for example). So,
> > you can't rely on tr(1) being available when this script is run.
>
> Well, I explicitly checked the script and it contained uuidgen(1) which
> also stays under /usr/bin so I blindly concluded that tr(1) is fine, too
> ;-) So, although we can try to replace tr(1) with something else we have
> a problem as we cannot easily replace the uuidgen(1) call. Hmmm... I'll
> investigate. Thanks for the hint.

Ok, checked again and now fixed. First, sorry for posting
misinformation: uuidgen(1) already stays in /bin in CURRENT, of course.
So this is no problem. And the tr(1) usage I've now replaced with an
equivalent csh(1) usage. Not really nice but it was the best alternative
I've found with the limited tools available in /bin. Thanks for catching
this issue.

--
rse at FreeBSD.org                        Ralf S. Engelschall
FreeBSD.org/~rse                       rse at engelschall.com
FreeBSD committer                      www.engelschall.com



More information about the cvs-all mailing list