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 192578 - Review Request: metamonitor - A Simple program that watches log files and popup its changes
Summary: Review Request: metamonitor - A Simple program that watches log files and pop...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michael J Knox
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-05-20 22:24 UTC by Hugo Cisneiros
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-22 21:23:57 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)

Description Hugo Cisneiros 2006-05-20 22:24:23 UTC 
Spec URL: http://www.devin.com.br/eitch/rpm/fedora/metamonitor.spec
SRPM URL: http://www.devin.com.br/eitch/rpm/fedora/5/SRPMS/metamonitor-0.4.5-1.src.rpm
Description:

metamonitor is a simple program written for KDE, which watches the syslog's or metalog's log file and pops up the window whenever the new message comes. You can specify the file to watch and a regular expression for parsing the log line, so you can watch other than log files too.
Comment 1 Parag AN(पराग) 2006-06-01 09:54:21 UTC 
Not a Review but some hints to improve Packaging
1) Got error when ran rpmlint on SRPM
E: metamonitor description-line-too-long can specify the file to watch and a
regular expression for parsing the log line,
Wrap the %description at 80 columns
2) A lot of compilation warnings
Comment 2 Hugo Cisneiros 2006-06-09 00:24:15 UTC 
Thanks for your comment.

Updated package:

Spec URL: http://www.devin.com.br/eitch/rpm/fedora/metamonitor.spec
SRPM URL: 
http://www.devin.com.br/eitch/rpm/fedora/5/SRPMS/metamonitor-0.4.5-2.src.rpm

Changes:

- Fixed description field size
- Applied patch (0) to modify a symlink to be relative
Comment 3 Michael J Knox 2006-07-21 09:16:46 UTC 
Hey.. I will review this submission...

----------------------------------------

Review for release 2:

* RPM name is OK
* Source metamonitor-0.4.5.tar.bz2 is the same as upstream
* Builds fine in mock
* File list looks OK

Needs work:
* rpmlint: need to fix rpmlint warnings

Minor:
* Duplicate BuildRequires: arts-devel (by kdelibs-devel), qt-devel (by
kdelibs-devel)


Notes:
rpmlint of metamonitor:
W: metamonitor dangling-relative-symlink
/usr/share/doc/HTML/en/metamonitor/common ../../en/common

Package looks good, fixup the above and I will give it the green light.
Comment 4 Hugo Cisneiros 2006-07-21 19:37:46 UTC 
Hi Michael,

Searching about the dangling link says that it could (and should) be ignored, 
it's not a problem, even if rpmlint warns against it. Many KDE apps have this 
warning with symlinks on their help files, including the base ones.

But I removed the BR as suggested. Thanks!
Comment 5 Michael J Knox 2006-07-22 19:47:41 UTC 
OK, none of those are blockers, so cosider this APPROVED. 

Please remember to close this review once the package has been imported into CVS
and built.
Comment 6 Hugo Cisneiros 2006-07-22 21:23:57 UTC 
Imported and built! Closing. Thanks.
Note You need to log in before you can comment on or make changes to this bug.