[Buildroot] [PATCH 1/1] libmraa: new package
Arnout Vandecappelle
arnout at mind.be
Tue Jan 5 00:48:46 UTC 2016
Hi Pieterjan,
On 04-01-16 20:46, Pieterjan Camerlynck wrote:
> libmraa provides a unified interface for IO on Galileo, Edison,
> Raspberry Pi and others
>
> Signed-off-by: Pieterjan Camerlynck <pieterjan.camerlynck at gmail.com>
> ---
> package/Config.in | 1 +
> package/libmraa/Config.in | 17 +++++++++++++++++
> package/libmraa/libmraa.mk | 22 ++++++++++++++++++++++
> 3 files changed, 40 insertions(+)
> create mode 100644 package/libmraa/Config.in
> create mode 100644 package/libmraa/libmraa.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 442021f..52f6b5b 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -930,6 +930,7 @@ menu "Hardware handling"
> source "package/libiqrf/Config.in"
> source "package/libllcp/Config.in"
> source "package/libmbim/Config.in"
> + source "package/libmraa/Config.in"
> source "package/libnfc/Config.in"
> source "package/libpciaccess/Config.in"
> source "package/libphidget/Config.in"
> diff --git a/package/libmraa/Config.in b/package/libmraa/Config.in
> new file mode 100644
> index 0000000..cfbe37c
> --- /dev/null
> +++ b/package/libmraa/Config.in
> @@ -0,0 +1,17 @@
> +config BR2_PACKAGE_LIBMRAA
> + bool "libmraa"
> + depends on BR2_INSTALL_LIBSTDCPP
Actually it doesn't contain any C++ code as far as I can see. So you should
patch CMakeLists.txt to set
project (mraa C)
(and send that patch upstream).
> + depends on BR2_TOOLCHAIN_HAS_THREADS
> + depends on !BR2_STATIC_LIBS
I don't see anything that would require dynamic libraries. Can you add a
comment why this is needed?
> + depends on BR2_ARM_CPU_ARMV7A || BR2_i386 || BR2_x86_64
It actually builds on any platform, except that the CMakeLists.txt explicitly
checks for arm or x86. But there is nothing armv7-specific, and supposedly the
library would still be useful on generic boards. So I'd just put BR2_arm.
> + help
> + Libmraa is a C/C++ library with bindings to javascript & python
Well, the js and python bindings aren't built... But that could be added in
follow-up patches so OK.
> + to interface with the IO on Galileo, Edison & other platforms,
> + with a structured and sane API where port names/numbering matches
> + the board that you are on.
> +
> + http://iotdk.intel.com/docs/master/mraa
> +
> +comment "libmraa needs a toolchain w/ C++, threads, dynamic library"
> + depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_INSTALL_LIBSTDCPP \
> + || BR2_STATIC_LIBS
> diff --git a/package/libmraa/libmraa.mk b/package/libmraa/libmraa.mk
> new file mode 100644
> index 0000000..099e883
> --- /dev/null
> +++ b/package/libmraa/libmraa.mk
> @@ -0,0 +1,22 @@
> +################################################################################
> +#
> +# libmraa
> +#
> +################################################################################
> +
> +LIBMRAA_VERSION = v0.9.0
> +LIBMRAA_SITE = $(call github,intel-iot-devkit,mraa,$(LIBMRAA_VERSION))
Upstream calls it mraa, not libmraa, so we prefer to stick to the same name.
Otherwise looks good.
Regards,
Arnout
> +LIBMRAA_LICENSE = MIT
> +LIBMRAA_LICENSE_FILES = COPYING
> +LIBMRAA_INSTALL_STAGING = YES
> +
> +LIBMRAA_CONF_OPTS += \
> + -DBUILDSWIG=OFF \
> + -DUSBPLAT=OFF \
> + -DFTDI4222=OFF \
> + -DIPK=OFF \
> + -DRPM=OFF \
> + -DENABLEEXAMPLES=OFF \
> + -DTESTS=OFF
> +
> +$(eval $(cmake-package))
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
More information about the buildroot
mailing list