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? Pierre -- http://lists.linuxfromscratch.org/listinfo/blfs-support FAQ: http://www.linuxfromscratch.org/blfs/faq.html Unsubscribe: See the above information page
