Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: gkrellm - Multiple stacked system monitors in one process
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=197967
------- Additional Comments From ville.skytta(a)iki.fi 2006-07-10 15:57 EST -------
I'm a firm believer in that users/groups created by packages should *not* be
erased when the package is removed, and won't set myself as the reviewer because
of that. Let me know if you're willing to leave them behind when erasing
-daemon, and I'll take care of the review.
Anyway, a couple of comments:
The switch to lm_sensors means that ppc will need special treatment, as
lm_sensors is not (nor obviously -devel) available for it. ExcludeArch: ppc
would sound like a regression.
-daemon has grown a "Provides: gkrellm-server" for no apparent reason, and
-daemon has become self-obsoleting because of that, which may or may not cause
problems. I'd suggest removing it and adding a "<
$some_known_version-$release"
to Obsoletes: gkrellm-server.
There's a mismatch between %install and %files where the former uses various
hardcoded paths while the latter uses macros (at least /etc vs %{_sysconfdir}).
--
Configure bugmail:
https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.