Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=562504 --- Comment #5 from Thomas Spura <tomspur@xxxxxxxxxxxxxxxxx> 2010-02-07 16:21:21 EST --- Next try ;) (In reply to comment #4) > LAM has been obsoleted long ago and dropped from rawhide, so drop it. Done > The correct locations for the modules are > %{python_sitearch}/openmpi/ > for the Open MPI library and and > %{python_sitearch}/mpich2/ > for the MPICH2 library. > > Is there *really* a *serial* version of the library? No, I deleted that part again, just building the parallel versions. > The -mpi package must be named -openmpi. Done > Drop the optional compilers support, it's only meant for MPI compiler packages > (e.g. openmpi and mpich2). Done > Is the egg independent of the MPI library used? Yes, but now it's shipped in every supbackage > I don't like the use of > %{_builddir}/$MPI_COMPILER > please create the temporary directories inside the same buildroot, e.g. > mv build $MPI_COMPILER Now I move build around, should be better now. - koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1967719 - rpmlint complains about devel-file-in-non-devel-package and no-documentation. Rest is ok. Remaining 'maybe' TODO: enable testsuite Spec URL: http://tomspur.fedorapeople.org/review/mpi4py.spec SRPM URL: http://tomspur.fedorapeople.org/review/mpi4py-1.2-3.fc12.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review