[Buildroot] [buildroot][PATCH] Add isl package. This includes the correct URL for the library. This libary is used by needed by components that support the GCC graphite optimisations.

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Mon May 19 06:47:57 UTC 2014


Dear Steve Thomas,

The way the commit log is formatted is wrong. All git commits should
look like:

"""
isl: new package

Here some more details, after a *mandatory* empty blank line, to
explain some more things.

And I can add some more details here.

And even more there, through a much longer paragraph, and just to make
it longer, I'm typing some more random things.

Signed-off-by: John Doe <john.doe at buildroot.org>
---
Optionally here a changelog of what changed between iterations of the
patch.
"""

On Mon, 19 May 2014 02:34:01 +0000, Steve Thomas wrote:

> diff --git a/package/Config.in b/package/Config.in
> index 07fd166..2e71210 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -778,6 +778,7 @@ source "package/fftw/Config.in"
>  source "package/flann/Config.in"
>  source "package/glibmm/Config.in"
>  source "package/gmp/Config.in"
> +source "package/isl/Config.in"
>  source "package/gsl/Config.in"
>  source "package/gtest/Config.in"
>  source "package/libargtable2/Config.in"
> diff --git a/package/isl/Config.in b/package/isl/Config.in
> new file mode 100644
> index 0000000..bc2a3ef
> --- /dev/null
> +++ b/package/isl/Config.in
> @@ -0,0 +1,7 @@
> +config BR2_PACKAGE_ISL
> +	bool "isl"
> +	select BR2_PACKAGE_GMP
> +	help
> +	  Library for manipulating sets and relations of integer points bounded by linear constraints.
> +
> +	  http://freecode.com/projects/isl

I don't think there's a need for a target version of this library: this
library is only used on the host, by the compiler. So I would remove
the addition of package/isl/Config.in.

> diff --git a/package/isl/isl-000-libtool.patch b/package/isl/isl-000-libtool.patch
> new file mode 100644
> index 0000000..319c45a
> --- /dev/null
> +++ b/package/isl/isl-000-libtool.patch

All patches must include a description + Signed-off-by. Especially this
patch, which does some magic things.

> @@ -0,0 +1,28 @@
> +--- a/ltmain.sh	2014-04-23 23:52:39.104640815 +0000
> ++++ b/ltmain.sh	2014-04-23 23:56:59.818000038 +0000
> +@@ -2583,8 +2583,8 @@
> + 	# was found and let the user know that the "--tag" command
> + 	# line option must be used.
> + 	if test -z "$tagname"; then
> +-	  func_echo "unable to infer tagged configuration"
> +-	  func_fatal_error "specify a tag with '--tag'"
> ++	  func_echo "defaulting to \`CC'"
> ++	  func_echo "if this is not correct, specify a tag with \`--tag'"
> + #	else
> + #	  func_verbose "using $tagname tagged configuration"
> + 	fi
> +@@ -4172,8 +4172,12 @@
> + 	  # At present, this check doesn't affect windows .dll's that
> + 	  # are installed into $libdir/../bin (currently, that works fine)
> + 	  # but it's something to keep an eye on.
> +-	  test "$inst_prefix_dir" = "$destdir" && \
> +-	    func_fatal_error "error: cannot install '$file' to a directory not ending in $libdir"
> ++	  #
> ++	 # This breaks install into our staging area.  -PB
> ++	 #
> ++	 # test "$inst_prefix_dir" = "$destdir" && \
> ++	 #   func_fatal_error "error: cannot install \`$file' to a directory not ending in $libdir"
> ++
> + 
> + 	  if test -n "$inst_prefix_dir"; then
> + 	    # Stick the inst_prefix_dir data into the link command.
> diff --git a/package/isl/isl.mk b/package/isl/isl.mk
> new file mode 100644
> index 0000000..614cae1
> --- /dev/null
> +++ b/package/isl/isl.mk
> @@ -0,0 +1,17 @@
> +################################################################################
> +#
> +# isl
> +#
> +################################################################################
> +
> +ISL_VERSION = 0.13
> +ISL_SITE = http://isl.gforge.inria.fr/
> +ISL_LICENSE = MIT
> +ISL_LICENSE_FILES = MIT
> +ISL_INSTALL_STAGING = YES

You can get rid of this line (since we're not implementing the target
variant)

> +ISL_DEPENDENCIES = gmp
> +
> +ISL_LIBTOOL_PATCH = NO

Above this line, we probably want a comment:

# We have a custom ltmain.sh patch, so don't apply Buildroot libtool
# patches that are conflicting

> +
> +$(eval $(autotools-package))

And get rid of that one.

> +$(eval $(host-autotools-package))

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



More information about the buildroot mailing list