ooops, s/CPU//

I don't have access to the .src.rpm.

I have many comments on the spec file (some are in fact blockers, some 
are really comments), though:

* I would personally have dropped 'for window managers such as WindowMaker'
  from the summary, since it will be shorter, and also those apps 
  are well suited to fluxbox, for example if I'm not wrong.
* the provide wmmemload is unusefull, it is automatically set by rpm
* The Epoch is not needed. In my opinion it is clearer if it is not 
  mentioned when set to 0
* the buildroot is not the preferred one
http://fedoraproject.org/wiki/Packaging/Guidelines#head-f196e7b2477c2f5dd97ef64e8eacddfb517f1aa1
  (although it has been agreed that %(%{__id_u} -n) could be removed).
* The Requires are not needed, there are picked up automatically by rpm
* BuildRequires: libX11-devel is optional since libX11-devel is required
  by libXext-devel or libXpm-devel
* export CFLAGS="$RPM_OPT_FLAGS" is unneeded, it is part of %configure
* if I'm not wrong, prefixing with %{_builddir}/%{name}-%{version}/ is 
  not needed in %install since it is what the current working directory 
  is set to.
* in %files, I think it is better to use
%{_mandir}/man1/wmmemload.1*
instead of
%{_mandir}/man1/wmmemload.1.gz
  to catch no compression and different compression schemes.
* in the changelog, I think the 0: corresponding with epoch is unneeded

(In reply to comment #2)
> I don't have access to the .src.rpm.
Sorry, the permissions got messed up while copying


> * I would personally have dropped 'for window managers such as WindowMaker'
>   from the summary, since it will be shorter, and also those apps 
>   are well suited to fluxbox, for example if I'm not wrong.
Agreed, fixed


> * the provide wmmemload is unusefull, it is automatically set by rpm
removed


> * The Epoch is not needed. In my opinion it is clearer if it is not 
>   mentioned when set to 0
removed


> * the buildroot is not the preferred one
>
http://fedoraproject.org/wiki/Packaging/Guidelines#head-f196e7b2477c2f5dd97ef64e8eacddfb517f1aa1
>   (although it has been agreed that %(%{__id_u} -n) could be removed).
not much of a difference, but no proble to fix :-)



> * The Requires are not needed, there are picked up automatically by rpm
oh, didn't know about that, thanks for the info, fixed



> * BuildRequires: libX11-devel is optional since libX11-devel is required
>   by libXext-devel or libXpm-devel
aaah, ok, removed


> * export CFLAGS="$RPM_OPT_FLAGS" is unneeded, it is part of %configure
removed


> * if I'm not wrong, prefixing with %{_builddir}/%{name}-%{version}/ is 
>   not needed in %install since it is what the current working directory 
>   is set to.
ok, removed



> * in %files, I think it is better to use
> %{_mandir}/man1/wmmemload.1*
> instead of
> %{_mandir}/man1/wmmemload.1.gz
>   to catch no compression and different compression schemes.
looking good to me, modified


> * in the changelog, I think the 0: corresponding with epoch is unneeded
fixed :-)

still have to upload the files together with the other package, I'll add another
comment when I do that tomorrow morning

Uploaded the fixed files:
http://www.kovalsky.cz/packages/wmmemload.spec
http://www.kovalsky.cz/packages/wmmemload-0.1.6-1.src.rpm

Thank you for the review notes!

Everytime you repost the package you are submitting after 
having made a change, you should bump the release and add
a changelog entry (just like how you would have done when
modifying a package), remake a src.rpm and repost it. There may
be exceptions for minor changes, but the rule is to repost a 
new srpm.

parallel make is preferred:
http://fedoraproject.org/wiki/Packaging/Guidelines#head-525c7d76890cb22df33b759c65c35c82bf434d2e

regarding the glob for the man pages
%{_mandir}/man1/wmmemload.1.*
won't catch the case when there is no compression, while
%{_mandir}/man1/wmmemload.1*
do. This is not a blocker.

During the build make triggers the whole autotools stuff. This is bad:

.......
+ make
cd . && aclocal
acinclude.m4:8: warning: underquoted definition of DA_CHECK_LIB
  run info '(automake)Extending aclocal'
  or see http://sources.redhat.com/automake/automake.html#Extending-aclocal
acinclude.m4:20: warning: underquoted definition of DA_CHECK_HEADER
cd . && automake --gnu --include-deps Makefile
cd . && autoconf
/bin/sh ./config.status --recheck
....

The files ChangeLog AUTHORS THANKS COPYING should go in %doc

What is the status of this submission?

Hi Patrice, 

wmmemload uses internal libdockapp as well as wmcpuload. After a long time I've
about rewriten wmcpuload to work with libdockap 0.6 which is already in extras,
this will need the same work.

I'll get right on it once I submit wmcpuload again.

Ok. As a side note I am the libdockapp maintainer in fedora 
(was needed for wmacpi update). Be sure to use the libdockapp api
for command line args processing to have the -w option that
allows to run in a window. 

Has there been any progress here?

No progress in three months; setting NEEDINFO.

Sorry, been swamped with other things over the last few months. I'll try to post
the work that I have done so far so if someone with spares cycles wants to
continue can do so. 

Any progress?  It's been another five months; perhaps this ticket should be closed.

Indeed, I can't find the time to invest the effort ATM.

Sorry.