Bug 176784
Summary: | Review Request: gnome-schedule: A GTK+ based user interface for cron and at | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Frank Arnold <frank> |
Component: | Package Review | Assignee: | John Mahowald <jpmahowald> |
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-extras-list |
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-02-27 14:20: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: | 163779 |
Description
Frank Arnold
2006-01-02 17:59:07 UTC
Looks pretty good. Built fine in Mock, though I'm wondering if you should have a BR on scrollkeeper (refer to http://fedoraproject.org/wiki/ScriptletSnippets#head-3c9f517f0cd4aaabb369a8805226d85dc2f02793) I'm unable to verify that everything runs as expected, since I'm not running Rawhide yet, but I'm not getting any errors with rpmlint on the binary rpm produced with Mock. Unfortunately, I can't sponser new members, so someone else will need to do an offical review for you. Thank you for your comments. Regarding BR on scrollkeeper: It's not needed here. In this case gnome-doc-utils is used for documentation building, which is already a BR. The only invoking of scrollkeeper without --disable-scrollkeeper is a scrollkeeper-update. Good: rpmlint of gnome-schedule-1.0.0-1.noarch.rpm:E: gnome-schedule only-non-binary-in-usr-lib bonobo, can be ignored. - package meets naming guidelines - package meets packaging guidelines - license ( GPL) OK, text in %doc, matches source - spec file legible, in am. english - source matches upstream - package compiles on devel (x86_64) - no missing BR - no unnecessary BR - locales handled by %find_lang - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - .desktop file in order - works Looks good. Send me your account details and I'll sponsor you. Thanks for the review and your sponsoring. |