Skip to contentSkip to conversation

ntp: fix path to drift file

PATCH In review [email protected] 3 participants 4 messages Oct 13, 2014

lore
Share & tools
lore
#0 lore raw

v3 cover ntp: fix path to drift file

From: Wenzong Fan <[email protected]>

v3 changes:

* Update the drift file path in recipes-support/ntp/files/ntp.conf;
* Drop the logic for creating /var/lib/ntp, This addressed by Peter A. Bigot's patch:

[oe] [meta-networking][PATCH] ntp: create and package ntp home directory

The following changes since commit a4bdcbdbca05bbb4a452e06982038f0a38bcb91f:

openldap: use PN for PACKAGES_DYNAMIC (2014-10-10 12:47:35 +0200)

are available in the git repository at:

git://git.pokylinux.org/poky-contrib wenzong/ntp-fix
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=wenzong/ntp-fix

Wenzong Fan (1):
ntp: fix path to drift file

 meta-networking/recipes-support/ntp/files/ntp.conf |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Signature · 1 line
1.7.9.5
#1 lore raw

PATCH v3 1/1 ntp: fix path to drift file

From: Wenzong Fan <[email protected]>

The default path of ntp drift file is /etc/ntp.drift, ntp daemon maybe fails to create this file since the user ntp is not always permitted to write /etc.

Refer to other distributions such as RedHat, Debian, just moving the file to /var/lib/ntp which the home dir of user ntp.

---
 meta-networking/recipes-support/ntp/files/ntp.conf |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
meta-networking/recipes-support/ntp/files/ntp.conf +1 -1
index 875b7eb..676e186 100644
@@ -1,7 +1,7 @@
 # This is the most basic ntp configuration file
 # The driftfile must remain in a place specific to this
 # machine - it records the machine specific clock error
-driftfile /etc/ntp.drift
+driftfile /var/lib/ntp/drift
 # This should be a server that is close (in IP terms)
 # to the machine.  Add other servers as required.
 # Unless you un-comment the line below ntpd will sync
Signature · 1 line
1.7.9.5
#2 lore raw
Quoted 8 lines · On 10/13/2014 01:17 AM, [email protected] wrote:

From: Wenzong Fan <[email protected]>

The default path of ntp drift file is /etc/ntp.drift, ntp daemon maybe fails to create this file since the user ntp is not always permitted to write /etc.

Refer to other distributions such as RedHat, Debian, just moving the file to /var/lib/ntp which the home dir of user ntp.

Note that this new version depends on http://patches.openembedded.org/patch/81585/ which provides the packaging of ntp's home directory, which wasn't captured into master-next. That would be obscured by the presence of the previous version of this patch which (IMO) improperly uses volatile to create that directory.

Peter

Quoted 19 lines
---
  meta-networking/recipes-support/ntp/files/ntp.conf |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
meta-networking/recipes-support/ntp/files/ntp.conf +1 -1
index 875b7eb..676e186 100644
@@ -1,7 +1,7 @@
  # This is the most basic ntp configuration file
  # The driftfile must remain in a place specific to this
  # machine - it records the machine specific clock error
-driftfile /etc/ntp.drift
+driftfile /var/lib/ntp/drift
  # This should be a server that is close (in IP terms)
  # to the machine.  Add other servers as required.
  # Unless you un-comment the line below ntpd will sync
#3 lore raw
Quoted 10 lines · Peter A. Bigot
Quoted 8 lines · On 10/13/2014 01:17 AM, [email protected] wrote:

From: Wenzong Fan <[email protected]>

The default path of ntp drift file is /etc/ntp.drift, ntp daemon maybe fails to create this file since the user ntp is not always permitted to write /etc.

Refer to other distributions such as RedHat, Debian, just moving the file to /var/lib/ntp which the home dir of user ntp.

Quoted 6 lines

Note that this new version depends on
http://patches.openembedded.org/patch/81585/ which provides the packaging of ntp's home directory, which wasn't captured into master-next. That would be obscured by the presence of the previous version of this patch which (IMO) improperly uses volatile to create that directory.

Thanks for keeping eye on me, it should be fixed in master-next now.

Quoted 28 lines

Peter

Quoted 19 lines
---
  meta-networking/recipes-support/ntp/files/ntp.conf |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
meta-networking/recipes-support/ntp/files/ntp.conf +1 -1
index 875b7eb..676e186 100644
@@ -1,7 +1,7 @@
  # This is the most basic ntp configuration file
  # The driftfile must remain in a place specific to this
  # machine - it records the machine specific clock error
-driftfile /etc/ntp.drift
+driftfile /var/lib/ntp/drift
  # This should be a server that is close (in IP terms)
  # to the machine.  Add other servers as required.
  # Unless you un-comment the line below ntpd will sync
Signature · 4 lines
_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-devel
Signature · 1 line
Martin 'JaMa' Jansa     jabber: [email protected]