[Buildroot] [PATCH 1/2 v2] new package: rasqal
Romain Naour
romain.naour at gmail.com
Mon Apr 17 11:33:57 UTC 2017
Hi Erwan,
Thanks for this contribution!
Le 09/03/2017 à 08:36, Erwan Gautron a écrit :
> Signed-off-by: Erwan Gautron <erwan.gautron at yahoo.fr>
>
> A free C library that handles
> Resource Description Framework (RDF)
> query syntaxes, query construction and
> query execution returning result bindings
No need to add twice your SoB line, keep the one below.
Also no need to indent. (it seems a copy paste from Config.in)
>
> Signed-off-by: Erwan Gautron <erwan.gautron at yahoo.fr>
> ---
> DEVELOPERS | 3 +++
> package/Config.in | 1 +
> package/rasqal/Config.in | 10 ++++++++++
> package/rasqal/rasqal.mk | 15 +++++++++++++++
> 4 files changed, 29 insertions(+)
> create mode 100755 package/rasqal/Config.in
> create mode 100755 package/rasqal/rasqal.mk
>
> diff --git a/DEVELOPERS b/DEVELOPERS
> index 6c74cac..afcf983 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -463,6 +463,9 @@ F: package/mxsldr/
> N: Ernesto L. Williams Jr <realcontrols at gmail.com>
> F: package/szip/
>
> +N: Erwan Gautron <erwan.gautron at yahoo.fr>
> +F: package/rasqal/
> +
> N: Evan Zelkowitz <evan.zelkowitz at gmail.com>
> F: package/sdl_gfx/
>
> diff --git a/package/Config.in b/package/Config.in
> index 9eb6a22..d46758d 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -1160,6 +1160,7 @@ menu "JSON/XML"
> source "package/rapidjson/Config.in"
> source "package/rapidxml/Config.in"
> source "package/raptor/Config.in"
> + source "package/rasqal/Config.in"
> source "package/tinyxml/Config.in"
> source "package/tinyxml2/Config.in"
> source "package/valijson/Config.in"
> diff --git a/package/rasqal/Config.in b/package/rasqal/Config.in
> new file mode 100755
> index 0000000..40a3adb
> --- /dev/null
> +++ b/package/rasqal/Config.in
> @@ -0,0 +1,10 @@
> +config BR2_PACKAGE_RASQAL
> + bool "rasqal"
> + select BR2_PACKAGE_RAPTOR
> + help
> + A free C library that handles
> + Resource Description Framework (RDF)
> + query syntaxes, query construction and
> + query execution returning result bindings
> +
> + http://librdf.org/rasqal/
> diff --git a/package/rasqal/rasqal.mk b/package/rasqal/rasqal.mk
> new file mode 100755
> index 0000000..8fb1e70
> --- /dev/null
> +++ b/package/rasqal/rasqal.mk
> @@ -0,0 +1,15 @@
> +################################################################################
> +#
> +# rasqal
> +#
> +################################################################################
Add a new line here.
> +RASQAL_VERSION = 0.9.33
> +RASQAL_SITE = http://download.librdf.org/source
> +RASQAL_DEPENDENCIES = raptor
You need to add host-pkgconf since the configure script use PKG_CHECK_MODULES macro.
Also, pcre dependency is auto detected and seems to be optional.
Can you try to add something like:
ifeq ($(BR2_PACKAGE_PCRE),y)
RASQAL_DEPENDENCIES += pcre
RASQAL_CONF_OPTS += --enable-pcre
else
RASQAL_CONF_OPTS += --disable-pcre
endif
Otherwise the build can fail with:
rasqal_regex.c:41:18: fatal error: pcre.h: No such file or directory
#include <pcre.h>
> +RASQAL_LICENSE = GPLv2+ or LGPLv2.1+ or Apache-2.0+
We recently switched to SPDX license format:
https://spdx.org/licenses/
It should be:
GPL-2.0 or LGPL-2.1+ or Apache-2.0
> +RASQAL_LICENSE_FILES = LICENSE.txt
Also add LICENSE-2.0.txt
> +
> +RASQAL_AUTORECONF = YES
Usually we add a comment to explain why autoreconf is needed.
I'll mark your patch "Changes Requested" in patchwork, can you resend it after
taking into account all comments ?
Also I suggest to use test-pkg script in order to detect most of the obvious
build issues [1] and use the new check-package script [2] for the coding style.
Note, you have to work against master branch to get all these new features.
[1] http://nightly.buildroot.org/manual.html#_tips_and_tricks
[2] ./support/scripts/check-package package/rasqal
PS: ignore the issue with br-arm-cortex-m4-full toolchain:
elf2flt terminated with signal 11
Best regards,
Romain
> +RASQAL_INSTALL_STAGING = YES
> +
> +$(eval $(autotools-package))
>
More information about the buildroot
mailing list