[PATCH 0/1 v3][meta-networking] ntp: fix path to drift file

[PATCH 0/1 v3][meta-networking] ntp: fix path to drift file

[wenzong.fan]

From: Wenzong Fan <wenzong.fan@windriver.com>

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(-)

-- 
1.7.9.5

[PATCH 1/1 v3][meta-networking] ntp: fix path to drift file

[wenzong.fan]

From: Wenzong Fan <wenzong.fan@windriver.com>

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.

Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
---
 meta-networking/recipes-support/ntp/files/ntp.conf |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-networking/recipes-support/ntp/files/ntp.conf b/meta-networking/recipes-support/ntp/files/ntp.conf
index 875b7eb..676e186 100644
--- a/meta-networking/recipes-support/ntp/files/ntp.conf
+++ b/meta-networking/recipes-support/ntp/files/ntp.conf
@@ -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
-- 
1.7.9.5

Re: [PATCH 1/1 v3][meta-networking] ntp: fix path to drift file

[Peter A. Bigot]

On 10/13/2014 01:17 AM, wenzong.fan@windriver.com wrote:
> From: Wenzong Fan <wenzong.fan@windriver.com>
>
> 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

>
> Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
> ---
>   meta-networking/recipes-support/ntp/files/ntp.conf |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-networking/recipes-support/ntp/files/ntp.conf b/meta-networking/recipes-support/ntp/files/ntp.conf
> index 875b7eb..676e186 100644
> --- a/meta-networking/recipes-support/ntp/files/ntp.conf
> +++ b/meta-networking/recipes-support/ntp/files/ntp.conf
> @@ -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

Re: [PATCH 1/1 v3][meta-networking] ntp: fix path to drift file

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 2126 bytes --]

On Mon, Oct 13, 2014 at 07:38:03AM -0500, Peter A. Bigot wrote:
> On 10/13/2014 01:17 AM, wenzong.fan@windriver.com wrote:
> > From: Wenzong Fan <wenzong.fan@windriver.com>
> >
> > 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.

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

> 
> Peter
> 
> >
> > Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
> > ---
> >   meta-networking/recipes-support/ntp/files/ntp.conf |    2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta-networking/recipes-support/ntp/files/ntp.conf b/meta-networking/recipes-support/ntp/files/ntp.conf
> > index 875b7eb..676e186 100644
> > --- a/meta-networking/recipes-support/ntp/files/ntp.conf
> > +++ b/meta-networking/recipes-support/ntp/files/ntp.conf
> > @@ -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
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]