[Buildroot] [PATCH 2/3] tclap: new package

[Thomas Petazzoni <thomas.petazzoni@free-electrons.com>]

Dear Matt Weber,

On Tue, 10 Jun 2014 08:02:05 -0700, Matt Weber wrote:

> diff --git a/package/tclap/Config.in b/package/tclap/Config.in
> new file mode 100644
> index 0000000..e50704b
> --- /dev/null
> +++ b/package/tclap/Config.in
> @@ -0,0 +1,12 @@
> +config BR2_PACKAGE_TCLAP
> +	bool "tclap"

This package needs C++ support, so the appropriate 'depends on' +
comment is needed.

> +	help
> +	  This is a small, flexible library that provides a simple interface for
> +	  defining and accessing command line arguments. It was intially inspired

intially -> initially.

> +	  by the user friendly CLAP libary. The difference is that this library is
> +	  templatized, so the argument class is type independent. Type independence
> +	  avoids identical-except-for-type objects, such as IntArg, FloatArg, and
> +	  StringArg. While the library is not strictly compliant with the GNU or
> +	  POSIX standards, it is close.

Please check lines do not exceed 72 chars.

> +	  http://tclap.sourceforge.net/
> diff --git a/package/tclap/tclap.mk b/package/tclap/tclap.mk
> new file mode 100644
> index 0000000..268b90b
> --- /dev/null
> +++ b/package/tclap/tclap.mk
> @@ -0,0 +1,16 @@
> +################################################################################
> +#
> +# tclap
> +#
> +################################################################################
> +
> +TCLAP_VERSION = 1.2.1
> +TCLAP_SITE = http://downloads.sourceforge.net/project/tclap
> +TCLAP_SOURCE = tclap-$(TCLAP_VERSION).tar.gz

Not needed, that's the default.

> +TCLAP_LICENSE = MIT
> +TCLAP_LICENSE_FILES = COPYING
> +
> +TCLAP_INSTALL_STAGING = YES
> +TCLAP_INSTALL_TARGET = NO

A short comment that explains why INSTALL_TARGET = NO is useful. Just
say it's a pure C++ template library, only made of headers.

Thanks!

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