Bug 193191
Summary: | Review Request: rename of notify-daemon to notification-daemon | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | John (J5) Palmieri <johnp> |
Component: | Package Review | Assignee: | David Cantrell <dcantrell> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | bdpepple, fedora-package-review, jkeck |
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-06-14 20:33: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: | 188268 |
Description
John (J5) Palmieri
2006-05-25 22:14:30 UTC
Couple of items: 1. Fails to build in Mock. Looks like it is due to a missing BR on GConf2-devel. 2. The Requires on gtk2 & dbus are unnecessary, since the devel packages sonames will pull these in. 3. SOURCE should be a full URL. 4. You can probably pass the --disable-static flag to the configure macro, and not even build the static libs since your just removing them anyhow. 5. You should probably use the gconf scriptlets from the wiki. http://fedoraproject.org/wiki/ScriptletSnippets#head-ff64cd482595764f672082d5a3b83e1fc22962e8 Fixed with suggestions. Spec URL: http://people.redhat.com/johnp/files/notification-daemon/notification-daemon.spec SRPM URL: http://people.redhat.com/johnp/files/notification-daemon/notification-daemon-0.3.5-2.src.rpm NEEDSWORK: - Buildroot needs to follow guideline - BuildRequires: autconf automake - Fails to build in mock: checking for NOTIFICATION_DAEMON... configure: error: Package requirements ( gtk+-2.0 >= 2.4.0, glib-2.0 >= 2.4.0, dbus-1 >= 0.36, dbus-glib-1 >= 0.36, gconf-2.0, libwnck-1.0 ) were not met: No package 'libwnck-1.0' found. Also, the manual Requires, is this because rpm require checking isn't getting it right? Fixed with suggestions. Not sure why Buildroot isn't just handled by RPM. Spec URL: http://people.redhat.com/johnp/files/notification-daemon/notification-daemon.spec SRPM URL: http://people.redhat.com/johnp/files/notification-daemon/notification-daemon-0.3.5-3.src.rpm THis was built into rawhide. |