[Bug 1144802] Review Request: globus-xio-rate-driver - Globus Toolkit - Globus XIO Rate Limiting Driver

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

 



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



--- Comment #2 from Mattias Ellert <mattias.ellert@xxxxxxxxxxxx> ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #1)

Many thanks for taking on the review!

> The same as #1181118:
> 5. %description reads like an advertisement. It should at least mention what
> "grid" is. Who develops it is not crucial. Also the last paragraph "The
> %{name package contains..." is not really useful, since it repeats the
> Summary.

The first part of the description is taken from upstream's website
http://toolkit.globus.org/toolkit/. It is upstream's own description of the
software, so I think it is relevant. The same sentence appears in the
description of the 50+ Globus Toolkit packages that already exists in Fedora.

I can recognize that grid is not the same hype and buzz word it was 5-10 years
ago, but the meaning of the word hasn't changed. Is it really forgotten to the
extent that it is not possible to use it anymore?

> 8. %files should have (note extra dot)
> %{_libdir}/libglobus_net_manager.so.*

This looks like a cut and paste error from the globus-net-manager review.

> 9. where does Source8 come from?

I am not sure what you are asking here. There is no upstream source for this
file anywhere. It is a collection of links to the upstream online documentation
compiled for the purpose of creating the package. Since the upstream project
groups their packages and provide documentation per group, all packages in the
same group have the same README file, so in this case it was copied from an
other package in the XIO group. These files were added to the packages in order
to address complaints about missing documentation.

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