Re: [PATCH] Add mpich support

["Aníbal Limón" <anibal.limon@linux.intel.com>]

[-- Attachment #1.1: Type: text/plain, Size: 5941 bytes --]

Hi all,

The issue for shell are because DEPENDS includes tcsh i remove it, also 
i fixed the rpath issue passing to the
OE_EXTRACONF --disable-rpath but still exists problem building with 
hydra pm (default) i changed to gforker.

I think the Hydra issues are for some Makefiles contains -rpath but it 
need more debug, here is the two patches that
actually works.

Kind regards,
     alimon


On 20/02/15 11:01, Alexandru Vaduva wrote:
> Yes, it is my opinion that adding to RDPENDS libxml2 and libpciaccess and changing the /bin/csh and /bin/bash to /bin/sh should solve those warnings.
> The Warnings are related with the patch done by Richard Yang. Here`s a link:[OE-core] [PATCH 0/6] meta: fix rdepends issues for test related packages
> |   |
> |   |   |   |   |   |
> | [OE-core] [PATCH 0/6] meta: fix rdepends issues for test related packages[OE-core] [PATCH 0/6] meta: fix rdepends issues for test related packages Robert Yang liezhi.yang at windriver.com Mon Aug 18 06:06:31 UTC 2014  |
> |  |
> | View on lists.openembedded.org | Preview by Yahoo |
> |  |
> |   |
>
> Hope I am not wrong  with my assumption, but I am pretty sure about this one ;)
>
> Alex Vaduva
>
>
>       On Friday, February 20, 2015 6:19 PM, "Rodriguez Bahena, Victor" <victor.rodriguez.bahena@intel.com> wrote:
>     
>
>   So let me see if I understand, the following error ( build log from this
> patch):
>
> NOTE: Executing RunQueue Tasks
> ERROR: QA Issue: package mpich contains bad
> RPATH /home/vrodri3/poky/build/tmp/sysroots/qemux86/usr/lib in
> file /home/vrodri3/poky/build/tmp/work/i586-poky-linux/mpich/3.1.1-r0/packages-split/mpich/usr/bin/i586-poky-linux-mpiexec.hydra [rpaths]
> WARNING: QA Issue: mpich rdepends on libxml2, but it isn't a build
> dependency? [build-deps]
> WARNING: QA Issue: mpich rdepends on libpciaccess, but it isn't a build
> dependency? [build-deps]
> WARNING: QA Issue: mpich requires /usr/bin/bash, but no providers in its
> RDEPENDS [file-rdeps]
> ERROR: QA run found fatal errors. Please consider fixing them.
> ERROR: Function failed: do_package_qa
> ERROR: Logfile of failure stored
> in: /home/vrodri3/poky/build/tmp/work/i586-poky-linux/mpich/3.1.1-r0/temp/log.do_package_qa.20022
> ERROR: Task 9
> (/home/vrodri3/meta-openembedded/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb, do_package_qa) failed with exit code '1'
> NOTE: Tasks Summary: Attempted 667 tasks of which 646 didn't need to be
> rerun and 1 failed.
> Waiting for 0 running tasks to finish:
>
>
> Is due to the fact that some of the QA scripts need sh instead of bash
> ( or something similar ) ?
>
> I am a little bit lost here
>
> Thanks for the help
>
> Victor Rodriguez
>
>
>
> On Thu, 2015-02-19 at 21:49 +0000, Alexandru Vaduva wrote:
>> Forgot to mention that I identified the line:
>> src/pm/hydra/tools/topo/hwloc/hwloc/contrib/hg/set-hg-share-perms.csh:1:#!/bin/csh -f
>>
>>
>> It seems that one script uses /bin/ch, and also another one
>> using /bin/bash.
>> Adding RDEPENDS_${PN} += "bash perl tcsh" did not solved the problem.
>> but maybe changing them o /bin/sh will do.
>> The idea is similar to the change from
>>
>> meta/recipes-extended/lsb/lsbtest/LSB_Test.s
>> Link: [OE-core] [PATCH 6/9] lsbtest: no bashism in LSB_Test.sh
>>
>>    
>>    
>>    
>>    
>>    
>>    
>> [OE-core] [PATCH 6/9] lsbtest: no bashism in LSB_Test.sh
>> [OE-core] [PATCH 6/9] lsbtest: no bashism in LSB_Test.sh Robert Yang
>> liezhi.yang at windriver.com Wed Aug 27 15:57:11 UTC 2014
>>
>> View on lists.openembedded.org
>> Preview by Yahoo
>>
>>    
>> What do you say?
>>
>>
>>
>>
>> Alex Vaduva
>>
>>
>>
>> On Thursday, February 19, 2015 11:44 PM, Alexandru. Vaduva
>> <Alexandru.Vaduva@enea.com> wrote:
>>
>>
>>
>> From: Victor Rodriguez <victor.rodriguez.bahena@intel.com>
>>
>> This patch add mpich support to yocto in order to run MPI framework on
>> embedded
>> systems.
>>
>> Signed-off-by: Alejandro Hernandez
>> <alejandro.hernandez@linux.intel.com>
>> Signed-off-by: Victor Rodriguez <victor.rodriguez.bahena@intel.com>
>> Signed-off-by: Alexandru.Vaduva <Alexandru.Vaduva@enea.com>
>> ---
>> meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb | 29
>> +++++++++++++++++++++++++++
>> 1 file changed, 29 insertions(+)
>> create mode 100644 meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb
>>
>> diff --git a/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb
>> b/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb
>> new file mode 100644
>> index 0000000..2f4226e
>> --- /dev/null
>> +++ b/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb
>> @@ -0,0 +1,29 @@
>> +SUMMARY = "Message Passing Interface(MPI) implementation"
>> +HOMEPAGE = "http://git.mpich.org/mpich.git/"
>> +SECTION = "devel"
>> +
>> +LICENSE = "BSD-2-Clause"
>> +LIC_FILES_CHKSUM =
>> "file://COPYRIGHT;md5=2106f0435056f3dd9349747a766e5816"
>> +
>> +SRC_URI =
>> "http://www.mpich.org/static/downloads/${PV}/mpich-${PV}.tar.gz"
>> +SRC_URI[md5sum] = "40dc408b1e03cc36d80209baaa2d32b7"
>> +SRC_URI[sha256sum] =
>> "455ccfaf4ec724d2cf5d8bff1f3d26a958ad196121e7ea26504fd3018757652d"
>> +
>> +RDEPENDS_${PN} += "bash perl tcsh"
>> +S = "${WORKDIR}/${PN}-${PV}"
>> +
>> +EXTRA_OECONF = "--enable-debuginfo \
>> +                --enable-fast \
>> +                --enable-shared  \
>> +                --disable-f77 \
>> +                --disable-fc \
>> +                --disable-fortran \
>> +                --disable-cxx"
>> +
>> +inherit autotools-brokensep gettext
>> +
>> +do_configure_prepend() {
>> +    autoreconf --verbose --install --force -I . -I confdb/ -I maint/
>> +    oe_runconf
>> +    exit
>> +}
>> -- 
>> 1.9.1
>>
>> -- 
>> _______________________________________________
>> yocto mailing list
>> yocto@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/yocto
>>
>>
>>
>
>
>


[-- Attachment #1.2: Type: text/html, Size: 7217 bytes --]

[-- Attachment #2: 0001-Add-mpich-support.patch --]
[-- Type: text/x-patch, Size: 1911 bytes --]

From 49b5e0ec2b8b7ac3ebbdc0307d193c2e2476e272 Mon Sep 17 00:00:00 2001
From: Victor Rodriguez <victor.rodriguez.bahena@intel.com>
Date: Thu, 19 Feb 2015 22:43:09 +0100
Subject: [PATCH 1/2] Add mpich support

This patch add mpich support to yocto in order to run MPI framework on embedded
systems.

Signed-off-by: Alejandro Hernandez <alejandro.hernandez@linux.intel.com>
Signed-off-by: Victor Rodriguez <victor.rodriguez.bahena@intel.com>
Signed-off-by: Alexandru.Vaduva <Alexandru.Vaduva@enea.com>
---
 meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb | 29 +++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)
 create mode 100644 meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb

diff --git a/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb b/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb
new file mode 100644
index 0000000..2f4226e
--- /dev/null
+++ b/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb
@@ -0,0 +1,29 @@
+SUMMARY = "Message Passing Interface(MPI) implementation"
+HOMEPAGE = "http://git.mpich.org/mpich.git/"
+SECTION = "devel"
+
+LICENSE = "BSD-2-Clause"
+LIC_FILES_CHKSUM = "file://COPYRIGHT;md5=2106f0435056f3dd9349747a766e5816"
+
+SRC_URI = "http://www.mpich.org/static/downloads/${PV}/mpich-${PV}.tar.gz"
+SRC_URI[md5sum] = "40dc408b1e03cc36d80209baaa2d32b7"
+SRC_URI[sha256sum] = "455ccfaf4ec724d2cf5d8bff1f3d26a958ad196121e7ea26504fd3018757652d"
+
+RDEPENDS_${PN} += "bash perl tcsh"
+S = "${WORKDIR}/${PN}-${PV}"
+
+EXTRA_OECONF = "--enable-debuginfo \
+                --enable-fast \
+                --enable-shared  \
+                --disable-f77 \
+                --disable-fc \
+                --disable-fortran \
+                --disable-cxx"
+
+inherit autotools-brokensep gettext
+
+do_configure_prepend() {
+    autoreconf --verbose --install --force -I . -I confdb/ -I maint/
+    oe_runconf
+    exit
+}
-- 
1.8.4.5


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #3: 0002-mpich-Disable-RPATH-to-avoid-issues-in-package_qa.patch --]
[-- Type: text/x-patch; name="0002-mpich-Disable-RPATH-to-avoid-issues-in-package_qa.patch", Size: 2061 bytes --]

From 621a1acff318d525cc6db82e6159adeeceb87f29 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?An=C3=ADbal=20Lim=C3=B3n?= <anibal.limon@linux.intel.com>
Date: Fri, 20 Feb 2015 13:31:13 +0000
Subject: [PATCH 2/2] mpich: Disable RPATH to avoid issues in package_qa
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Set --disable-rpath to extra opts in order to avoid issues
in package_qa also change default pm to gforker because
hydra has problems with RPATH.

TODO: Make a patch for solve hydra rpath issues.

Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
---
 meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb b/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb
index 2f4226e..0225cee 100644
--- a/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb
+++ b/meta-oe/recipes-devtools/mpich/mpich_3.1.1.bb
@@ -1,20 +1,25 @@
-SUMMARY = "Message Passing Interface(MPI) implementation"
-HOMEPAGE = "http://git.mpich.org/mpich.git/"
+SUMMARY = "Message Passing Interface (MPI) implementation"
+HOMEPAGE = "http://www.mpich.org/"
 SECTION = "devel"
 
 LICENSE = "BSD-2-Clause"
 LIC_FILES_CHKSUM = "file://COPYRIGHT;md5=2106f0435056f3dd9349747a766e5816"
 
-SRC_URI = "http://www.mpich.org/static/downloads/${PV}/mpich-${PV}.tar.gz"
+SRC_URI = " \
+	http://www.mpich.org/static/downloads/${PV}/mpich-${PV}.tar.gz \
+"
+
 SRC_URI[md5sum] = "40dc408b1e03cc36d80209baaa2d32b7"
 SRC_URI[sha256sum] = "455ccfaf4ec724d2cf5d8bff1f3d26a958ad196121e7ea26504fd3018757652d"
 
-RDEPENDS_${PN} += "bash perl tcsh"
+RDEPENDS_${PN} += "bash perl libxml2"
 S = "${WORKDIR}/${PN}-${PV}"
 
 EXTRA_OECONF = "--enable-debuginfo \
                 --enable-fast \
                 --enable-shared  \
+                --with-pm=gforker  \
+		--disable-rpath \
                 --disable-f77 \
                 --disable-fc \
                 --disable-fortran \
-- 
1.8.4.5