[Buildroot] [PATCH] chrony: new package
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Mon May 6 17:20:44 UTC 2013
Dear Nathan Lynch,
Thanks, this looks almost good. A few comments below.
On Mon, 6 May 2013 11:12:55 -0500, Nathan Lynch wrote:
> diff --git a/package/chrony/Config.in b/package/chrony/Config.in
> new file mode 100644
> index 0000000..a42bd77
> --- /dev/null
> +++ b/package/chrony/Config.in
> @@ -0,0 +1,5 @@
> +config BR2_PACKAGE_CHRONY
> + bool "chrony"
> + help
> + Chrony is a pair of programs which are used to maintain the
> + accuracy of the system clock.
We generally want a empty new line here, and then the upstream URL of
the project.
Also, did you check that this program indeed builds with a minimal
uClibc toolchain (no thread, no wchar, no locale, no nothing) ?
> diff --git a/package/chrony/chrony.mk b/package/chrony/chrony.mk
> new file mode 100644
> index 0000000..b9733f9
> --- /dev/null
> +++ b/package/chrony/chrony.mk
> @@ -0,0 +1,6 @@
We generally like to have our nice header here, like in all other
packages:
#############################################################
#
# bind
#
#############################################################
> +CHRONY_VERSION = 1.27
> +CHRONY_SITE = http://download.tuxfamily.org/chrony/
> +CHRONY_LICENSE = GPLv2
> +CHRONY_LICENSE_FILES = COPYING
> +
> +$(eval $(autotools-package))
Looks good.
Thanks!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
More information about the buildroot
mailing list