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: ganglia - Ganglia Distributed Monitoring System https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193846 jwilson@xxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bugzilla-sink@xxxxxxxxxxxxx |tibbs@xxxxxxxxxxx ------- Additional Comments From jwilson@xxxxxxxxxx 2006-06-14 00:49 EST ------- Slapping this back in, since its gone with the wind (and bugzilla db failure) otherwise... ------- Additional Comments From tibbs@xxxxxxxxxxx 2006-06-13 11:27 EST ------- Crap, I had a bunch of info typed up and then mozilla died for no reason. I'll try again.... This builds fine in mock on x86_64, development. rpmlint has this to say: W: ganglia-devel no-dependency-on ganglia Generally the -devel package depends on the main package, but in this case there is no main package. OK. W: ganglia-devel no-documentation This is not abnormal. OK. W: ganglia-gmetad no-documentation There is a manpage for gmetad; any reason not to package it? E: ganglia-gmetad non-standard-uid /var/lib/ganglia/rrds ganglia E: ganglia-gmetad non-standard-gid /var/lib/ganglia/rrds ganglia OK. W: ganglia-gmetad dangerous-command-in-%postun userdel Generally the user is not deleted when the package is removed; rpm may leave the config files around and they would become unowned if you do this. W: ganglia-gmetad incoherent-init-script-name gmetad W: ganglia-gmond incoherent-init-script-name gmond Generally it's best to try and keep the init files named after the package, but these are reasonably named and I don't think this is a blocker. W: ganglia-web no-documentation There does seem to be some documentation but it's in /usr/share/ganglia. Any reason why AUTHORS, COPYING and Changelog aren't %doc? Other oddities I noticed: /usr/share/ganglia/Makefile.am /usr/share/ganglia/version.php.in Why package these? What is /usr/lib64/libganglia-3.0.3.so.0.0.0 for? It's odd to see a versioned shared library in a -devel package, and nothing seems to link against it. If anything did, it would end up pulling in this -devel package, which would be odd. Still, it doesn't make much sense to split two files out of a five-file -devel package. Why not package the gmetric, gmond and gstat manpages in ganglia-gmond? -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review