All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Peter A. Bigot" <pab@pabigot.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 07:38:03 -0500	[thread overview]
Message-ID: <543BC7AB.6060908@pabigot.com> (raw)
In-Reply-To: <0ba4f373d7be41147888b6a71728f7771ef28bb8.1413180347.git.wenzong.fan@windriver.com>

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



  reply	other threads:[~2014-10-13 12:38 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 [this message]
2014-10-13 12:45     ` Martin Jansa

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=543BC7AB.6060908@pabigot.com \
    --to=pab@pabigot.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.