From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Mon, 19 May 2014 08:47:57 +0200 Subject: [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. In-Reply-To: References: Message-ID: <20140519084757.50a58189@free-electrons.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net 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 --- 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