Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 174065
Summary: | Review Request: jabberd server | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Adrian Reber <adrian> |
Component: | Package Review | Assignee: | Aurelien Bompard <gauret> |
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-extras-list, scop |
Target Milestone: | --- | Flags: | huzaifas:
fedora-cvs+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://jabberd.jabberstudio.org/2/ | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2005-11-28 07:38:23 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: | |||
Bug Depends On: | |||
Bug Blocks: | 159810, 163779 |
Description
Adrian Reber
2005-11-24 08:59:52 UTC
Looks good. A few comments : * File listed twice: /etc/jabberd/server.pem * Missing dependancy on /sbin/service for %post, %postun and %preun (initscripts package) * don't userdel jabber on removal, to avoid files belonging to nobody Jabberd is enabled at boot by default. We usually don't enable network-listening daemons by default. In the init script, just change this : -chkconfig: 2345 85 15 +chkconfig: - 85 15 The user can enable it with "chkconfig jabberd on" afterwards if needed. Thanks for the review. I have checked in the changes required for the comment #1 and for comment #2. Review for release 0.s10.8: * RPM name is OK * Source jabberd-2.0s10.tar.gz is the same as upstream * Builds fine in mock * rpmlint of jabberd looks OK * File list of jabberd looks OK * Works fine APPROVED. Funny thing: it crashes rpmlint. Ville, do you want to look at that ? Here's the binary rpm in case you can't reproduce: http://gauret.free.fr/fichiers/rpms/fedora/jabberd-2.0-0.s10.8.i386.rpm (In reply to comment #4) > Funny thing: it crashes rpmlint. Ville, do you want to look at that ? Looked, found the bug, fix will be in the next release. Thanks for the heads up. Thanks. jabberd has been successfully built in devel. Is this supposed to appear in the FC3/4 repositories? It has been built for devel only. > Is this supposed to appear in the FC3/4 repositories? It has been built for
> devel only.
Not is hasn't. This is also available for FC4.
New Package CVS Request ======================= Package Name: jabberd Short Description: OpenSource server implementation of the Jabber protocols Owners: adrian bernie Branches: F-10 InitialCC: jabberd is now branched for F-10 |