Bug 198879

Summary: Review Request: kdnssd-avahi
Product: [Fedora] Fedora Reporter: Petr Rockai <prockai>
Component: Package ReviewAssignee: David Cantrell <dcantrell>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, rdieter
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-07-18 12:38:17 EDT Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 188268    

Description Petr Rockai 2006-07-14 07:22:21 EDT
Spec URL: http://kde-redhat.unl.edu/apt/kde-redhat/SPECS/kdnssd-avahi.spec
SRPM URL: http://kde-redhat.unl.edu/apt/kde-redhat/all/SRPMS.stable/kdnssd-avahi-0.1.2-6.20060713svn.src.rpm
Description: KDE zeroconf implementation based on avahi

The default upstream-provided zeroconf support in KDE (kdnssd) depends on Apple mDNSresponder, which is covered by APSL. The currently shipped KDE in FC has no zeroconf support whatsoever. This package, together with some changes to kdelibs package should provide KDE with zeroconf functionality.

Thanks go to Rex Dieter for working on this.
Comment 1 Bill Nottingham 2006-07-14 12:07:02 EDT
Ok from a tech persepctive.
Comment 2 Jesse Keating 2006-07-17 13:49:16 EDT
NEEDSWORK:
- prerelease package release tags start with 0., as per
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-d97a3f40b6dd9d2288206ac9bd8f1bf9b791b22a
- description for -devel should mention that it is development stuff.
- RPMLINT output

W: kdnssd-avahi-devel no-documentation
W: kdnssd-avahi strange-permission kdnssd-avahi-svn_checkout.sh 0755

First one is ignorable.  Second one as well.
Comment 3 Rex Dieter 2006-07-17 23:01:20 EDT
> prerelease package release tags start with 0.

It's not a pre-release, but a post-0.1.2-release svn checkout.
Comment 4 Petr Rockai 2006-07-18 06:56:49 EDT
Rex, could you please update the -devel description? I think that's the 
only complaint now. I'd like to get this in before devel freeze tomorrow, 
too...
Comment 5 Rex Dieter 2006-07-18 07:34:00 EDT
< - description for -devel should mention that it is development stuff.

*Other* than saying (as it is now):
Development files for kdnssd
?
Comment 6 Jesse Keating 2006-07-18 08:17:39 EDT
Er.. my bad.  I saw %description as %{summary} and thought it was the summary
from the main package, not from the devel package.

And since it is a snapshot, it is considered a "pre" release of the next release.

Comment 7 Rex Dieter 2006-07-18 09:47:31 EDT
%changelog
* Tue Jul 18 2006 Rex Dieter <rexdieter[AT]users.sf.net> 0.1.3-0.1.20060713svn
- since using snapshot, consider it a pre-release of next version


Spec URL: http://kde-redhat.unl.edu/apt/kde-redhat/SPECS/kdnssd-avahi.spec
SRPM URL:
http://kde-redhat.unl.edu/apt/kde-redhat/all/SRPMS.stable/kdnssd-avahi-0.1.3-0.1.20060713svn.src.rpm
Comment 8 Jesse Keating 2006-07-18 11:33:27 EDT
New version looks good.  All other things look good.

I assume that this will be a Requires of something else, and doesn't need its
own comps entry, correct?

Setting Peter to owner and adding to dist-fc6.  Please close when built into
rawhide.
Comment 9 Petr Rockai 2006-07-18 12:38:17 EDT
Built. I'm test-building kdelibs which Require the new kdnssd-avahi 
package through Provides: libkdnssd, it should go into rawhide as soon as 
it builds and installs on my test machine.