[Bug 1282893] Review Request: quantum-espresso - A suite for electronic-structure calculations and materials modeling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1282893

Dave Love <d.love@xxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |d.love@xxxxxxxxxxxxxxx



--- Comment #10 from Dave Love <d.love@xxxxxxxxxxxxxxx> ---
Created attachment 1107142
  --> https://bugzilla.redhat.com/attachment.cgi?id=1107142&action=edit
fix requires

I looked at this, as I had an earlier version in copr (not suitable for
review).
The questions might well not be appropriate, but I think it would help to have
notes in the spec.

* It won't install because it requires an arch-specific -common package; patch
attached.

* I'd have thought iotk should be unbundled, but I don't know if it's of more
general use.

* Shouldn't this build against atlas or lapack/blas on non-x86?  (ppc64le seems
a plausible architecture to run it on.)  I know that makes it particularly
suffer the BLAS mess in Fedora, but scalapack links against reference blas
anyhow.

* It's not using the default compilation and linking flags (not that I agree
with that requirement for computational programs).

* Will smp make not work?  There's no comment, and the build takes a while.

* Why not use elpa?  (I haven't tried with this version and the Fedora elpa.)

* Shouldn't the doc be installed?

* Would the GUI be useful?  (I don't know.)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]