[Bug 1323227] Review Request: parmetis - Parallel Graph Partitioning and Sparse Matrix Ordering library

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

 



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



--- Comment #1 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> ---
%description should be wrapped to 80 columns.

Suggestion:
%global _description \
PARMETIS is ... \
...

%description %_description

%description openmpi %_description

(This form gets the spacing right.)

Likewise for the cmake flags, please define a macro for the common stuff. This
will make it easier to see what flags are different between the four
compilations.

Group can be dropped (not required for EPEL6+ or Fedora).

Looks good. I'm building it now, and will continue the review when that's done.

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