[Buildroot] [PATCH 1/2] Add libjson-glib to packages
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Mon Jan 24 17:20:54 UTC 2011
Hello,
On Mon, 17 Jan 2011 02:17:24 -0500
Elias Woods <eliaswoods at gmail.com> wrote:
> +config BR2_PACKAGE_LIBJSONGLIB
> + bool "libjson-glib"
> + depends on BR2_PACKAGE_LIBGLIB2 # glib2
We usually prefer "select" dependencies for library dependencies, but
in that case, it's true that the library is very libglib2-specific, so
we may decide otherwise. I'll let Peter decide on this, I don't have a
strong opinion.
> + help
> + Many high-level languages already provide native modules for parsing, generating and manipulating JSON data streams. JSON-GLib is a C library based on GLib and released under the terms of the GNU Lesser General Public License version 2.1. It provides a parser and a generator GObject classes and various wrappers for the complex data types employed by JSON, such as arrays and objects.
Needs to be wrapped at ~80 columns. And the indentation level for the
help text is one tab + 2 spaces.
> --- /dev/null
> +++ b/package/libjson-glib/libjson-glib.mk
> @@ -0,0 +1,31 @@
> +#############################################################
> +#
> +# libjson-glib
> +#
> +#############################################################
> +LIBJSONGLIB_VERSION_MAJOR = 0
> +LIBJSONGLIB_VERSION_MINOR = 12
> +LIBJSONGLIB_VERSION_MICRO = 2
Very minor nitpick: maybe there could be only two variables: one with
0.12 and the other with 2 ?
LIBJSONGLIB_VERSION_MAJOR = 0.12
LIBJSONGLIB_VERSION_MINOR = 2
Like this is done in package/libglib2/libglib2.mk or
package/libgtk2/libgtk2.mk.
> +LIBJSONGLIB_LIBTOOL_PATCH = NO
Why ?
> +LIBJSONGLIB_INSTALL_STAGING = YES
> +LIBJSONGLIB_INSTALL_TARGET = YES
> +#LIBJSONGLIB_INSTALL_STAGING_OPT = DESTDIR=$(STAGING_DIR) LDFLAGS=-L$(STAGING_DIR)/usr/lib install
If it's commented, it's useless, so remove.
> +$(eval $(call AUTOTARGETS,package,LIBJSONGLIB))
LIBJSONGLIB should be lowercase here.
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