Re: [meta-networking][meta-systemd][PATCH] ntp: Create a separate package for sntp

[Paul Eggleton <paul.eggleton@linux.intel.com>]

On Friday 15 February 2013 11:22:18 Khem Raj wrote:
> sntp provides the functionality of obsoleted ntpdate
> so we have option of using ntpd as well as sntp
> sntp does the immediate one time sync with time
> server and corrects the tme immediately. it
> replaces ntpdate nicely.
> 
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>  meta-networking/recipes-support/ntp/ntp.inc        |    3 ++-
>  .../meta-networking/recipes-support/ntp/ntp/sntp   |    1 +
>  .../recipes-support/ntp/ntp/sntp.service           |   11 +++++++++++
>  .../recipes-support/ntp/ntp_4.2.6p5.bbappend       |    9 ++++++++-
>  4 files changed, 22 insertions(+), 2 deletions(-)
>  create mode 100644
> meta-systemd/meta-networking/recipes-support/ntp/ntp/sntp create mode
> 100644 meta-systemd/meta-networking/recipes-support/ntp/ntp/sntp.service
> 
> diff --git a/meta-networking/recipes-support/ntp/ntp.inc
> b/meta-networking/recipes-support/ntp/ntp.inc index d93f22d..cec0f22 100644
> --- a/meta-networking/recipes-support/ntp/ntp.inc
> +++ b/meta-networking/recipes-support/ntp/ntp.inc
> @@ -54,7 +54,7 @@ do_install_append() {
>  	ln -s ${bindir}/ntpdate-sync ${D}/${sysconfdir}/network/if-up.d
>  }
> 
> -PACKAGES += "ntpdate ${PN}-tickadj ${PN}-utils"
> +PACKAGES += "ntpdate sntp ${PN}-tickadj ${PN}-utils"
>  # NOTE: you don't need ntpdate, use "ntpd -q -g -x"
> 
>  # ntp originally includes tickadj. It's split off for inclusion in small
> firmware images on platforms @@ -69,6 +69,7 @@ RSUGGESTS_${PN} = "iana-etc"
> 
>  FILES_${PN} = "${bindir}/ntpd ${sysconfdir}/ntp.conf
> ${sysconfdir}/init.d/ntpd ${sbindir} ${libdir}" FILES_${PN}-tickadj =
> "${bindir}/tickadj"
> +FILES_sntp = "${bindir}/sntp"
>  FILES_${PN}-utils = "${bindir}"
>  FILES_ntpdate = "${bindir}/ntpdate
> ${sysconfdir}/network/if-up.d/ntpdate-sync ${bindir}/ntpdate-sync
> ${sysconfdir}/default/ntpdate"
> 
> diff --git a/meta-systemd/meta-networking/recipes-support/ntp/ntp/sntp
> b/meta-systemd/meta-networking/recipes-support/ntp/ntp/sntp new file mode
> 100644
> index 0000000..9529cdf
> --- /dev/null
> +++ b/meta-systemd/meta-networking/recipes-support/ntp/ntp/sntp
> @@ -0,0 +1 @@
> +NTPSERVER="pool.ntp.org"

Argh, no, we cannot use this as a default. We discussed this at length 
earlier.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre