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 165487

Summary: Review Request: Wmapmload monitors your apm status in an lcd display fashion
Product: [Fedora] Fedora Reporter: Andreas Bierfert <andreas.bierfert>
Component: Package ReviewAssignee: José Matos <jamatos>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://tnemeth.free.fr/projets/dockapps.html
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-09-15 12:27:51 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 Andreas Bierfert 2005-08-09 19:09:46 UTC
Spec Name or Url: http://fedora.lowlatency.de/review/wmapmload.spec
SRPM Name or Url: http://fedora.lowlatency.de/review/wmapmload-0.3.4-1.src.rpm
Description: 
Wmapmload monitors your apm status in an lcd display fashion

Comment 1 José Matos 2005-09-12 07:47:16 UTC
Would you consider to rename the package to x11-applet-wmapmload? 
 
This kind of packages are really small, they are very usefull to small/older 
systems that do not support the heavier desktop environments (kde/gnome/xfce). 
 
In this case they are sufficiently small to be reasonable to have a single 
package containing them all. I know that this has lots of drawbacks and so 
I propose to give them a commom prefix. I suggest x11-applet but any other 
name that conveys the same meaning is OK with me. 
 
I have approved a previous package wmacpi from you and I propose to rename 
it as well. 
 
Comments? 

Comment 2 Andreas Bierfert 2005-09-12 08:23:07 UTC
I am not sure on how to deal with this. For me there is the conflict of original
name vs. prefix-name. I don't know which one is better. Maybe we can satisfy
both and make a virtual package which requires all dockapps that are part of FE.
This way we can still use the original names of the packages and also have a
nice and easy way to keep track of all the dockapps.

As to the naming of such a package? Maybe x11-dockapps? x11-applets? And if we
take this approach we might want to have some discussions about that as we would
need some sort of common process for this.

Comment 3 José Matos 2005-09-12 11:30:26 UTC
I will present these doubts in fedora packaging list and I will wait for some 
output before returning here. 
 
FWIW, the package looks OK. Nice work. 

Comment 4 José Matos 2005-09-15 12:26:56 UTC
+ the package builds in mock/x86_64   
+ rpmlint   
W: wmapmload no-version-in-last-changelog  
W: wmapmload-debuginfo no-version-in-last-changelog  
   
  These can be ignored.   
   
+ package name follows the guideline   
+ package follows packaging guidelines   
+ license is valid, matches upstream and is included  
+ spec file is legible and is written in American English   
+ source matches upstream   
+ Requires and BR OK   
+ files ownership OK  
  
Approved.  
  
One suggestion, replace the package version in Source by %{version}.  
It will save you some typing for future versions. :-)