[Bug 580169] Review Request: loggerhead - Web viewer for the Bazaar version control system

[bugzilla at redhat.com]

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

--- Comment #6 from Toshio Ernie Kuratomi <a.badger at gmail.com> 2010-04-08 15:38:19 EDT ---
(In reply to comment #3)
> Thinking about mod_wsgi, it get it work some steps needed feels like should be
> done by the package itself (e.g. moving files to correct location).
> 
> I might be a good idea to create logggerhead-mod_wsgi subpackage with files
> in correct location and add deps on mod_wsgi. Comments?    

I thought about doing this but decided not to -- the files that must be moved
into position are sample configuration files so there is precedent for having
people create those in order to start a service.  The act of moving the files
into place causes the service to start which is something that we try to avoid
for services other than web apps.  The mod_wsgi scripts are not upstream yet. 
The service can be a dislcosure hazard as it will show regular files and
directories as well as bzr branches.

Pros of doing this: You get deps for mod_wsgi.  The last problem, I think I've
mitigated by making the config file not point to a http_path by default.  We
could add a comment in the loggerhead/bazaar.conf file that the
/etc/httpd/conf.d/loggerhead.conf file needs to be changed as well.

So I'm leaning towards not doing this... at least until the mod_wsgi script
gets integrated upstream but the major blockers have been dealt with.

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