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 173551 - Review Request: xfce4-notes-plugin
Summary: Review Request: xfce4-notes-plugin
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Kevin Fenzi
QA Contact: David Lawrence
URL: http://xfce-goodies.berlios.de/
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-11-18 01:55 UTC by Christoph Wickert
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2005-12-17 02:37:21 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Christoph Wickert 2005-11-18 01:55:08 UTC
Spec Name or Url: 
http://home.arcor.de/christoph.wickert/fedora/extras-review/SPECS/xfce4-notes-plugin.spec
SRPM Name or Url: 
http://home.arcor.de/christoph.wickert/fedora/extras-review/SRPMS/xfce4-notes-plugin-0.11.1-1.fc4.src.rpm
Description: 
This plugin provides sticky notes for your desktop. You can create a note by 
clicking on the customizable icon with the middle button of your mouse, 
show/hide the notes using the left one, edit the titlebar, change the notes 
background color and much more.

Comments:
rpmlint is clean
License GPL and included

Comment 1 Christoph Wickert 2005-12-01 13:25:12 UTC
- Add libxfcegui4-devel BuildReqs.
- Fix %defattr.

New SRPM:
http://home.arcor.de/christoph.wickert/fedora/extras-review/SRPMS/xfce4-notes-plugin-0.11.1-2.fc4.src.rpm


Comment 2 Kevin Fenzi 2005-12-02 03:55:55 UTC
good: 
- sources match upstream
- builds and works on fc4. 
- license ok. 
- rpmlint is clean. 
- builds fine in mock under devel. 

nits:
- NEWS file is pretty useless, might not include it. 

Everything looks good... APPROVED. 

Comment 3 Christoph Wickert 2005-12-17 02:37:21 UTC
Thanks Kevin.


Note You need to log in before you can comment on or make changes to this bug.