[Buildroot] [PATCH v2 next 1/2] openmpi: new package

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Thu Jun 9 20:31:38 UTC 2016


Hello Vicente,

On Fri, 20 May 2016 12:01:00 +0100, Vicente Olivert Riera wrote:

> diff --git a/package/openmpi/0001-vt_iowrap.c-check-if-fgetc-and-fputc-have-been-alrea.patch b/package/openmpi/0001-vt_iowrap.c-check-if-fgetc-and-fputc-have-been-alrea.patch
> new file mode 100644
> index 0000000..00562d0
> --- /dev/null
> +++ b/package/openmpi/0001-vt_iowrap.c-check-if-fgetc-and-fputc-have-been-alrea.patch
> @@ -0,0 +1,48 @@
> +From 56a27e5dce8fc78941f26ccb0e688f331fb0641e Mon Sep 17 00:00:00 2001
> +From: Vicente Olivert Riera <Vincent.Riera at imgtec.com>
> +Date: Tue, 17 May 2016 20:46:11 +0100
> +Subject: [PATCH] vt_iowrap.c: check if fgetc and fputc have been already
> + defined

This patch is only needed to make the "Vampirtrace" stuff build, and
this is just in the "contrib" part of OpenMPI. So instead of adding a
patch, I've disabled VampirTrace by passing --disable-vt. Let me know
if you really needed VampirTrace. In the mean time, feel free to submit
your patch upstream so that we can re-enable the feature later on.

> +config BR2_PACKAGE_OPENMPI_EXTRACONF
> +	string "Additional parameters for ./configure"
> +	default ""
> +	help
> +	  Extra parameters that will be appended to openmpi's
> +	  ./configure commandline.

I really don't like this thing, since many of the configure options
imply a dependency on another package, which needs to be expressed in
the .mk file. So I've dropped this option, and I prefer to see
additional options being added gradually, depending on the needs. There
are indeed lots of options, but I am not sure we will actually have a
sub-option for each of them in Buildroot.

> +OPENMPI_VERSION_MAJOR = 1.10
> +OPENMPI_VERSION = $(OPENMPI_VERSION_MAJOR).2
> +OPENMPI_SITE = https://www.open-mpi.org/software/ompi/v$(OPENMPI_VERSION_MAJOR)/downloads
> +OPENMPI_SOURCE = openmpi-$(OPENMPI_VERSION).tar.bz2
> +OPENMPI_LICENSE = BSD

BSD is never a correct value. The right value here was BSD-3c.

Applied with those changes, thanks!

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


More information about the buildroot mailing list