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=749132 --- Comment #9 from Steve Traylen <steve.traylen@xxxxxxx> 2011-11-03 16:04:21 EDT --- Ricardo, If possible can you at least take the newer file from globus.org since is after globus switched to the apache license. Otherwise this package becomes 'ASL2.0 and Globus' where globus is now the old defunct Globus license which is today not fedora approved. There would be no problem getting it approved I'm sure if that's the only option. Mattias's suggestion to request globus exposes this header makes sense, could you request this so it may get fixed one day. New items or things I missed first time: (i) Can you parallelize the make? make %{?_smp_mflags} (ii) Source code does not match. Your instructions say to use tar -czvf dpm-dsi-1.8.2.tar.gz dpm-dsi-1.8.2 however your .src.rpm contains a misnamed tar file only $ file ../SOURCES/dpm-dsi-1.8.2.tar.gz ../SOURCES/dpm-dsi-1.8.2.tar.gz: POSIX tar archive (GNU) Moreover when I compare what is checkout vs what is in the tar ball they are different. $ diff --brief -r dpm-dsi-1.8.2 ../SOURCES/dpm-dsi-1.8.2 Only in ../SOURCES/dpm-dsi-1.8.2: config.status Only in ../SOURCES/dpm-dsi-1.8.2: Makefile i.e these files are only in the .src.rpm and not in the checkout. (iii) Redundant files %doc LICENSE RELEASE-NOTES are not needed in devel since it can't be installed without the main package. (iv) dpm-dsi-devel should probably Require (v) Reading the init.d script if [ `uname -m` = "x86_64" ]; then LD_LIBRARY_PATH=/opt/glite/lib64:/opt/lcg/lib64:$GLOBUS_LOCATION/lib else LD_LIBRARY_PATH=/opt/glite/lib:/opt/lcg/lib:$GLOBUS_LOCATION/lib fi export LD_LIBRARY_PATH The /opt directories have no place on FHS system, would better to junk it or at least case it so does not get used. Everything else from comment #2 is good. Other wise looking good. -- 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