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

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

--- Comment #6 from Toshio Ernie Kuratomi <a.badger@xxxxxxxxx> 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.
_______________________________________________
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]