Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: mapserver - Environment for building spatially-enabled internet applications https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=232792 ------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx 2007-05-10 11:40 EST ------- Well, for 4.10.2-1: * Dependency for main package: - Well, for unknown reason I didn't notice, however none of 4 subpackages have no dependency for main (mapserver) package. Please check if this is correct. IMO at least perl/python/java binding subpackage should have the release number dependent dependency for main package (i.e. should have: "Requires: %{name} = %{version}-%[release}"). * Source - The URL of the source returns "not found". Maybe: http://download.osgeo.org/mapserver/mapserver-4.10.2.tar.gz ? * (minor) Macros - /usr/sbin/ can be replaced with %{_sbindir} (preferred) - And in the line "mkdir -p %{buildroot}/etc/php.d" /etc should be %{_sysconfdir} Other things are okay. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review