[Bug 1028164] Review Request: globus-xio-gridftp-driver - Globus Toolkit - Globus XIO GridFTP Driver

[bugzilla at redhat.com]

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



--- Comment #2 from Mattias Ellert <mattias.ellert at fysast.uu.se> ---
(In reply to Adrien Devresse from comment #1)
> 
> Issues:
> =======
> - globus-core > 8 is already needed by globus-ftp-client-devel, no need of a
> direct dependency
> - globus-xio-devel%{?_isa} >= 3  of -devel package already included by 
> globus-xio-gsi-driver-devel%{?_isa} >= 2

These build dependencies are direct build dependencies. Using indirect
dependencies of other build dependencies to fulfil direct build dependences is
in most cases not a good idea, since indirect dependencies can change when the
version of the dependencies change. While it is true that the current version
of globus-xio-gsi-driver-devel has a dependency on globus-xio-devel >= 3 this
does not imply that all previous versions of globus-xio-gsi-driver-devel had
such a requirement, or that all future versions will have it.

> - depending on a doc package "globus-xio-doc >= 3" for compilation sounds
> wierd to me, Is it strictly needed  ?

Yes. Without it configure fails:

checking documentation dependencies... failed
configure: error: Documentation dependencies could not be satisfied.

> - Koji build on EPEL 5 ppc failed

You tried to build a SRPM generated on Fedora 19 on EPEL 5. This fails since
newer versions of rpm uses more modern digest algorithms that are not supported
by the older rpm version on EPEL 5. If this should work you need to force the
use of MD5 instead of the default.

New version with proper reference for the patch:

http://www.grid.tsl.uu.se/review/globus-xio-gridftp-driver-1.2-2.fc19.src.rpm
http://www.grid.tsl.uu.se/review/globus-xio-gridftp-driver.spec

(This srpm was generated using rpmbuild-md5 so sould build on EPEL 5)

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