Openembedded Devel Discussions
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH 1/1 v3][meta-networking] ntp: fix path to drift file
Date: Mon, 13 Oct 2014 14:45:10 +0200	[thread overview]
Message-ID: <20141013124510.GG3000@jama> (raw)
In-Reply-To: <543BC7AB.6060908@pabigot.com>

[-- 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 --]

      reply	other threads:[~2014-10-13 12:43 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-13  6:17 [PATCH 0/1 v3][meta-networking] ntp: fix path to drift file wenzong.fan
2014-10-13  6:17 ` [PATCH 1/1 " wenzong.fan
2014-10-13 12:38   ` Peter A. Bigot
2014-10-13 12:45     ` Martin Jansa [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141013124510.GG3000@jama \
    --to=martin.jansa@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox