All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.