Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-08-19 Thread Dave Love
Sandro Mani manisan...@gmail.com writes: Where are we with this? I see some F23 builds but no updates yet. Formally, this Change has been removed from the scope of F23 as it was not ready for Alpha, which is a requirement to have a Change in a release. However Zbyszek told me

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-08-19 Thread Orion Poplawski
On 08/19/2015 06:36 AM, Dave Love wrote: Should MPI packages be OK to push in f23 currently? I've just had failures from auto-qa like this, though the build actually ran an MPI test successfully: not ok - depcheck for Koji build scalasca-2.2.2-2.fc23 # FAIL --- arch:

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-08-19 Thread Zbigniew Jędrzejewski-Szmek
On Wed, Aug 19, 2015 at 10:11:04AM -0600, Orion Poplawski wrote: On 08/19/2015 06:36 AM, Dave Love wrote: Should MPI packages be OK to push in f23 currently? I've just had failures from auto-qa like this, though the build actually ran an MPI test successfully: not ok - depcheck for

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-08-15 Thread Alexander Ploumistos
On Sat, Aug 15, 2015 at 2:49 PM, Sandro Mani manisan...@gmail.com wrote: So the issues I pointed out in my previous mail (conerning MPI_PYTHON_SITEARCH and MPI_FORTRAN_MOD_DIR) above have been resolved. A batch of F24 rebuilts has been done by Zbyszek, hitting some build failures along the

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-08-15 Thread Sandro Mani
On 15.08.2015 13:31, Jan Kurik wrote: On Sat, Aug 15, 2015 at 5:45 AM, Orion Poplawski or...@cora.nwra.com mailto:or...@cora.nwra.com wrote: On 07/27/2015 04:05 PM, Sandro Mani wrote: On 27.07.2015 23 tel:27.07.2015%2023:56, Sandro Mani wrote: Ok I've now got one

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-08-15 Thread Jan Kurik
On Sat, Aug 15, 2015 at 5:45 AM, Orion Poplawski or...@cora.nwra.com wrote: On 07/27/2015 04:05 PM, Sandro Mani wrote: On 27.07.2015 23:56, Sandro Mani wrote: Ok I've now got one full build of all MPI packages [1]. Investigating the output, things are looking good, except for the fact

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-08-14 Thread Orion Poplawski
On 07/27/2015 04:05 PM, Sandro Mani wrote: On 27.07.2015 23:56, Sandro Mani wrote: Ok I've now got one full build of all MPI packages [1]. Investigating the output, things are looking good, except for the fact that I realized that I'll also need to handle binaries MPI_PYTHON_SITEARCH and

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-27 Thread Sandro Mani
Ok I've now got one full build of all MPI packages [1]. Investigating the output, things are looking good, except for the fact that I realized that I'll also need to handle binaries MPI_PYTHON_SITEARCH and MPI_FORTRAN_MOD_DIR - these directories are outside MPI_HOME and hence currently don't

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-27 Thread Sandro Mani
On 27.07.2015 23:56, Sandro Mani wrote: Ok I've now got one full build of all MPI packages [1]. Investigating the output, things are looking good, except for the fact that I realized that I'll also need to handle binaries MPI_PYTHON_SITEARCH and MPI_FORTRAN_MOD_DIR - these directories are

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-21 Thread Orion Poplawski
On 07/21/2015 09:45 AM, Sandro Mani wrote: On 21.07.2015 17:41, Orion Poplawski wrote: On 07/17/2015 09:50 AM, Sandro Mani wrote: Yep - I'm now building things in copr [1]. [1] https://copr.fedoraproject.org/coprs/smani/rpm-mpi-hooks-testing/ Great. Looks like you need to build a newer

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-21 Thread Sandro Mani
On 21.07.2015 18:03, Orion Poplawski wrote: On 07/21/2015 09:45 AM, Sandro Mani wrote: On 21.07.2015 17:41, Orion Poplawski wrote: On 07/17/2015 09:50 AM, Sandro Mani wrote: Yep - I'm now building things in copr [1]. [1] https://copr.fedoraproject.org/coprs/smani/rpm-mpi-hooks-testing/

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-21 Thread Orion Poplawski
On 07/17/2015 09:50 AM, Sandro Mani wrote: Yep - I'm now building things in copr [1]. [1] https://copr.fedoraproject.org/coprs/smani/rpm-mpi-hooks-testing/ Great. Looks like you need to build a newer openmpi package in your copr since I updating it to 1.8.7 in rawhide. -- Orion

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-21 Thread Sandro Mani
On 21.07.2015 17:41, Orion Poplawski wrote: On 07/17/2015 09:50 AM, Sandro Mani wrote: Yep - I'm now building things in copr [1]. [1] https://copr.fedoraproject.org/coprs/smani/rpm-mpi-hooks-testing/ Great. Looks like you need to build a newer openmpi package in your copr since I updating

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-17 Thread Orion Poplawski
On 07/14/2015 08:09 AM, Sandro Mani wrote: On 09.07.2015 23:17, Orion Poplawski wrote: On 07/09/2015 03:06 PM, Sandro Mani wrote: Ah yes sorry didn't read the contents of fileattrs/libsymlink.attr properly. But couldn't that be handled with a %__libsymlink_path ^.*\.so$

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-17 Thread Sandro Mani
On 17.07.2015 17:40, Orion Poplawski wrote: On 07/14/2015 08:09 AM, Sandro Mani wrote: On 09.07.2015 23:17, Orion Poplawski wrote: On 07/09/2015 03:06 PM, Sandro Mani wrote: Ah yes sorry didn't read the contents of fileattrs/libsymlink.attr properly. But couldn't that be handled with a

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-14 Thread Sandro Mani
On 09.07.2015 23:17, Orion Poplawski wrote: On 07/09/2015 03:06 PM, Sandro Mani wrote: On 09.07.2015 21:42, Orion Poplawski wrote: On 07/09/2015 01:14 PM, Sandro Mani wrote: On 09.07.2015 20:18, Orion Poplawski wrote: Also, it doesn't seem to get all of the requires quite right. For

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-09 Thread Orion Poplawski
On 07/07/2015 03:12 AM, Sandro Mani wrote: Hello I've got an initial implementation of this using the rpm dependency generator hooks, as suggested in the other thread [1]. The resulting scripts are here: https://smani.fedorapeople.org/rpm-mpi-hooks/ There is just one problem: an elf

Re: F23 Self Contained Change: RPM MPI Requires Provides - Review request

2015-07-09 Thread Orion Poplawski
On 07/08/2015 05:03 PM, Sandro Mani wrote: On 08.07.2015 22:28, Orion Poplawski wrote: It appears to be sufficient to define this macro anywhere, not just in elf.attr. So I think it could be added in to a rpm macros file in openmpi/mpich-devel. Ah cool, actually it seems to also work if

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-09 Thread Sandro Mani
On 09.07.2015 20:18, Orion Poplawski wrote: Also, isn't the -x86_64 redundant? Also though I guess we don't have an mpi variable MPI_NAME. Yes it is redundant, but it is the prettiest variable I could find, given the lack of MPI_NAME. Also, your trick of using: for module in $(module

Re: F23 Self Contained Change: RPM MPI Requires Provides - Review request

2015-07-09 Thread Sandro Mani
On 09.07.2015 21:39, Orion Poplawski wrote: On 07/08/2015 05:03 PM, Sandro Mani wrote: Once the package is approved, openmpi and mpich will need to BuildRequire: rpm-mpi-hooks, and openmpi-devel and mpich-devel will need to Require: rpm-mpi-hooks. And finally, once that is done too, all

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-09 Thread Orion Poplawski
On 07/09/2015 03:06 PM, Sandro Mani wrote: On 09.07.2015 21:42, Orion Poplawski wrote: On 07/09/2015 01:14 PM, Sandro Mani wrote: On 09.07.2015 20:18, Orion Poplawski wrote: Also, it doesn't seem to get all of the requires quite right. For scorep-openmpi I have: Provides:

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-09 Thread Sandro Mani
On 09.07.2015 21:42, Orion Poplawski wrote: On 07/09/2015 01:14 PM, Sandro Mani wrote: On 09.07.2015 20:18, Orion Poplawski wrote: Also, it doesn't seem to get all of the requires quite right. For scorep-openmpi I have: Provides:

Re: F23 Self Contained Change: RPM MPI Requires Provides - Review request

2015-07-09 Thread Orion Poplawski
On 07/08/2015 05:03 PM, Sandro Mani wrote: Once the package is approved, openmpi and mpich will need to BuildRequire: rpm-mpi-hooks, and openmpi-devel and mpich-devel will need to Require: rpm-mpi-hooks. And finally, once that is done too, all *-openmpi and *-mpich packages will need to

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-09 Thread Orion Poplawski
On 07/09/2015 01:14 PM, Sandro Mani wrote: On 09.07.2015 20:18, Orion Poplawski wrote: Also, it doesn't seem to get all of the requires quite right. For scorep-openmpi I have: Provides: libscorep_adapter_compiler_event.so.2()(64bit)(openmpi-x86_64) but: Requires:

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-08 Thread Orion Poplawski
On 07/07/2015 03:12 AM, Sandro Mani wrote: Hello I've got an initial implementation of this using the rpm dependency generator hooks, as suggested in the other thread [1]. The resulting scripts are here: https://smani.fedorapeople.org/rpm-mpi-hooks/ There is just one problem: an elf

Re: F23 Self Contained Change: RPM MPI Requires Provides - Review request

2015-07-08 Thread Sandro Mani
On 08.07.2015 22:28, Orion Poplawski wrote: It appears to be sufficient to define this macro anywhere, not just in elf.attr. So I think it could be added in to a rpm macros file in openmpi/mpich-devel. Ah cool, actually it seems to also work if I add it directly to mpi.attrs, meaning that

Re: F23 Self Contained Change: RPM MPI Requires Provides

2015-07-07 Thread Sandro Mani
Hello I've got an initial implementation of this using the rpm dependency generator hooks, as suggested in the other thread [1]. The resulting scripts are here: https://smani.fedorapeople.org/rpm-mpi-hooks/ There is just one problem: an elf binary in an $MPI_HOME subfolder will now

F23 Self Contained Change: RPM MPI Requires Provides

2015-06-18 Thread Jan Kurik
= Proposed Self Contained Change: RPM MPI Requires Provides = https://fedoraproject.org/wiki/Changes/RpmMPIReqProv Change owner(s): Sandro Mani manisandro at gmail dot com Have the rpm-build find-provides and find-requires scripts encode the MPI compiler name in the provides string of a binary

F23 Self Contained Change: RPM MPI Requires Provides

2015-06-18 Thread Jan Kurik
= Proposed Self Contained Change: RPM MPI Requires Provides = https://fedoraproject.org/wiki/Changes/RpmMPIReqProv Change owner(s): Sandro Mani manisandro at gmail dot com Have the rpm-build find-provides and find-requires scripts encode the MPI compiler name in the provides string of a binary