svn commit: r326095 - head/usr.sbin/bsdinstall/scripts

[Emmanuel Vadot]

On 2017-11-22 17:38, Ian Lepore wrote:
> On Wed, 2017-11-22 at 15:27 +0000, Emmanuel Vadot wrote:
>> Author: manu
>> Date: Wed Nov 22 15:27:47 2017
>> New Revision: 326095
>> URL: https://svnweb.freebsd.org/changeset/base/326095
>> 
>> Log:
>>   bsdinstall: Add ntpdate option
>>   
>>   When you install a computer for the first time, the date in the CMOS 
>> sometimes
>>   not accurate and you need to ntpdate as ntpd will fail a the time 
>> difference
>>   is too big.
>>   Add an option in bsdinstall to enable ntpdate that will do that for 
>> us.
>>   
>>   Reviewed by:	allanjude
>>   Differential Revision:	https://reviews.freebsd.org/D13149
>> 
>> Modified:
>>   head/usr.sbin/bsdinstall/scripts/services
>> 
>> Modified: head/usr.sbin/bsdinstall/scripts/services
>> ==============================================================================
>> --- head/usr.sbin/bsdinstall/scripts/services	Wed Nov 22 15:18:11 
>> 2017	(r326094)
>> +++ head/usr.sbin/bsdinstall/scripts/services	Wed Nov 22 15:27:47 
>> 2017	(r326095)
>> @@ -46,6 +46,8 @@ DAEMONS=$( dialog --backtitle "FreeBSD Installer" \
>>  	local_unbound "Local caching validating resolver" 
>> ${local_unbound:-off} \
>>  	sshd	"Secure shell daemon" ${sshd_enable:-off} \
>>  	moused	"PS/2 mouse pointer on console" ${moused_enable:-off} \
>> +	ntpdate	"Synchronize system and network time at bootime" \
>> +		${ntpdate_enable:-off} \
>>  	ntpd	"Synchronize system and network time" ${ntpd_enable:-off} \
>>  	powerd	"Adjust CPU frequency dynamically if supported" \
>>  		${powerd_enable:-off} \
>> 
> 
> The right way to enable a time-step at boot is to set the rc conf
> variable ntpd_sync_on_start to YES.  ntpdate has been deprecated for
> *years*.
> 
> -- Ian

  Hi Ian,

  Thanks I didn't know about option -g (nor ntpd_sync_on_start), this 
bring a few questions :

  - Is there any reason to not always use -g for ntpd ? As a lambda user 
I just want my time to be set, no matter what.
  - Should we remove ntpdate in -current before 12 if it's deprecated ?

  I'll make the necessary changes according to your (or others) answers.

  Cheers,

-- 
Emmanuel Vadot <manu at bidouilliste.com> <manu at freebsd.org>