Re: F23 Self Contained Change: RPM MPI Requires Provides

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 - dep

Re: F23 Self Contained Change: RPM MPI Requires Provides

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

Sandro Mani 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 yesterday that

Re: F23 Self Contained Change: RPM MPI Requires Provides

On Sat, Aug 15, 2015 at 2:49 PM, Sandro Mani 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 way. Most of these a

Re: F23 Self Contained Change: RPM MPI Requires Provides

On 15.08.2015 13:31, Jan Kurik wrote: On Sat, Aug 15, 2015 at 5:45 AM, Orion Poplawski > 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].

Re: F23 Self Contained Change: RPM MPI Requires Provides

On Sat, Aug 15, 2015 at 5:45 AM, Orion Poplawski 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 that I

Re: F23 Self Contained Change: RPM MPI Requires Provides

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 MPI_F

Re: F23 Self Contained Change: RPM MPI Requires Provides

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 out

Re: F23 Self Contained Change: RPM MPI Requires Provides

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 ge

Re: F23 Self Contained Change: RPM MPI Requires Provides

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/ G

Re: F23 Self Contained Change: RPM MPI Requires Provides

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 nee

Re: F23 Self Contained Change: RPM MPI Requires Provides

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

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 Popla

Re: F23 Self Contained Change: RPM MPI Requires Provides

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

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 ^.*\

Re: F23 Self Contained Change: RPM MPI Requires Provides

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 score

Re: F23 Self Contained Change: RPM MPI Requires Provides

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:

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

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 *-ope

Re: F23 Self Contained Change: RPM MPI Requires Provides

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: libscorep_adapter_compiler_event.so.2()(64bit)(openmpi-x86_64

Re: F23 Self Contained Change: RPM MPI Requires Provides

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: libscore

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

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 > nee

Re: F23 Self Contained Change: RPM MPI Requires Provides

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 ava

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

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

Re: F23 Self Contained Change: RPM MPI Requires Provides

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

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

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 al

Re: F23 Self Contained Change: RPM MPI Requires Provides

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

Re: F23 Self Contained Change: RPM MPI Requires Provides

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 trigger

F23 Self Contained Change: RPM MPI Requires Provides

= Proposed Self Contained Change: RPM MPI Requires Provides = https://fedoraproject.org/wiki/Changes/RpmMPIReqProv Change owner(s): Sandro Mani Have the rpm-build find-provides and find-requires scripts encode the MPI compiler name in the provides string of a binary to distinguish otherwise i