[Buildroot] [PATCH] add new library: libpugixml

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Mon Sep 5 21:30:27 UTC 2016


Hello,

Thanks for your contribution!

This review is in addition to what Samuel Martin already said on the
same patch.

On Sun,  4 Sep 2016 23:08:46 +0200, Theo Debrouwere wrote:

> +LIBPUGIXML_VERSION = 1.7
> +LIBPUGIXML_SOURCE = pugixml-$(LIBPUGIXML_VERSION).tar.gz

Please use pugixml for the package name, since it's the upstream name.
Then you can drop this line as the value will be the default.

> +LIBPUGIXML_SITE = http://github.com/zeux/pugixml/releases/download/v$(LIBPUGIXML_VERSION)
> +LIBPUGIXML_LICENSE = MIT
> +LIBPUGIXML_LICENSE_FILES = docs/manual.html

Using an HTML file here is not really appropriate, as we concatenate
all license files into a big text file, so it's not really going to
look nice :/

The readme.txt is good enough as a license file.

> +LIBPUGIXML_INSTALL_STAGING = YES
> +
> +LIBPUGIXML_SUBDIR = scripts

Hum, why?

> +
> +# build libpugixml as a shared library
> +LIBPUGIXML_CONF_OPTS = -DBUILD_SHARED_LIBS=ON
> +
> +$(eval $(cmake-package))

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


More information about the buildroot mailing list