[Buildroot] [PATCH v2 3/6] libatasmart: new package.
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Mon Jan 7 21:30:46 UTC 2013
Dear Marek Belisko,
On Mon, 7 Jan 2013 21:43:16 +0100, Marek Belisko wrote:
> diff --git a/package/Config.in b/package/Config.in
> index c889d8a..53bb5be 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -239,6 +239,8 @@ source "package/iostat/Config.in"
> source "package/irda-utils/Config.in"
> source "package/kbd/Config.in"
> source "package/lcdproc/Config.in"
> +source "package/libatasmart/Config.in"
> +source "package/linux-firmware/Config.in"
Seems like a mismerge. You shouldn't be adding a
linux-firmware/Config.in include.
> +config BR2_PACKAGE_LIBATASMART
> + bool "libatasmart"
> + depends on BR2_PACKAGE_UDEV
> + help
> + Reading and Parsing Library.
This description sounds odd. Reading and Parsing Library for what?
> + As the name suggests libatasmart only does ATA S.M.A.R.T.
> +
> + http://www.linuxfromscratch.org/blfs/view/svn/general/libatasmart.html
> +
> +comment "libatasmart requires udev to be enabled"
> + depends on !BR2_PACKAGE_UDEV
Could you detail a little why udev is a dependency. Is libudev a
dependency? Something else? We need to understand if it's a build
dependency, a runtime dependency, the requirement for the system to
actually use and run udev, etc.
> diff --git a/package/libatasmart/libatasmart.mk b/package/libatasmart/libatasmart.mk
> new file mode 100644
> index 0000000..3f8e36b
> --- /dev/null
> +++ b/package/libatasmart/libatasmart.mk
> @@ -0,0 +1,17 @@
> +#############################################################
> +#
> +# libatasmart
> +#
> +#############################################################
> +LIBATASMART_VERSION = 0.19
> +LIBATASMART_SOURCE = libatasmart-$(LIBATASMART_VERSION).tar.xz
> +LIBATASMART_SITE = http://0pointer.de/public
> +LIBATASMART_LICENCE = LGPL
LICENSE. Be more specific that LGPL.
> +LIBATASMART_LICELCE_FILE = LGPL
LICENSE_FILES
> +LIBATASMART_INSTALL_STAGING = YES
> +
> +LIBATASMART_AUTORECONF = YES
Please add a comment here that explains why AUTORECONF = YES is needed.
> +
> +LIBATASMART_DEPENDENCIES = udev
> +
> +$(eval $(autotools-package))
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