On 2/25/20 11:15 AM, Pierre Labastie via blfs-support wrote:
Le 25/02/2020 à 16:31, Bruce Dubbs via blfs-support a écrit :
On 2/25/20 4:57 AM, Pierre Labastie via blfs-support wrote:
Le 25/02/2020 à 10:36, Pierre Labastie via blfs-support a écrit :
Le 25/02/2020 à 10:22, Thomas Seeling via blfs-support a écrit :
Hallo,

I made 2 simple changes to the script:
We already have a sed in the book which addresses that, don't we?

I didn't verify before because we were discussing the current release,
but I did now and it's not in the book. The book mentions update-leap
but does not suggest any usage or modification hints.

Tschau...Thomas
--
Do you wanna be a legend or a passing footprint on the sands of time?


There is this:
---
sed -e 's/"(\\S+)"/"?([^\\s"]+)"?/' \
      -i scripts/update-leap/update-leap.in
---
Doesn't this is enough?


Actually, we are doing something wrong in the book: filenames should be quoted
in the .conf file, see the example in <ntp source dir>/ntpd/complete.conf.in

I'll fix that (and remove the sed). I'll also move the leapseconds file to
/var/lib/ntp, as discussed in this thread.

Will do some tests first, so maybe not today.

I just fixed it.  I prefer to remove the quotes.  The are only needed if there
are spaces in the file name (e.g. windows).

Hmmm, are you sure? update-leap is not the only program to read the .conf
file, and I am not sure what the others (ntpd is the most important) expect!
Since the example they give has quotes around filenames (an not around
hostnames for example), it may be that the the quotes are needed.
BTW, how do we know that the leapseconds file is used?

I tested with the update-leap script and it "did the right thing".
Looking through the code, I didn't see any place else where 'leapfile' is used.

  -- Bruce





--
http://lists.linuxfromscratch.org/listinfo/blfs-support
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page

Reply via email to