Bug 244752

Summary: Review Request: mod_dnssd - An Apache HTTPD module which adds Zeroconf support
Product: [Fedora] Fedora Reporter: Ignacio Vazquez-Abrams <ivazqueznet>
Component: Package ReviewAssignee: Ruben Kerkhof <ruben>
Status: CLOSED RAWHIDE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: lpoetter
Target Milestone: ---Flags: ruben: fedora-review+
wtogami: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-06-30 17:19:58 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Ignacio Vazquez-Abrams 2007-06-18 22:04:56 UTC
Spec URL: http://www.hotlinkfiles.com/files/85177_zolzc/mod_dnssd.spec
SRPM URL: 
http://www.hotlinkfiles.com/files/85178_0fghm/mod_dnssd-0.5-1.fc7.src.rpm
Description: mod_dnssd is an Apache HTTPD module which adds Zeroconf support via DNS-SD using Avahi.

Comment 1 Ruben Kerkhof 2007-06-20 18:36:37 UTC
Hi Ignacio,

Review for release 1:
* RPM name is OK
* Source mod_dnssd-0.5.tar.gz is the same as upstream
* This is the latest version
* Builds fine in mock
* rpmlint looks OK
* File list looks OK
* Config files of mod_dnssd looks OK
* I have been able to publish some virtual hosts, guided by the included README file. They showed up 
nicely in Safari on my Mac

Notes:
- Can you add LoadModule dnssd_module modules/mod_dnssd.so to mod_dnssd.conf?
- Publishing ~/user dirs didn't show up (public_html in my homedir)



Comment 2 Ignacio Vazquez-Abrams 2007-06-25 20:35:49 UTC
Updated.

http://www.hotlinkfiles.com/files/93587_vi1nn/mod_dnssd.spec
http://www.hotlinkfiles.com/files/93588_78oll/mod_dnssd-0.5-2.fc7.src.rpm

(In reply to comment #1)
> Notes:
> - Can you add LoadModule dnssd_module modules/mod_dnssd.so to mod_dnssd.conf?

Fixed.

> - Publishing ~/user dirs didn't show up (public_html in my homedir)

Even when uncommenting DNSSDAutoRegisterUserDir? Hrm. CCing upstream.

Comment 3 Ruben Kerkhof 2007-06-25 20:51:50 UTC
> Even when uncommenting DNSSDAutoRegisterUserDir? Hrm. CCing upstream

Yes, I tried to walk through the module with gdb, but it forks a child process early during httpd startup, 
and I couldn't step through it.

But anyway, the package itself looks good, and most of the functionality works, so it's approved.

Comment 4 Ignacio Vazquez-Abrams 2007-06-26 03:47:46 UTC
New Package CVS Request
=======================
Package Name: mod_dnssd
Short Description: An Apache HTTPD module which adds Zeroconf support
Owners: ivazqueznet
Branches: F-6, F-7
InitialCC: 

Comment 5 Kevin Fenzi 2007-06-26 17:09:37 UTC
cvs done.

Comment 6 Lennart Poettering 2007-07-24 21:55:42 UTC
Ignacio: thanks for packaging this. I'd love to be added as a CC for bugs.

Hmm, DNSSDAutoRegisterUserDir might not have worked on your machine due to a
couple of reasons:

1. You hit the trivial bug i fixed in upstream r63. (Ignacio, you probably want
to add this patch to your package!)

2. Your userdir is a symlink? 

3. Your userdir is not accessible for the user apache uses?

Most likely is reason #1.


Comment 7 Ignacio Vazquez-Abrams 2007-07-24 23:56:17 UTC
Package Change Request
======================
Package Name: mod_dnssd
Updated Fedora CC: lpoetter