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 167863
Summary: | Review Request: gpp - gnome-photo-printer | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Matt Domsch <matt_domsch> |
Component: | Package Review | Assignee: | Gérard Milmeister <gemi> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | David Lawrence <dkl> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://domsch.com/linux/fedora/extras/gpp | ||
Whiteboard: | |||
Fixed In Version: | 0.6.5-2.fc3 and .fc3 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2005-11-11 04:06:35 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
Matt Domsch
2005-09-09 01:15:19 UTC
- in the description there are a few too many spaces, e.g., "or Print ," - in the desktop file, use printer.png instead of full path; better yet would be a new icon, because the printer.png icon is pretty generic; maybe you concoct something from a printer icon and a photo icon - in the desktop file, name should be "Gnome Photo Printer" or simply "Photo Printer" or simply "Print Photos"; I prefer the second, so there is no issue with "Gnome" and "GNOME" :-) - in the desktop file, why is StartupNotify false? it seems to work with true - rpmlint is completely satisfied Thanks for the review. I fixed the description (the extra spaces are present in the original package's description), removed the full path to printer.png, changed .desktop Name="Photo Printer" and StartupNotify=true. New RPMs and spec posted to the same location. I did not create a new icon, as *any* existing icon would be better than what I could create myself. If someone wishes to create an icon for this package, I'll be happy to add it later. The example .desktop file in http://fedoraproject.org/wiki/PackagingGuidelines has StartupNotify=false. Perhaps this should be changed? (In reply to comment #2) > The example .desktop file in http://fedoraproject.org/wiki/PackagingGuidelines > has StartupNotify=false. Perhaps this should be changed? This is a safe default. For regular GNOME apps StartupNotify=true works. For KDE apps I don't know. APPROVED Committed to cvs and building now for FC3 and FC4. Closing. |