[Bug 749299] Review Request: lcgdm-dav - HTTP/DAV frontend to the DPM/LFC services

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

 



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 <steve.traylen@xxxxxxx> 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
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]