Bug 310631 - Review Request: wmnd - A network monitoring dock app for WindowMaker
Summary: Review Request: wmnd - A network monitoring dock app for WindowMaker
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-09-28 11:35 UTC by Steve
Modified: 2009-01-28 14:13 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-10-25 23:01:00 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Steve 2007-09-28 11:35:42 UTC
Spec URL: http://lonetwin.net/pkgs/wmnd.spec
SRPM URL: http://lonetwin.net/pkgs/wmnd-0.4.12-0.fc7.src.rpm
Description:

WMND (WindowMaker Network Devices) is a network monitoring dock app improved
and based on WMiFS 1.3b.

URL of original package: http://www.thregr.org/~wavexx/software/wmnd/

It displays various network interface stats in real time using a variety of different graphing options. Currently, there is no similar network monitoring dockapp for windowmaker. This is my first submission to Fedora (so, please be gentle, don't hurt me :)). I intend to submit a few more dockapps which I use myself.

Comment 1 Patrice Dumas 2007-10-05 20:32:16 UTC
The source should be an url allowing to download the tarball. 
In that case, something along:
http://www.thregr.org/~wavexx/software/wmnd/releases/wmnd-%{version}.tar.gz

At first sight the buildroot doesn't seems to be in the preferred ones.

This package depends on trend, that doesn't seems to be in fedora
http://www.thregr.org/~wavexx/software/trend/
it would be nice to package it too.

The CFLAGS are already set by %configure.

You can use
make install DESTDIR=...
if you go that way, you could add
INSTALL='install -p' on the make install command line to keep timestamp.
And you'll have to remove the example config file that is better shipped
as doc as you do.

make %{?_smp_mflags}
should be used unless it is broken.

I suggest that you install the rpmdevtools package and have 
a look at the template in /etc/rpmdevtools/spectemplate-minimal.spec 
Also some of the issue I post are explained in the guidelines
http://fedoraproject.org/wiki/Packaging/Guidelines

the package can use the snmp libs to have more functionalities.

The README.Fedora doesn't seems to be useful to me, the dockapp
will work out of the box as expected.

It would be nice to ask the copyright holder of drivers.c to
add a proper license notice. Not a blocker since it is clearly
intended to be GPL.

The license seems to me to be GPLv2+ and not GPLv2.

rpmlint:
wmnd.i386: W: no-version-in-last-changelog

Suggestions:
use
%{_mandir}/man1/wmnd.1*
to catch all the possible compressions (and no compression)

replace %defattr (-,root,root) with %defattr (-,root,root,-)


Comment 2 Rakesh Pandit 2008-09-03 16:29:07 UTC
@Steve
Needs an update soon, or will be closed?

Comment 4 Patrice Dumas 2008-10-25 23:01:00 UTC
Closing, it has been a very long time without activity.


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