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=749299 --- Comment #11 from Steve Traylen <[email protected]> 2011-12-07 14:35:19 EST --- Hi, The explicit %if %{?fedora}%{!?fedora:0} >= 10 || %{?rhel}%{!?rhel:0} >= 6 BuildRequires: libcurl-devel %else BuildRequires: curl-devel %endif on devel package is not needed, it's there on the main package which is good enough. Regarding the static html files /var/www/css /var/www/css/lcgdm-dav.css /var/www/icons /var/www/icons/dpm20.png these are a web application so move them to /usr/share/lcgdm-dav and update alias to use them. http://fedoraproject.org/wiki/Packaging:Guidelines#Web_Applications Also the httpd configuration takes of the web server with <Location /> DAV dpm </Location> so now everything is dpmdav regardless of what else is installed. could this be restricted to some default path prefix? Also the configuration contains: ProxyCache /var/proxycache this directory does not exist but more importantly it if is needed it should be in /var/lib/lcgdpm-dav or something. Steve. -- 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 [email protected] https://admin.fedoraproject.org/mailman/listinfo/package-review
