[Buildroot] [PATCH] libqmi: new package
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Tue Jun 18 12:06:01 UTC 2013
Dear Patrick Ziegler,
On Tue, 18 Jun 2013 12:22:51 +0200, Patrick Ziegler wrote:
> +config BR2_PACKAGE_LIBQMI
> + bool "libqmi"
> + select BR2_PACKAGE_LIBGLIB2
You need to propagate the "depends on" that libglib2 has, i.e WCHAR.
> + help
> + libqmi is a glib-based library for talking to WWAN modems and devices
> + which speak the Qualcomm MSM Interface (QMI) protocol.
Upstream URL missing.
> +LIBQMI_VERSION = 1.4.0
> +LIBQMI_SITE = http://cgit.freedesktop.org/libqmi/snapshot/
> +LIBQMI_SOURCE = libqmi-$(LIBQMI_VERSION).tar.gz
This is the default, so you could remove this line.
> +LIBQMI_LICENSE = LGPLv2+ (library), GPLv2+ (programs)
> +LIBQMI_LICENSE_FILES = COPYING
> +LIBQMI_INSTALL_STAGING = YES
> +LIBQMI_AUTORECONF = YES
It would be good to have a comment that explains why autoreconf is
needed, i.e "The tarball is a Git snapshot, doesn't contain the
genrated configure script and Makefile" or something like this.
> +LIBQMI_DEPENDENCIES += libglib2
You could use a = here.
Best regards,
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